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

[PATCH v2] test: sched_perf: add user area touch #1999

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

psavol
Copy link
Collaborator

@psavol psavol commented Jan 16, 2024

Added options to read (-u) and modify (-U) user area content per event reception.

@odpbuild odpbuild changed the title test: sched_perf: add user area touch [PATCH v1] test: sched_perf: add user area touch Jan 16, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyright can be updated.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated in v2.

Comment on lines 979 to 981
uint32_t uarea_size = test_options->uarea_size;
uint32_t uarea_rd = test_options->uarea_rd;
uint32_t uarea_rw = test_options->uarea_rw;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be const.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in v2.

Added options to read (-u) and modify (-U) user area content
per event reception.

Signed-off-by: Petri Savolainen <[email protected]>
Reviewed-by: Matias Elo <[email protected]>
@psavol psavol force-pushed the test-sched-perf-userarea branch from 2b80933 to a028aa2 Compare January 22, 2024 10:30
@odpbuild odpbuild changed the title [PATCH v1] test: sched_perf: add user area touch [PATCH v2] test: sched_perf: add user area touch Jan 22, 2024
@psavol psavol merged commit 5eca9c4 into OpenDataPlane:master Jan 22, 2024
171 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants