Re: [PATCH 1/5] maple_tree: Allow external locks to be configured with their map

From: Mark Brown
Date: Thu Aug 22 2024 - 16:46:44 EST


On Thu, Aug 22, 2024 at 08:55:20PM +0100, Matthew Wilcox wrote:
> On Thu, Aug 22, 2024 at 08:48:56PM +0100, Mark Brown wrote:

> > 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

> When it's an uncontended spinlock, there's really no overhead. I wish I'd
> been firmer on that point earlier and prohibited the external lock hack.

> The point is that the lock protects the tree. If we are ever going to
> be able to defragment slabs (and I believe this is an ability that Linux
> must gain), we must be able to go from the object (the maple node) to
> a lock that will let us reallocate the node. If there's some external
> lock that protects the tree, we can't possibly do that.

If the external lock guarantees that nothing can possibly be contending
access to the tree (including the read side) I don't see any issue
there?

Attachment: signature.asc
Description: PGP signature