]> Git Repo - linux.git/commit
drm/i915/ehl: unconditionally flush the pages on acquire
authorMatthew Auld <[email protected]>
Fri, 23 Jul 2021 10:50:45 +0000 (11:50 +0100)
committerMatthew Auld <[email protected]>
Tue, 27 Jul 2021 08:16:46 +0000 (09:16 +0100)
commit13d29c823738c0bcc72a631b8e6c3c5cb89090d6
tree265817238c5ebb7eb95f9c1788eccbd71203e1d2
parent3821cc7fc0b90a55c3708628336a97993e464dba
drm/i915/ehl: unconditionally flush the pages on acquire

EHL and JSL add the 'Bypass LLC' MOCS entry, which should make it
possible for userspace to bypass the GTT caching bits set by the kernel,
as per the given object cache_level. This is troublesome since the heavy
flush we apply when first acquiring the pages is skipped if the kernel
thinks the object is coherent with the GPU. As a result it might be
possible to bypass the cache and read the contents of the page directly,
which could be stale data. If it's just a case of userspace shooting
themselves in the foot then so be it, but since i915 takes the stance of
always zeroing memory before handing it to userspace, we need to prevent
this.

v2: this time actually set cache_dirty in put_pages()
v3: move to get_pages() which looks simpler

BSpec: 34007
References: 046091758b50 ("Revert "drm/i915/ehl: Update MOCS table for EHL"")
Signed-off-by: Matthew Auld <[email protected]>
Cc: Tejas Upadhyay <[email protected]>
Cc: Francisco Jerez <[email protected]>
Cc: Lucas De Marchi <[email protected]>
Cc: Jon Bloomfield <[email protected]>
Cc: Chris Wilson <[email protected]>
Cc: Matt Roper <[email protected]>
Cc: Daniel Vetter <[email protected]>
Reviewed-by: Daniel Vetter <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drivers/gpu/drm/i915/gem/i915_gem_object_types.h
drivers/gpu/drm/i915/gem/i915_gem_shmem.c
This page took 0.064373 seconds and 4 git commands to generate.