lkml.org 
[lkml]   [2023]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v6] media: uvcvideo: Fix race condition with usb_kill_urb
Hi Hillf

On Wed, 4 Jan 2023 at 21:54, Hillf Danton <hdanton@sina.com> wrote:
>
> On 3 Jan 2023 10:08:56 +0100 Ricardo Ribalda <ribalda@chromium.org>
> >
> > Would it work?
>
> How did you test your patch v6?

I have not tested v6 besides the compile test. The last one that I
device tested was v4 (with atomics) and it didn't crash on x86. I
assumed that since this is more restrictive it was less likely to
fail.

I put this patch on top of my "granular power saving" patch, because
it is more likely to run into race conditions, and then try to
exercise the driver.

So you have a suggestion to trigger the race conditions more effectively?

To be clear, what I mean by would it work with smp_store_release and
smp_load_aqcuire, is that based on the doc it seems like the right
choe, but I am sure that you have more experience than me here :). I
usually rely on mutexes for this.

Thanks!



--
Ricardo Ribalda

\
 
 \ /
  Last update: 2023-03-26 23:27    [W:0.061 / U:0.756 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site