Messages in this thread | | | From | Bartosz Golaszewski <> | Date | Thu, 5 Jan 2023 15:30:47 +0100 | Subject | Re: [PATCH 1/2] spi: spidev: fix a race condition when accessing spidev->spi |
| |
On Thu, Jan 5, 2023 at 2:52 PM Mark Brown <broonie@kernel.org> 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
| |