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

Dixit, Ashutosh ashutosh.dixit at intel.com
Wed Feb 2 16:08:39 UTC 2022


On Tue, 01 Feb 2022 22:25:27 -0800, Zbigniew Kempczyński wrote:
>
> 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)

Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

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


More information about the igt-dev mailing list