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.

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

Shorter name for ES_BackgroundTask #976

Closed
jphickey opened this issue Oct 28, 2020 · 0 comments · Fixed by #977 or #991
Closed

Shorter name for ES_BackgroundTask #976

jphickey opened this issue Oct 28, 2020 · 0 comments · Fixed by #977 or #991
Assignees
Milestone

Comments

@jphickey
Copy link
Contributor

Describe the bug
With the recent addition of setting the name at the Linux OS kernel in nasa/PSP#199, task names are mostly shown in the debugger but not for the ES background task. This is because the glibc/Linux has a hard limit of 16 characters for this name. So even though OS_MAX_API_NAME is 20, names need to be less than 16 chars in order to show up properly.

To Reproduce
Run CFE in debugger and look at task names - "ES_BackgroundTask" does not show up.

Expected behavior
Should show the name.

System observed on:
Ubuntu 20.04

Additional context
Recommend keeping CFE task names to be all less than 16 chars even if OS_MAX_API_NAME is set to a larger limit. OSAL is fine with longer names, but they just don't show up in the debugger due to the underlying glibc/linux limit.

Reporter Info
Joseph Hickey, Vantage Systems, Inc.

jphickey added a commit to jphickey/cFE that referenced this issue Oct 28, 2020
Keep names under 16 chars to make more debugger friendly, regardless
of the OSAL limit.
@jphickey jphickey self-assigned this Oct 28, 2020
astrogeco added a commit that referenced this issue Oct 30, 2020
@astrogeco astrogeco added this to the 7.0.0 milestone Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants