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 v3] speed up testing #1996

Merged
merged 8 commits into from
Jan 22, 2024

Conversation

JannePeltonen
Copy link
Collaborator

Speed up 'make check' by making certain tests (mainly timer validation and dma perf tests) complete faster.

@odpbuild odpbuild changed the title speed up testing [PATCH v1] speed up testing Jan 16, 2024
Copy link
Collaborator

@JereLeppanen JereLeppanen left a comment

Choose a reason for hiding this comment

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

I'm not sure how much time these changes actually save, but looks okay to me.

@odpbuild odpbuild changed the title [PATCH v1] speed up testing [PATCH v2] speed up testing Jan 17, 2024
@JannePeltonen
Copy link
Collaborator Author

I'm not sure how much time these changes actually save

After this 'make check' takes ~22 % less time in my system. PR 1998 will take another ~20 % away.

v2: rebased and added reviewed-by tag.

Increase timer resolution and shorten various timer intervals
to make the test faster.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Lower the number of scheduling rounds during 'make check' to make
'make check' a bit faster.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Use floating point arithmetic when truncation to integer may cause
unexpected rounding errors. This also makes sub-second measurement
intervals possible.

Fix also the kilo prefix (k) in statistics printout.
Reviewed-by: Jere Leppänen <[email protected]>

Signed-off-by: Janne Peltonen <[email protected]>
Allow decimal run time parameter to enable sub-second run times.
Also reduce the wait time for queue cleanup at the end.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Reduce test run time during make check.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Allow decimal run time parameter to enable sub-second run times.
Also reduce the wait time for queue cleanup at the end.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Reduce test run time during make check.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
Speed up the program slightly by reducing the wait time for
scheduled queue draining.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Jere Leppänen <[email protected]>
@odpbuild odpbuild changed the title [PATCH v2] speed up testing [PATCH v3] speed up testing Jan 22, 2024
@MatiasElo MatiasElo enabled auto-merge (rebase) January 22, 2024 08:06
@MatiasElo MatiasElo merged commit 58c5733 into OpenDataPlane:master Jan 22, 2024
171 checks passed
@JannePeltonen JannePeltonen deleted the more-test-speedup branch January 22, 2024 11:11
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.

3 participants