2019-08-27 12:31:38

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: dma-buf: Add selftests for dma-fence

Hi Chris,

When running the new dmabuf-selftests on two different systems, I get:

dma-buf: Running sanitycheck
dma-buf: Running dma_fence
sizeof(dma_fence)=48
dma-buf: Running dma_fence/sanitycheck
dma-buf: Running dma_fence/test_signaling
dma-buf: Running dma_fence/test_add_callback
dma-buf: Running dma_fence/test_late_add_callback
dma-buf: Running dma_fence/test_rm_callback
dma-buf: Running dma_fence/test_late_rm_callback
dma-buf: Running dma_fence/test_status
dma-buf: Running dma_fence/test_error
dma-buf: Running dma_fence/test_wait
dma-buf: Running dma_fence/test_wait_timeout
dma-buf: Running dma_fence/test_stub
dma-buf: Running dma_fence/race_signal_callback
thread_signal_callback[0] completed 28855 passes, 3929 misses
thread_signal_callback[1] completed 28846 passes, 3918 misses
thread_signal_callback[0] completed 37179 passes, 37179 misses
thread_signal_callback[1] completed 37184 passes, 37183 misses

and

dma-buf: Running sanitycheck
dma-buf: Running dma_fence
sizeof(dma_fence)=64
dma-buf: Running dma_fence/sanitycheck
dma-buf: Running dma_fence/test_signaling
dma-buf: Running dma_fence/test_add_callback
dma-buf: Running dma_fence/test_late_add_callback
dma-buf: Running dma_fence/test_rm_callback
dma-buf: Running dma_fence/test_late_rm_callback
dma-buf: Running dma_fence/test_status
dma-buf: Running dma_fence/test_error
dma-buf: Running dma_fence/test_wait
dma-buf: Running dma_fence/test_wait_timeout
dma-buf: Running dma_fence/test_stub
dma-buf: Running dma_fence/race_signal_callback
thread_signal_callback[0] completed 3423 passes, 2148 misses
thread_signal_callback[1] completed 2360 passes, 9 misses
thread_signal_callback[0] completed 4028 passes, 4028 misses
thread_signal_callback[1] completed 8080 passes, 8079 misses

Unfortunately it is not clear to me if this is good or bad?
Perhaps the test output can be improved, e.g. with a clear PASS/FAIL output?

Thanks!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


2019-08-27 12:37:58

by Chris Wilson

[permalink] [raw]
Subject: Re: dma-buf: Add selftests for dma-fence

Quoting Geert Uytterhoeven (2019-08-27 13:30:04)
> Hi Chris,
>
> When running the new dmabuf-selftests on two different systems, I get:
>
> dma-buf: Running sanitycheck
> dma-buf: Running dma_fence
> sizeof(dma_fence)=48
> dma-buf: Running dma_fence/sanitycheck
> dma-buf: Running dma_fence/test_signaling
> dma-buf: Running dma_fence/test_add_callback
> dma-buf: Running dma_fence/test_late_add_callback
> dma-buf: Running dma_fence/test_rm_callback
> dma-buf: Running dma_fence/test_late_rm_callback
> dma-buf: Running dma_fence/test_status
> dma-buf: Running dma_fence/test_error
> dma-buf: Running dma_fence/test_wait
> dma-buf: Running dma_fence/test_wait_timeout
> dma-buf: Running dma_fence/test_stub
> dma-buf: Running dma_fence/race_signal_callback
> thread_signal_callback[0] completed 28855 passes, 3929 misses
> thread_signal_callback[1] completed 28846 passes, 3918 misses
> thread_signal_callback[0] completed 37179 passes, 37179 misses
> thread_signal_callback[1] completed 37184 passes, 37183 misses
>
> and
>
> dma-buf: Running sanitycheck
> dma-buf: Running dma_fence
> sizeof(dma_fence)=64
> dma-buf: Running dma_fence/sanitycheck
> dma-buf: Running dma_fence/test_signaling
> dma-buf: Running dma_fence/test_add_callback
> dma-buf: Running dma_fence/test_late_add_callback
> dma-buf: Running dma_fence/test_rm_callback
> dma-buf: Running dma_fence/test_late_rm_callback
> dma-buf: Running dma_fence/test_status
> dma-buf: Running dma_fence/test_error
> dma-buf: Running dma_fence/test_wait
> dma-buf: Running dma_fence/test_wait_timeout
> dma-buf: Running dma_fence/test_stub
> dma-buf: Running dma_fence/race_signal_callback
> thread_signal_callback[0] completed 3423 passes, 2148 misses
> thread_signal_callback[1] completed 2360 passes, 9 misses
> thread_signal_callback[0] completed 4028 passes, 4028 misses
> thread_signal_callback[1] completed 8080 passes, 8079 misses
>
> Unfortunately it is not clear to me if this is good or bad?
> Perhaps the test output can be improved, e.g. with a clear PASS/FAIL output?

It's communicated via the error code, along with failure messages. For
more complex persistent selftest modules, we use an modparam to report
the error code.

See igt/dmabuf for the test runner.
-Chris