linux-kernel - Re: [PATCH v6] media: uvcvideo: Fix race condition with usb_kill

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <CANiDSCtUmS2rO6Pejutpy9kbbZTJ5LC_WaPwy=4un22c72kKkQ@mail.gmail.com>
Date:   Wed, 4 Jan 2023 22:11:55 +0100
From:   Ricardo Ribalda <ribalda@...omium.org>
To:     Hillf Danton <hdanton@...a.com>
Cc:     Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Max Staudt <mstaudt@...gle.com>,
        Sergey Senozhatsky <senozhatsky@...omium.org>,
        Yunke Cao <yunkec@...omium.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6] media: uvcvideo: Fix race condition with usb_kill_urb

Hi Hillf

On Wed, 4 Jan 2023 at 21:54, Hillf Danton <hdanton@...a.com> wrote:
>
> On 3 Jan 2023 10:08:56 +0100 Ricardo Ribalda <ribalda@...omium.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