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: performance: refactor bench applications #1907

Merged
merged 4 commits into from
Sep 20, 2023

Conversation

MatiasElo
Copy link
Collaborator

Move common code from odp_bench_* applications to bench_common.h/c.

@odpbuild odpbuild changed the title test: performance: refactor bench applications [PATCH v1] test: performance: refactor bench applications Sep 18, 2023
Utilize common data types among all odp_bench_* applications. Reduces the
amount of duplicate code.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Use common indefinite test runner function bench_run_indef() for all
odp_bench_* applications. Reduces the amount of duplicate code.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Use common test runner function bench_run() for all odp_bench_*
applications. Reduces the amount of duplicate code.

A separate warm-up round has been removed from odp_bench_packet, as
bench_run() already performs one.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Use the same 'rounds' option with all odp_bench_* applications.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@odpbuild odpbuild changed the title [PATCH v1] test: performance: refactor bench applications [PATCH v2] test: performance: refactor bench applications Sep 20, 2023
@MatiasElo MatiasElo merged commit c56cf83 into OpenDataPlane:master Sep 20, 2023
174 checks passed
@MatiasElo MatiasElo deleted the dev/bench-common branch September 21, 2023 12:02
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