Re: [GIT PULL] vfs mount

[Christoph Hellwig <hch@infradead.org>]

On Mon, Apr 07, 2025 at 09:00:10AM -0700, Linus Torvalds wrote:
> On Mon, 7 Apr 2025 at 01:51, Christoph Hellwig <hch@infradead.org> wrote:
> >
> > The scoped one with proper indentation is fine.  The non-scoped one is
> > the one that is really confusing and odd.
> 
> Ahh, I misunderstood you.
> 
> You're obviously right in a "visually obvious" way - even if it was
> the scoped one that caused problems.
> 
> But the non-scoped one is *so* convenient when you have a helper
> function that just wants to run with some local (or RCU) held.

I wish we'd just hage a way to run an existing scope, especially a
funtion fun with a lock, e.g.

int some_helper(....)
	scoped_lock(&some_mutex)
{
	...
}

which would give you that with a much more obvious and redable
syntax.  Not taking the resource in the middle of the block and
releasing it at the end will also fix tons of bugs for non-obvious
behavior.