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 v5] test: sched_pktio: remove unnecessary delay first tick #1877

Merged

Conversation

PavanNikhilesh
Copy link
Contributor

@PavanNikhilesh PavanNikhilesh commented Aug 11, 2023

Remove unnecessary delay for the first tick as it might cause
ODP_TIMER_TOO_FAR error base on the timer pool created.
Reorder start_pktios with start_timers to prevent timeouts due
to delays in pktio start.

Signed-off-by: Pavan Nikhilesh pbhagavatula@marvell.com
Reviewed-by: Matias Elo matias.elo@nokia.com

@odpbuild odpbuild changed the title test: sched_pktio: remove unnecessary delay first tick [PATCH v1] test: sched_pktio: remove unnecessary delay first tick Aug 11, 2023
@odpbuild odpbuild changed the title [PATCH v1] test: sched_pktio: remove unnecessary delay first tick [PATCH v2] test: sched_pktio: remove unnecessary delay first tick Aug 22, 2023
@PavanNikhilesh PavanNikhilesh changed the title [PATCH v2] test: sched_pktio: remove unnecessary delay first tick test: sched_pktio: remove unnecessary delay first tick Aug 22, 2023
@PavanNikhilesh PavanNikhilesh changed the title test: sched_pktio: remove unnecessary delay first tick [PATCH v2] test: sched_pktio: remove unnecessary delay first tick Aug 22, 2023
@MatiasElo MatiasElo self-requested a review August 23, 2023 14:18
@odpbuild odpbuild changed the title [PATCH v2] test: sched_pktio: remove unnecessary delay first tick [PATCH v3] test: sched_pktio: remove unnecessary delay first tick Aug 23, 2023
Remove unnecessary delay for the first tick as it might cause
ODP_TIMER_TOO_FAR error base on the timer pool created.
Reorder start_pktios with start_timers to prevent timeouts due
to delays in pktio start.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
@PavanNikhilesh PavanNikhilesh changed the title [PATCH v3] test: sched_pktio: remove unnecessary delay first tick [PATCH v4] test: sched_pktio: remove unnecessary delay first tick Aug 23, 2023
@odpbuild odpbuild changed the title [PATCH v4] test: sched_pktio: remove unnecessary delay first tick [PATCH v5] test: sched_pktio: remove unnecessary delay first tick Aug 23, 2023
@MatiasElo MatiasElo merged commit 1edb4b4 into OpenDataPlane:master Aug 24, 2023
174 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