-----Original Message----- From: Das, Nirmoy <nirmoy.das@xxxxxxxxx> Sent: Tuesday, March 12, 2024 4:18 AM To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx; Das, Nirmoy <nirmoy.das@xxxxxxxxx>; Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>; Janusz Krzysztofik <janusz.krzysztofik@xxxxxxxxxxxxxxx>; Cavitt, Jonathan <jonathan.cavitt@xxxxxxxxx> Subject: [PATCH] drm/i915/selftests: Pick correct caching mode. > > Caching mode is HW dependent so pick a correct one using > intel_gt_coherent_map_type(). > > Cc: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx> > Cc: Janusz Krzysztofik <janusz.krzysztofik@xxxxxxxxxxxxxxx> > Cc: Jonathan Cavitt <jonathan.cavitt@xxxxxxxxx> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10249 > Signed-off-by: Nirmoy Das <nirmoy.das@xxxxxxxxx> LGTM Acked-by: Jonathan Cavitt <jonathan.cavitt@xxxxxxxxx> -Jonathan Cavitt > --- > drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c > index d684a70f2c04..65a931ea80e9 100644 > --- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c > @@ -7,6 +7,7 @@ > #include "i915_drv.h" > #include "i915_selftest.h" > #include "gem/i915_gem_context.h" > +#include "gt/intel_gt.h" > > #include "mock_context.h" > #include "mock_dmabuf.h" > @@ -155,6 +156,7 @@ static int verify_access(struct drm_i915_private *i915, > struct file *file; > u32 *vaddr; > int err = 0, i; > + unsigned int mode; > > file = mock_file(i915); > if (IS_ERR(file)) > @@ -194,7 +196,8 @@ static int verify_access(struct drm_i915_private *i915, > if (err) > goto out_file; > > - vaddr = i915_gem_object_pin_map_unlocked(native_obj, I915_MAP_WB); > + mode = intel_gt_coherent_map_type(to_gt(i915), native_obj, true); > + vaddr = i915_gem_object_pin_map_unlocked(native_obj, mode); > if (IS_ERR(vaddr)) { > err = PTR_ERR(vaddr); > goto out_file; > -- > 2.42.0 > >
