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

Confirm valid PERF_ID use #1

Closed
skliper opened this issue Jun 24, 2019 · 4 comments · Fixed by #13
Closed

Confirm valid PERF_ID use #1

skliper opened this issue Jun 24, 2019 · 4 comments · Fixed by #13
Assignees
Labels
bug Something isn't working
Milestone

Comments

@skliper
Copy link
Contributor

skliper commented Jun 24, 2019

Originated by abrown4 (69 on babelfish):

cFE reserves 0-31 perf-ids. Need to confirm ci_lab does not use these.

@skliper skliper added the bug Something isn't working label Jun 24, 2019
@skliper skliper added this to the 2.3.0 milestone Jun 24, 2019
@avan989
Copy link
Contributor

avan989 commented Aug 26, 2019

From looking at ci_lab_perfids.h, ci_lab contains these id.

#define CI_MAIN_TASK_PERF_ID 21
#define CI_SOCKET_RCV_PERF_ID 24

@skliper
Copy link
Contributor Author

skliper commented Aug 26, 2019

Could you record here what the other apps are using also? They all need to be unique and not in the 0-31 range.

@avan989
Copy link
Contributor

avan989 commented Aug 26, 2019

  1. to_lab_perfids.h
    #define TO_MAIN_TASK_PERF_ID 22
    #define TO_SOCKET_SEND_PERF_ID 23
  2. sch_lab_perfids.h
    #define SCH_MAIN_TASK_PERF_ID 20
  3. sample_app_perfids.h
    #define SAMPLE_APP_PERF_ID 91
  4. ci_lab_perfids.h
    #define CI_MAIN_TASK_PERF_ID 21
    #define CI_SOCKET_RCV_PERF_ID 24

@skliper
Copy link
Contributor Author

skliper commented Aug 26, 2019

Valid range is 32 to 127. Reassign ci_lab, sch_lab and to_lab starting at 32.

skliper added a commit that referenced this issue Sep 10, 2019
skliper added a commit that referenced this issue Sep 10, 2019
Reviewed and approved at 2019-09-04 CCB
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants