Re: [PATCH v2 1/5] test: fbt return0: work on quiet systems
Kris Van Hees <[email protected]>
| Newsgroups | dev.linux.lists.dtrace |
|---|---|
| Message-ID | <[email protected]> |
On Wed, Oct 08, 2025 at 12:16:42PM +0100, Nick Alcock wrote: > These two tests probe on sys_poll and just hope something calls it. > On sufficiently quiet systems, nothing does. > > Use hrtimer_nanosleep instead. > > Signed-off-by: Nick Alcock <[email protected]> Reviewed-by: Kris Van Hees <[email protected]> > --- > test/unittest/fbtprovider/tst.return0.d | 4 +++- > test/unittest/fbtprovider/tst.return0.r | 2 +- > test/unittest/providers/rawfbt/tst.return0.d | 4 +++- > test/unittest/providers/rawfbt/tst.return0.r | 2 +- > 4 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/test/unittest/fbtprovider/tst.return0.d b/test/unittest/fbtprovider/tst.return0.d > index 0b7091b320904..f06120a55e5d0 100644 > --- a/test/unittest/fbtprovider/tst.return0.d > +++ b/test/unittest/fbtprovider/tst.return0.d > @@ -11,10 +11,12 @@ > * SECTION: FBT Provider/Probe arguments > */ > > +/* @@trigger: periodic_output */ > + > #pragma D option quiet > #pragma D option statusrate=10ms > > -fbt::do_sys_poll:return > +fbt:vmlinux:hrtimer_nanosleep:return > /arg1 == 0/ > { > printf("%s %x returned 0", probefunc, arg0); > diff --git a/test/unittest/fbtprovider/tst.return0.r b/test/unittest/fbtprovider/tst.return0.r > index 53308c30f27ab..e4065e20062e0 100644 > --- a/test/unittest/fbtprovider/tst.return0.r > +++ b/test/unittest/fbtprovider/tst.return0.r > @@ -1 +1 @@ > -do_sys_poll ffffffffffffffff returned 0 > +hrtimer_nanosleep ffffffffffffffff returned 0 > diff --git a/test/unittest/providers/rawfbt/tst.return0.d b/test/unittest/providers/rawfbt/tst.return0.d > index 0146d684f1033..3a39740071c49 100644 > --- a/test/unittest/providers/rawfbt/tst.return0.d > +++ b/test/unittest/providers/rawfbt/tst.return0.d > @@ -9,10 +9,12 @@ > * ASSERTION: simple rawfbt provider arg0 and probefunc print test. > */ > > +/* @@trigger: periodic_output */ > + > #pragma D option quiet > #pragma D option statusrate=10ms > > -rawfbt::do_sys_poll:return > +fbt:vmlinux:hrtimer_nanosleep:return > /arg1 == 0/ > { > printf("%s %x returned 0", probefunc, arg0); > diff --git a/test/unittest/providers/rawfbt/tst.return0.r b/test/unittest/providers/rawfbt/tst.return0.r > index 53308c30f27ab..e4065e20062e0 100644 > --- a/test/unittest/providers/rawfbt/tst.return0.r > +++ b/test/unittest/providers/rawfbt/tst.return0.r > @@ -1 +1 @@ > -do_sys_poll ffffffffffffffff returned 0 > +hrtimer_nanosleep ffffffffffffffff returned 0 > > base-commit: 877fcdeb25b370be182d51f27fb4bd8e4f739fb1 > -- > 2.51.0.284.g117bcb8de7 >