Subject: Re: CVS commit: src
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Bill Studenmund <wrstuden@netbsd.org>
List: tech-kern
Date: 06/20/2004 19:02:44
--tMbDGjvJuJijemkf
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Jun 21, 2004 at 09:09:10AM +0900, YAMAMOTO Takashi wrote:
> > > i think that having some tricks in VOP_LOCK/UNLOCK/ISLOCKED is enough.
> > > (in addition to stopping exporting v_vnlock, of course.)
> >=20
> > What "tricks" could merge locks A and B into B?
>=20
> something like this:
> 	ffs_lock(...)
> 	{
>=20
> 		genfs_lock(vp, ...);
> 		if (vp is snapshot) {
> 			lockmgr(snaplk, ...);
> 		}
> 	}

That 1) forces all callers to pay a (slight) penalty for snapshot support=
=20
being present, even if they never use it, and 2) it destroys our ability=20
to use v_vnlock. v_vnlock makes life much nicer for layered file systems,=
=20
as it lets us readily access one lock for a stack of vnodes.

Take care,

Bill

--tMbDGjvJuJijemkf
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (NetBSD)

iD8DBQFA1kHEWz+3JHUci9cRAsWrAJ0WiqSDnSgnLWtwxOr8AbESlilLBACfd+sb
w7SBiUVzhuJpRocDoAQV56M=
=fN2f
-----END PGP SIGNATURE-----

--tMbDGjvJuJijemkf--