Re: [PATCH i-g-t 11/16] i915/gem_exec_whisper: debugfs/next_seqno is defunct

Tvrtko Ursulin
2019-05-14T12:48:26Z

On 08/05/2019 11:09, Chris Wilson wrote:
> We removed next_seqno in 5.1, so time to wave goodbye.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
> tests/i915/gem_exec_whisper.c | 12 ------------
> 1 file changed, 12 deletions(-)
>
> diff --git a/tests/i915/gem_exec_whisper.c b/tests/i915/gem_exec_whisper.c
> index d5afc8119..61b8d6dac 100644
> --- a/tests/i915/gem_exec_whisper.c
> +++ b/tests/i915/gem_exec_whisper.c
> @@ -44,15 +44,6 @@
>
> #define VERIFY 0
>
> -static void write_seqno(int dir, unsigned offset)
> -{
> - uint32_t seqno = UINT32_MAX - offset;
> -
> - igt_sysfs_printf(dir, "i915_next_seqno", "0x%x", seqno);
> -
> - igt_debug("next seqno set to: 0x%x\n", seqno);
> -}
> -
> static void check_bo(int fd, uint32_t handle, int pass)
> {
> uint32_t *map;
> @@ -355,9 +346,6 @@ static void whisper(int fd, unsigned engine, unsigned flags)
> igt_until_timeout(150) {
> uint64_t offset;
>
> - if (nchild == 1)
> - write_seqno(debugfs, pass);
> -
> if (flags & HANG)
> submit_hang(&hang, engines, nengine, flags);
>
>

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Regards,

Tvrtko