Re: [PATCH v8 17/26] media/v4l2-core: set pages dirty upon releasing DMA buffers

From: Andrew Morton
Date: Mon Dec 09 2019 - 19:56:32 EST


On Mon, 9 Dec 2019 14:53:35 -0800 John Hubbard <jhubbard@xxxxxxxxxx> wrote:

> After DMA is complete, and the device and CPU caches are synchronized,
> it's still required to mark the CPU pages as dirty, if the data was
> coming from the device. However, this driver was just issuing a
> bare put_page() call, without any set_page_dirty*() call.
>
> Fix the problem, by calling set_page_dirty_lock() if the CPU pages
> were potentially receiving data from the device.
>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> Acked-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx>

What are the user-visible effects of this change?

As it's cc:stable I'd normally send this to Linus within 1-2 weeks, or
sooner. Please confirm that this is a standalone fix, independent of
the rest of this series.