Commit 064b2663 authored by Liam Howlett's avatar Liam Howlett Committed by Linus Torvalds
Browse files

drm/i915/selftests: use vma_lookup() in __igt_mmap() 

vma_lookup() will look up the vma at a specific address.  find_vma() will
start the search for a specific address and continue upwards.  This fixes
an issue with the selftest as the returned vma may not be the newly
created vma, but simply the vma at a higher address.

objects

Link: https://lkml.kernel.org/r/20210521174745.2219620-3-Liam.Howlett@Oracle.com


Fixes: 6fedafac (drm/i915/selftests: Wrap vm_mmap() around GEM
Signed-off-by: default avatarLiam R. Howlett <Liam.Howlett@Oracle.com>
Reviewed-by: default avatarLaurent Dufour <ldufour@linux.ibm.com>
Acked-by: default avatarDavid Hildenbrand <david@redhat.com>
Acked-by: default avatarDavidlohr Bueso <dbueso@suse.de>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent ce6d42f2

drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c

+1 −1
Original line number Diff line number Diff line
@@ -871,7 +871,7 @@ static int __igt_mmap(struct drm_i915_private *i915, 


	pr_debug("igt_mmap(%s, %d) @ %lx\n", obj->mm.region->name, type, addr);



	area = find_vma(current->mm, addr);

	area = vma_lookup(current->mm, addr);

	if (!area) {

		pr_err("%s: Did not create a vm_area_struct for the mmap\n",

		       obj->mm.region->name);