Re: [PATCH 1/2] spi: spidev: fix a race condition when accessing spidev->spi

From: Bartosz Golaszewski
Date: Thu Jan 05 2023 - 09:31:06 EST


On Thu, Jan 5, 2023 at 2:52 PM Mark Brown <broonie@xxxxxxxxxx> wrote:
>
> On Thu, Jan 05, 2023 at 01:40:59PM +0100, Bartosz Golaszewski wrote:
>
> > can replace the spinlock with a sleeping lock. Using an RW semaphore
> > allows the syscalls to run concurrently unless protected otherwise. We
>
> I'm not sure this is important or useful, there's not a lot that can
> practically happen concurrently when we get to actually interacting with
> the device and it's making the code a bit less clear.

You suggest to just use a mutex instead?

Bart