[v1] tests/qtest: Fix npcm7xx_timer-test.c flaky test

[PATCH] tests/qtest: Fix npcm7xx_timer-test.c flaky test

Chris Rauer posted 1 patch 7 months, 1 week ago

Diff against v2
Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230928034505.255480-1-crauer@google.com

Maintainers: Tyrone Ting <kfting@nuvoton.com>, Hao Wu <wuhaotsh@google.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>

There is a newer version of this series

tests/qtest/npcm7xx_timer-test.c | 1 +
1 file changed, 1 insertion(+)

Expand all Fold all

[PATCH] tests/qtest: Fix npcm7xx_timer-test.c flaky test

Posted by Chris Rauer 7 months, 1 week ago

Fixes: https://gitlab.com/qemu-project/qemu/-/issues/1897
Signed-off-by: Chris Rauer <crauer@google.com>
---
 tests/qtest/npcm7xx_timer-test.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/qtest/npcm7xx_timer-test.c b/tests/qtest/npcm7xx_timer-test.c
index 43711049ca..58f58c2f71 100644
--- a/tests/qtest/npcm7xx_timer-test.c
+++ b/tests/qtest/npcm7xx_timer-test.c
@@ -465,6 +465,7 @@ static void test_periodic_interrupt(gconstpointer test_data)
     int i;
 
     tim_reset(td);
+    clock_step_next();
 
     tim_write_ticr(td, count);
     tim_write_tcsr(td, CEN | IE | MODE_PERIODIC | PRESCALE(ps));
-- 
2.42.0.515.g380fc7ccd1-goog

Re: [PATCH] tests/qtest: Fix npcm7xx_timer-test.c flaky test

Posted by Thomas Huth 7 months, 1 week ago

On 28/09/2023 05.45, Chris Rauer wrote:

Could you please add a proper patch description how this is fixing the issue?

  Thanks,
   Thomas


> Fixes: https://gitlab.com/qemu-project/qemu/-/issues/1897
> Signed-off-by: Chris Rauer <crauer@google.com>
> ---
>   tests/qtest/npcm7xx_timer-test.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/qtest/npcm7xx_timer-test.c b/tests/qtest/npcm7xx_timer-test.c
> index 43711049ca..58f58c2f71 100644
> --- a/tests/qtest/npcm7xx_timer-test.c
> +++ b/tests/qtest/npcm7xx_timer-test.c
> @@ -465,6 +465,7 @@ static void test_periodic_interrupt(gconstpointer test_data)
>       int i;
>   
>       tim_reset(td);
> +    clock_step_next();
>   
>       tim_write_ticr(td, count);
>       tim_write_tcsr(td, CEN | IE | MODE_PERIODIC | PRESCALE(ps));

Re: [PATCH] tests/qtest: Fix npcm7xx_timer-test.c flaky test

Posted by Chris Rauer 7 months, 1 week ago

Ack.  will send out v2.

On Wed, Sep 27, 2023 at 9:44 PM Thomas Huth <thuth@redhat.com> wrote:

> On 28/09/2023 05.45, Chris Rauer wrote:
>
> Could you please add a proper patch description how this is fixing the
> issue?
>
>   Thanks,
>    Thomas
>
>
> > Fixes: https://gitlab.com/qemu-project/qemu/-/issues/1897
> > Signed-off-by: Chris Rauer <crauer@google.com>
> > ---
> >   tests/qtest/npcm7xx_timer-test.c | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/qtest/npcm7xx_timer-test.c
> b/tests/qtest/npcm7xx_timer-test.c
> > index 43711049ca..58f58c2f71 100644
> > --- a/tests/qtest/npcm7xx_timer-test.c
> > +++ b/tests/qtest/npcm7xx_timer-test.c
> > @@ -465,6 +465,7 @@ static void test_periodic_interrupt(gconstpointer
> test_data)
> >       int i;
> >
> >       tim_reset(td);
> > +    clock_step_next();
> >
> >       tim_write_ticr(td, count);
> >       tim_write_tcsr(td, CEN | IE | MODE_PERIODIC | PRESCALE(ps));
>
>