[igt-dev] [PATCH i-g-t] i915/api_intel_bb: Release the crc32 reloc offset after use

[Zbigniew Kempczyński]

From: Chris Wilson <chris.p.wilson at linux.intel.com>

Upon completion of the crc32 calculation, release the relocation offset
assigned to us by the allocator. Since gem_close()/gem_create() will
reuse the same handle, the next loop tries to reuse the same relocation
offset, but complains that it is no longer a match.

api_intel_bb:11918) DEBUG: size:          4, cpu crc: 0x2144df1c (time: 0.000), gpu crc: 0x2144df1c (time: 0.000) [ EQUAL ]
(api_intel_bb:11918) intel_allocator_reloc-CRITICAL: Test assertion failure function intel_allocator_reloc_alloc, file ../lib/intel_allocator_reloc.c:90:
(api_intel_bb:11918) intel_allocator_reloc-CRITICAL: Failed assertion: rec->size == size
(api_intel_bb:11918) intel_allocator_reloc-CRITICAL: error: 0x4 != 0x10
(api_intel_bb:11918) igt_core-INFO: Stack trace:
(api_intel_bb:11918) igt_core-INFO:   #0 ../lib/igt_core.c:2064 __igt_fail_assert()
(api_intel_bb:11918) igt_core-INFO:   #1 [intel_allocator_reloc_alloc+0x11b]
(api_intel_bb:11918) igt_core-INFO:   #2 ../lib/intel_allocator.c:604 handle_request()
(api_intel_bb:11918) igt_core-INFO:   #3 ../lib/intel_allocator.c:1125 __intel_allocator_alloc()
(api_intel_bb:11918) igt_core-INFO:   #4 ../lib/intel_allocator.c:1150 intel_allocator_alloc()
(api_intel_bb:11918) igt_core-INFO:   #5 ../lib/i915/i915_crc.c:256 i915_crc32()
(api_intel_bb:11918) igt_core-INFO:   #6 ../tests/i915/api_intel_bb.c:1519 __igt_unique____real_main1565()
(api_intel_bb:11918) igt_core-INFO:   #7 ../tests/i915/api_intel_bb.c:1565 main()
(api_intel_bb:11918) igt_core-INFO:   #8 ../sysdeps/nptl/libc_start_call_main.h:58 __libc_start_call_main()
(api_intel_bb:11918) igt_core-INFO:   #9 ../csu/libc-start.c:128 __libc_start_main@@GLIBC_2.34()
(api_intel_bb:11918) igt_core-INFO:   #10 [_start+0x25]

Signed-off-by: Chris Wilson <chris.p.wilson at linux.intel.com>
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
---
 tests/i915/api_intel_bb.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/i915/api_intel_bb.c b/tests/i915/api_intel_bb.c
index 980906f4ed..7ccc00aa25 100644
--- a/tests/i915/api_intel_bb.c
+++ b/tests/i915/api_intel_bb.c
@@ -1519,6 +1519,8 @@ static void test_crc32(int i915, const intel_ctx_t *ctx,
 			  "gpu crc: 0x%08x (time: %.3f) [ %s ]\n",
 			  (long long) size, cpu_crc, cpu_time, gpu_crc, gpu_time,
 			  cpu_crc == gpu_crc ? "EQUAL" : "DIFFERENT");
+
+		put_offset(ahnd, data);
 		gem_close(i915, data);
 		igt_assert(cpu_crc == gpu_crc);
 	}
-- 
2.34.1