On 9/16/21 12:44 PM, Thierry Reding wrote:
From: Thierry Reding <tred...@nvidia.com>
...
diff --git a/drivers/gpu/drm/tegra/uapi.c b/drivers/gpu/drm/tegra/uapi.c
index 794c400c38b1..66fe8717e747 100644
--- a/drivers/gpu/drm/tegra/uapi.c
+++ b/drivers/gpu/drm/tegra/uapi.c
@@ -201,7 +201,8 @@ int tegra_drm_ioctl_channel_map(struct drm_device *drm, 
void *data, struct drm_f
                goto put_gem;
        }
- mapping->map = host1x_bo_pin(context->client->base.dev, mapping->bo, direction);
+       mapping->map = host1x_bo_pin(context->client->base.dev, mapping->bo, 
direction,
+                                    &context->client->base.cache);

Do we need caching here? The map/unmap operation is explicit and should not be on the hot path, and this will complicate context isolation support where we cannot have an engine-specific cache.

Mikko

Reply via email to