Skip to content

Commit

Permalink
drm/i915/selftests: Pick correct caching mode.
Browse files Browse the repository at this point in the history
Caching mode is HW dependent so pick a correct one using
intel_gt_coherent_map_type().

Cc: Andi Shyti <andi.shyti@linux.intel.com>
Cc: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Jonathan Cavitt <jonathan.cavitt@intel.com>
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10249
Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
Acked-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240312111815.18083-1-nirmoy.das@intel.com
  • Loading branch information
Nirmoy Das authored and Andi Shyti committed Mar 13, 2024
1 parent af7c4a6 commit 8d4ba9f
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down Expand Up @@ -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))
Expand Down Expand Up @@ -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;
Expand Down

0 comments on commit 8d4ba9f

Please sign in to comment.