[igt-dev] [PATCH i-g-t] i915/api_intel_bb: Correct size for munmap

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Wed Feb 2 06:25:27 UTC 2022


If we don't release the entirety of the mmap we create, it still holds a
reference to the fd/object.

v2: changed to intel_buf_size() (ZK)

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

diff --git a/tests/i915/api_intel_bb.c b/tests/i915/api_intel_bb.c
index 5c996f644..e37472fb4 100644
--- a/tests/i915/api_intel_bb.c
+++ b/tests/i915/api_intel_bb.c
@@ -1169,7 +1169,7 @@ static void delta_check(struct buf_ops *bops)
 	ptr = gem_mmap__device_coherent(i915, buf->handle, 0,
 					intel_buf_size(buf), PROT_READ);
 	val = ptr[0x2000 / sizeof(uint32_t)];
-	gem_munmap(ptr, ibb->size);
+	gem_munmap(ptr, intel_buf_size(buf));
 
 	intel_buf_destroy(buf);
 	intel_bb_destroy(ibb);
-- 
2.32.0



More information about the igt-dev mailing list