Re: [RFC PATCH 4.19.y-cip 24/50] PCI: endpoint: Replace spinlock with mutex
toggle quoted messageShow quoted text
From: Pavel Machek <pavel@...>
Sent: 20 October 2020 21:44
To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...>
Cc: cip-dev@...; Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@...>; Pavel Machek <pavel@...>; Biju Das
Subject: Re: [RFC PATCH 4.19.y-cip 24/50] PCI: endpoint: Replace spinlock with mutex
commit 3d3248dbd018502f654064c78efcd2e165ab3486 upstream.Could I get some kind of explanation why this is good idea?
The pci_epc_ops is not intended to be invoked from interrupt context.
Hence replace spin_lock_irqsave and spin_unlock_irqrestore with
mutex_lock and mutex_unlock respectively.
Apart of one mentioned above other point I would add is on a single core machine mutex_lock/unlock would be good choice.
Also to add the callbacks in controller driver might sleep. For example in raise_irq callback , .
As long as code protected by the locks does not sleep, spinlocks are
okay... (but they should not need "_irqsave" variants).
They are likely to have better performance, too, when protected code
is small and fast.
I do agree with the above two points *if the code isn't sleeping*.
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Join firstname.lastname@example.org to automatically receive all group messages.