[Intel-gfx] [PATCH i-g-t 1/3] lib/igt_dummyload: libify checks for spin batch activation

[Chris Wilson]

Quoting Mika Kuoppala (2019-04-17 16:28:32)
> Instead of opencoding the poll into the spinner, use
> a helper to check if spinner has started.
> 
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Signed-off-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> ---
>  lib/igt_dummyload.c            | 35 +++++++++++++++++++---------------
>  lib/igt_dummyload.h            | 17 ++++++++++++++---
>  tests/i915/gem_ctx_exec.c      |  4 +---
>  tests/i915/gem_ctx_isolation.c |  4 ++--
>  tests/i915/gem_eio.c           |  4 ++--
>  tests/i915/gem_exec_latency.c  | 22 ++++++++++-----------
>  tests/i915/gem_exec_schedule.c |  5 ++---
>  tests/i915/gem_sync.c          | 28 ++++++++++++---------------
>  tests/perf_pmu.c               |  4 ++--
>  9 files changed, 66 insertions(+), 57 deletions(-)
> 
> diff --git a/lib/igt_dummyload.c b/lib/igt_dummyload.c
> index 47f6b92b..49b69737 100644
> --- a/lib/igt_dummyload.c
> +++ b/lib/igt_dummyload.c
> @@ -67,11 +67,13 @@ static pthread_mutex_t list_lock = PTHREAD_MUTEX_INITIALIZER;
>  
>  static void
>  fill_reloc(struct drm_i915_gem_relocation_entry *reloc,
> -          uint32_t gem_handle, uint32_t offset,
> +          uint32_t gem_handle, uint32_t offset, uint32_t delta,
>            uint32_t read_domains, uint32_t write_domains)
>  {
> +       reloc->presumed_offset = -1;

This is wrong, we do use the presumed_offset of 0.
-Chris