On Thu, Aug 22, 2024 at 08:21:40PM +0100, Matthew Wilcox wrote: > On Thu, Aug 22, 2024 at 08:13:35PM +0100, Mark Brown wrote: > > Currently the maple tree code allows external locks to be configured by > > passing the lock itself. This is generally helpful and convenient but is > No, it's a really bad idea. Stop doing it. Use the internal lock. > It's a temporary hack we put in and I'm really regretting allowing it. I mean, we do use the internal lock here since otherwise lockdep moans but it's pure overhead which just complicates the code. It's only ever taken within another lock, meaning it winds up protecting nothing for these maple trees. We can't go the other way round and use the maple tree lock as the regmap lock since apart from anything else it's a spin lock and we need to use a mutex most of the time to support busses that sleep during I/O.
Attachment:
signature.asc
Description: PGP signature
