[igt-dev] [PATCH i-g-t 1/2] tests/i915/perf_pmu: Add softpin flag for no-reloc path

[Kamil Konieczny]

On 2022-07-19 at 09:24:50 +0200, Zbigniew Kempczyński wrote:
> As Chris noticed semaphore object doesn't have EXEC_OBJECT_PINNED flag.
> It is likely not a problem as we provide offsets from reloc allocator
> thus there's no risk of overlapping offsets. But for purity ensure
> we passed pinned flag when we're in softpin mode.
> 
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

lgtm,

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/i915/perf_pmu.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/i915/perf_pmu.c b/tests/i915/perf_pmu.c
> index 1e385bebc7..39e9fc5fef 100644
> --- a/tests/i915/perf_pmu.c
> +++ b/tests/i915/perf_pmu.c
> @@ -861,6 +861,7 @@ __sema_busy(int gem_fd, uint64_t ahnd, int pmu, const intel_ctx_t *ctx,
>  		.handle = create_sema(gem_fd, ahnd, reloc, &obj.offset),
>  		.relocation_count = !ahnd ? 2 : 0,
>  		.relocs_ptr = to_user_pointer(reloc),
> +		.flags = !ahnd ? 0 : EXEC_OBJECT_PINNED,
>  	};
>  	struct drm_i915_gem_execbuffer2 eb = {
>  		.batch_start_offset = 64,
> -- 
> 2.34.1
>