Re: [PATCH] drm/i915/gem: Flush coherency domains on first set-domain-ioctl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 19 Oct 2020 at 21:38, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
>
> Avoid skipping what appears to be a no-op set-domain-ioctl if the cache
> coherency state is inconsistent with our target domain. This also has
> the utility of using the population of the pages to validate the backing
> store.
>
> The danger in skipping the first set-domain is leaving the cache
> inconsistent and submitting stale data, or worse leaving the clean data
> in the cache and not flushing it to the GPU. The impact should be small
> as it requires a no-op set-domain as the very first ioctl in a
> particular sequence not found in typical userspace.
>
> Reported-by: Zbigniew Kempczyński <zbigniew.kempczynski@xxxxxxxxx>
> Fixes: 754a25442705 ("drm/i915: Skip object locking around a no-op set-domain ioctl")
> Testcase: igt/gem_mmap_offset/blt-coherency
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> Cc: Matthew Auld <matthew.william.auld@xxxxxxxxx>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@xxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx> # v5.2+
Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx>




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux