Re: [PATCH] media: i2c: ov5648: Fix a data race on state->streaming

[Ginger]

Hi Sakari,

Thanks a lot. I will pay attention next time.

Best regards,
Ginger


On Fri, Sep 25, 2026 at 1:01 PM Sakari Ailus
<sakari.ailus@xxxxxxxxxxxxxxx> wrote:
>
> Hi Ginger,
>
> On Wed, Sep 23, 2026 at 09:56:31AM +0800, Ginger Li wrote:
> > ov5648_s_stream() updates state->streaming after it has released
> > sensor->mutex, while ov5648_resume() reads that field with sensor->mutex
> > held.  A concurrent resume can therefore observe a stale streaming state and
> > leave the sensor in standby while the software believes that it is streaming,
> > or the other way around.
>
> Please wrap at 75 characters next time, I've fixed it this time.
>
> --
> Sakari Ailus