On Thu, Nov 12, 2026 at 11:08:59PM +0100, David Hildenbrand (Arm) wrote:
> __igt_mmap_access() reads the buffer with access_process_vm() without
> FOLL_FORCE, but creates the mapping with only PROT_WRITE. Request
> PROT_READ as well so the test remains valid when we fix the ->access
> fallback to properly check VMA access permissions.
>
> Fixes: 9f909e215fea ("drm/i915: Implement vm_ops->access for gdb access into
> mmaps")
> Signed-off-by: David Hildenbrand (Arm) <[email protected]>
Seems sensible so:
Acked-by: Lorenzo Stoakes (ARM) <[email protected]>
> ---
> drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
> b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
> index 303be698891fd..d9c828032a1aa 100644
> --- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
> +++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
> @@ -1460,7 +1460,8 @@ static int __igt_mmap_access(struct drm_i915_private
> *i915,
> if (err)
> return err;
>
> - addr = igt_mmap_offset(i915, offset, obj->base.size, PROT_WRITE,
> MAP_SHARED);
> + addr = igt_mmap_offset(i915, offset, obj->base.size,
> + PROT_READ | PROT_WRITE, MAP_SHARED);
> if (IS_ERR_VALUE(addr))
> return addr;
> ptr = (unsigned long __user *)addr;
>
> --
> 2.43.0
>
--
Cheers, Lorenzo
