[PATCH 2/2] drm/xe: move DPT l2 flush to a more sensible place

[Lucas De Marchi]

On Fri, Jun 06, 2025 at 11:45:48AM +0100, Matthew Auld wrote:
>Only need the flush for DPT host updates here. Normal GGTT updates don't
>need special flush.
>
>Fixes: 01570b446939 ("drm/xe/bmg: implement Wa_16023588340")
>Signed-off-by: Matthew Auld <matthew.auld at intel.com>
>Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>Cc: <stable at vger.kernel.org> # v6.12+


Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>

Lucas De Marchi

>---
> drivers/gpu/drm/xe/display/xe_fb_pin.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/display/xe_fb_pin.c b/drivers/gpu/drm/xe/display/xe_fb_pin.c
>index 461ecdfdb742..b16a6e3ff4b4 100644
>--- a/drivers/gpu/drm/xe/display/xe_fb_pin.c
>+++ b/drivers/gpu/drm/xe/display/xe_fb_pin.c
>@@ -165,6 +165,9 @@ static int __xe_pin_fb_vma_dpt(const struct intel_framebuffer *fb,
>
> 	vma->dpt = dpt;
> 	vma->node = dpt->ggtt_node[tile0->id];
>+
>+	/* Ensure DPT writes are flushed */
>+	xe_device_l2_flush(xe);
> 	return 0;
> }
>
>@@ -334,8 +337,6 @@ static struct i915_vma *__xe_pin_fb_vma(const struct intel_framebuffer *fb,
> 	if (ret)
> 		goto err_unpin;
>
>-	/* Ensure DPT writes are flushed */
>-	xe_device_l2_flush(xe);
> 	return vma;
>
> err_unpin:
>-- 
>2.49.0
>