Commit b0cc4dca authored by Matthew Auld's avatar Matthew Auld
Browse files

drm/i915/gtt: stop caching the scratch page 



Normal users shouldn't be hitting this, likely this would indicate a
userspace bug. So don't bother caching, which should be safe now that we
manually flush the page.

Suggested-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarMatthew Auld <matthew.auld@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ramalingam C <ramalingam.c@intel.com>
Reviewed-by: default avatarThomas Hellström <thomas.hellstrom@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211028092638.3142258-2-matthew.auld@intel.com
parent 2ca77606

drivers/gpu/drm/i915/gt/gen8_ppgtt.c

+2 −2
Original line number Diff line number Diff line
@@ -651,7 +651,7 @@ static int gen8_init_scratch(struct i915_address_space *vm) 


	vm->scratch[0]->encode =

		gen8_pte_encode(px_dma(vm->scratch[0]),

				I915_CACHE_LLC, pte_flags);

				I915_CACHE_NONE, pte_flags);



	for (i = 1; i <= vm->top; i++) {

		struct drm_i915_gem_object *obj;
@@ -667,7 +667,7 @@ static int gen8_init_scratch(struct i915_address_space *vm) 
		}



		fill_px(obj, vm->scratch[i - 1]->encode);

		obj->encode = gen8_pde_encode(px_dma(obj), I915_CACHE_LLC);

		obj->encode = gen8_pde_encode(px_dma(obj), I915_CACHE_NONE);



		vm->scratch[i] = obj;

	}