Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #1281, Set RTEMS task name for cpuuse #1407

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Fix #1281, Set RTEMS task name for cpuuse #1407

merged 1 commit into from
Sep 13, 2023

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Aug 25, 2023

Checklist (Please check before submitting)

Describe the contribution

Testing performed
Ran on RTEMS 5. Sample output:

-------------------------------------------------------------------------------
                              CPU USAGE BY THREAD
------------+----------------------------------------+---------------+---------
 ID         | NAME                                   | SECONDS       | PERCENT
------------+----------------------------------------+---------------+---------
 0x09010001 | IDLE                                   |      6.924443 |  86.332
 0x0a010001 | UI1                                    |      0.000000 |   0.000
 0x0a010002 | ntwk                                   |      0.002736 |   0.034
 0x0a010003 | ETH0                                   |      0.000000 |   0.000
 0x0a010004 | FTPa                                   |      0.000000 |   0.000
 0x0a010005 | FTPD                                   |      0.000000 |   0.000
 0x0a010007 | cFS                                    |      0.000000 |   0.000
 0x0a010008 |                                        |      0.000213 |   0.002
 0x0a010009 |                                        |      0.330194 |   4.115
 0x0a01000a | BSWP                                   |      0.003068 |   0.038
 0x0a01000b | BRDA                                   |      0.010960 |   0.136
 0x0a01000c | CFE_EVS                                |      0.001207 |   0.015
 0x0a01000d | shel                                   |      0.015485 |   0.193
 0x0a01000e | CFE_SB                                 |      0.001240 |   0.015
 0x0a01000f | CFE_ES                                 |      0.035894 |   0.447
 0x0a010010 | ES_BG_TASK                             |      0.001430 |   0.017
 0x0a010011 | CFE_TIME                               |      0.009163 |   0.114
 0x0a010012 | TIME_TONE_TASK                         |      0.005439 |   0.067
 0x0a010013 | TIME_1HZ_TASK                          |      0.003104 |   0.038
 0x0a010014 | CFE_TBL                                |      0.001444 |   0.018
 0x0a010015 | FIFO_LIB_ChildT                        |      0.011508 |   0.143
 0x0a010016 | CI_LAB_APP                             |      0.005689 |   0.070
 0x0a010017 | TO_LAB_APP                             |      0.012372 |   0.154
 0x0a010018 | SCH_LAB_APP                            |      0.003868 |   0.048
 0x0a010019 | HW                                     |      0.023436 |   0.291
 0x0a01001a | CF                                     |      0.002247 |   0.028
 0x0a01001b | DS                                     |      0.025332 |   0.315
 0x0a01001c | FM                                     |      0.015175 |   0.189
 0x0a01001d | FM_CHILD_TASK                          |      0.029992 |   0.373
 0x0a01001e | LC                                     |      0.021521 |   0.268
 0x0a01001f | SC                                     |      0.025404 |   0.316
 0x0a010020 | VENDI                                  |      0.019558 |   0.243
 0x0a010021 | DP                                     |      0.481416 |   5.996
------------+----------------------------------------+---------------+---------
 TIME SINCE LAST CPU USAGE RESET IN SECONDS:                          8.028488
-------------------------------------------------------------------------------

Expected behavior changes
Reports task names (and doesn't conflict with classic task names used for OSAL taskids)

System(s) tested on

  • Leon3 QEMU
  • OS: RTEMS 5
  • Versions: basically main w/ local RTEMS updates

Additional context
Ping @thesamprice
Discussed at recent CCB, note adds posix/pthread dependency

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Aug 25, 2023
@chillfig chillfig added CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Sep 7, 2023
@dzbaker dzbaker mentioned this pull request Sep 13, 2023
Merged
2 tasks
dzbaker added a commit to nasa/cFS that referenced this pull request Sep 13, 2023
@dzbaker
IC: Caelum-rc4+dev58
db0814a 
*Combines:*

cFE v7.0.0-rc4+dev384
osal v6.0.0-rc4+dev229

**Includes:**

*cFE*
- nasa/cFE#2437

*osal*
- nasa/osal#1407
- nasa/osal#1408

Co-authored by: Justin Figueroa <chillfig@users.noreply.github.com>
Co-authored by: Jacob Hageman  <skliper@users.noreply.github.com>
@dzbaker dzbaker mentioned this pull request Sep 13, 2023
Merged
2 tasks
@dzbaker dzbaker merged commit 1401352 into nasa:main Sep 13, 2023
20 checks passed
dzbaker added a commit to nasa/cFS that referenced this pull request Sep 13, 2023
@dzbaker
IC: Caelum-rc4+dev58
1e94d84 
*Combines:*

cFE v7.0.0-rc4+dev384
osal v6.0.0-rc4+dev229

**Includes:**

*cFE*
- nasa/cFE#2437

*osal*
- nasa/osal#1407
- nasa/osal#1408

Co-authored by: Justin Figueroa <chillfig@users.noreply.github.com>
Co-authored by: Jacob Hageman  <skliper@users.noreply.github.com>
@skliper skliper mentioned this pull request Sep 25, 2023
Closed
@skliper skliper deleted the fix1281-set_rtems_task_name branch April 1, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rtems should show task names when typing cpuuse
3 participants
@skliper @dzbaker @chillfig