Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 6 Sep 2016 23:58:18 +0200
From:      Mateusz Guzik <mjguzik@gmail.com>
To:        Andriy Gapon <avg@FreeBSD.org>
Cc:        Mateusz Guzik <mjg@FreeBSD.org>, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org, Konstantin Belousov <kib@FreeBSD.org>
Subject:   Re: svn commit: r305504 - head/sys/fs/nullfs
Message-ID:  <20160906215818.GD23680@dft-labs.eu>
In-Reply-To: <20160906215149.GC23680@dft-labs.eu>
References:  <201609062122.u86LM3hf032522@repo.freebsd.org> <add2d275-797a-81e1-717f-ab2b62c3a946@FreeBSD.org> <20160906215149.GC23680@dft-labs.eu>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Sep 06, 2016 at 11:51:49PM +0200, Mateusz Guzik wrote:
> On Wed, Sep 07, 2016 at 12:46:44AM +0300, Andriy Gapon wrote:
> > On 07/09/2016 00:22, Mateusz Guzik wrote:
> > >   nullfs: stop special-casing directories in null_vptocnp
> > >   
> > >   The previous code was forcing an expensive walk in vop_stdvptocnp,
> > >   which was causing performance issues on highly contended zfs.
> > 
> > In other words, the name cache was always bypassed for directories?
> > 
> 
> Or whatever the target fs had in store for resolving names.
> 
> This particular thing was causing a lot of idle time on my poudriere
> tests due to lockmgr contention.
> 

To be clear, this is not name -> vp translation, but the other way
around (e.g. for getcwd).

-- 
Mateusz Guzik <mjguzik gmail.com>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20160906215818.GD23680>