On Mon, Jul 27, 2020 at 12:22 PM Ruifeng Wang <ruifeng.w...@arm.com> wrote:
>
> test_delay_us_sleep registers sleep based delay for testing.
> This changes the default delay function of testing environment.
> It is not expected.
>
> Restore default delay function after the test to fix the issue.
>
> Fixes: a51639cc720a ("eal: add nanosleep based delay function")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Ruifeng Wang <ruifeng.w...@arm.com>
> Reviewed-by: Gavin Hu <gavin...@arm.com>
> ---
> v2:
> Dropped other patches in the series that are not needed.
>
> app/test/test_cycles.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/app/test/test_cycles.c b/app/test/test_cycles.c
> index c78e6a5b1..015a9290f 100644
> --- a/app/test/test_cycles.c
> +++ b/app/test/test_cycles.c
> @@ -79,8 +79,14 @@ REGISTER_TEST_COMMAND(cycles_autotest, test_cycles);
> static int
> test_delay_us_sleep(void)
> {
> + int rv = 0;
No initialisation needed, I removed it while applying.
> +
> rte_delay_us_callback_register(rte_delay_us_sleep);
> - return check_wait_one_second();
> + rv = check_wait_one_second();
> + /* restore original delay function */
> + rte_delay_us_callback_register(rte_delay_us_block);
> +
> + return rv;
> }
>
> REGISTER_TEST_COMMAND(delay_us_sleep_autotest, test_delay_us_sleep);
Reviewed-by: David Marchand <david.march...@redhat.com>
Applied, thanks.
--
David Marchand
