From: Theo Buehler <tb@theobuehler.org>
Subject: Re: Simply uvn_attach() & kill UVM_VNODE_ALOCK
To: tech@openbsd.org
Date: Mon, 10 Nov 2025 16:26:50 +0100

On Mon, Nov 10, 2025 at 03:00:25PM +0000, Martin Pieuchot wrote:
> On 10/11/25(Mon) 10:57, Martin Pieuchot wrote:
> > On 09/11/25(Sun) 17:37, Martin Pieuchot wrote:
> > > By querying the size of the vnode before grabbing `vmobjlock'
> > > uvn_attach() can be greatly simplified.  Since the lock is no
> > > longer released, there is no need for UVM_VNODE_ALOCK.
> > 
> > As discussed in the hackroom, here's a simpler diff to review that do
> > not include the removal of UVM_VNODE_ALOCK. 
> 
> Here's the second part of the diff that removes duplicated code and now
> unused UVM_VNODE_ALOCK.

Thanks for splitting the diff. That made it much asier for me.

ok tb