Re: [Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-31 Thread Nirmoy Das
This is now merged to gt-next Thanks, Nirmoy On 10/18/2023 4:04 PM, Nirmoy Das wrote: On 10/18/2023 3:00 PM, Andi Shyti wrote: Hi Nirmoy, gen8_ggtt_invalidate() is only needed for limited set of platforms where GGTT is mapped as WC. This was added as way to fix WC based GGTT in commit 0

Re: [Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-18 Thread Nirmoy Das
On 10/18/2023 3:00 PM, Andi Shyti wrote: Hi Nirmoy, gen8_ggtt_invalidate() is only needed for limited set of platforms where GGTT is mapped as WC. This was added as way to fix WC based GGTT in commit 0f9b91c754b7 ("drm/i915: flush system agent TLBs on SNB") and there are no reference in HW do

Re: [Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-18 Thread Andi Shyti
Hi Nirmoy, > > > gen8_ggtt_invalidate() is only needed for limited set of platforms > > > where GGTT is mapped as WC. This was added as way to fix WC based GGTT in > > > commit 0f9b91c754b7 ("drm/i915: flush system agent TLBs on SNB") and > > > there are no reference in HW docs that forces us to u

Re: [Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-18 Thread Nirmoy Das
Hi Andi, On 10/18/2023 1:49 PM, Andi Shyti wrote: Hi Nirmoy, On Wed, Oct 18, 2023 at 11:38:15AM +0200, Nirmoy Das wrote: gen8_ggtt_invalidate() is only needed for limited set of platforms where GGTT is mapped as WC. This was added as way to fix WC based GGTT in commit 0f9b91c754b7 ("drm/i915:

Re: [Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-18 Thread Andi Shyti
Hi Nirmoy, On Wed, Oct 18, 2023 at 11:38:15AM +0200, Nirmoy Das wrote: > gen8_ggtt_invalidate() is only needed for limited set of platforms > where GGTT is mapped as WC. This was added as way to fix WC based GGTT in > commit 0f9b91c754b7 ("drm/i915: flush system agent TLBs on SNB") and > there are

[Intel-gfx] [PATCH v3] drm/i915: Flush WC GGTT only on required platforms

2023-10-18 Thread Nirmoy Das
gen8_ggtt_invalidate() is only needed for limited set of platforms where GGTT is mapped as WC. This was added as way to fix WC based GGTT in commit 0f9b91c754b7 ("drm/i915: flush system agent TLBs on SNB") and there are no reference in HW docs that forces us to use this on non-WC backed GGTT. This