Re: [PATCH v2 1/5] test: fbt return0: work on quiet systems

[Kris Van Hees <kris.van.hees@oracle.com>]

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 <nick.alcock@oracle.com>

Reviewed-by: Kris Van Hees <kris.van.hees@oracle.com>

> ---
>  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
>