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

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Tue Jul 19 07:24:50 UTC 2022


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



More information about the igt-dev mailing list