| Message ID | 20220105172106.154217-1-matthew.auld@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [i-g-t] tests/i915/userptr: fix mapping type | expand |
On Wed, 05 Jan 2022 09:21:06 -0800, Matthew Auld wrote: > > We need to use the FIXED mapping type on discrete platforms. Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > Cc: Priyanka Dandamudi <priyanka.dandamudi@intel.com> > --- > tests/i915/gem_userptr_blits.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/tests/i915/gem_userptr_blits.c b/tests/i915/gem_userptr_blits.c > index 3464db66..a4dca4c0 100644 > --- a/tests/i915/gem_userptr_blits.c > +++ b/tests/i915/gem_userptr_blits.c > @@ -2185,7 +2185,10 @@ static void test_probe(int fd) > */ > memset(&mmap_offset, 0, sizeof(mmap_offset)); > mmap_offset.handle = gem_create(fd, PAGE_SIZE); > - mmap_offset.flags = I915_MMAP_OFFSET_WB; > + if (gem_has_lmem(fd)) > + mmap_offset.flags = I915_MMAP_OFFSET_FIXED; > + else > + mmap_offset.flags = I915_MMAP_OFFSET_WB; > igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_I915_GEM_MMAP_OFFSET, &mmap_offset), 0); > > for (unsigned long pass = 0; pass < 4 * 4 * 4 * 4 * 4; pass++) { > -- > 2.31.1 >
diff --git a/tests/i915/gem_userptr_blits.c b/tests/i915/gem_userptr_blits.c index 3464db66..a4dca4c0 100644 --- a/tests/i915/gem_userptr_blits.c +++ b/tests/i915/gem_userptr_blits.c @@ -2185,7 +2185,10 @@ static void test_probe(int fd) */ memset(&mmap_offset, 0, sizeof(mmap_offset)); mmap_offset.handle = gem_create(fd, PAGE_SIZE); - mmap_offset.flags = I915_MMAP_OFFSET_WB; + if (gem_has_lmem(fd)) + mmap_offset.flags = I915_MMAP_OFFSET_FIXED; + else + mmap_offset.flags = I915_MMAP_OFFSET_WB; igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_I915_GEM_MMAP_OFFSET, &mmap_offset), 0); for (unsigned long pass = 0; pass < 4 * 4 * 4 * 4 * 4; pass++) {
We need to use the FIXED mapping type on discrete platforms. Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> Cc: Priyanka Dandamudi <priyanka.dandamudi@intel.com> --- tests/i915/gem_userptr_blits.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)