Subject: Re: CVS commit: src/sys/miscfs/genfs
To: Andrew Doran <ad@netbsd.org>
From: Bill Stouder-Studenmund <wrstuden@netbsd.org>
List: source-changes
Date: 01/30/2008 17:18:06
--cPi+lWm09sJ+d57q
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jan 23, 2008 at 08:11:32PM +0000, Andrew Doran wrote:
>=20
> Module Name:	src
> Committed By:	ad
> Date:		Wed Jan 23 20:11:32 UTC 2008
>=20
> Modified Files:
> 	src/sys/miscfs/genfs: layer_subr.c
>=20
> Log Message:
> layer_node_find: if we find a node being cleaned out, then ignore it and
> continue.  A thread trying to clean out the extant layer vnode needs to
> acquire the shared lock (i.e. the lower vnode's lock), which our caller
> already holds. To allow the cleaning to succeed the current thread must m=
ake
> progress.  So, for a brief time more than one vnode in a layered file sys=
tem
> may refer to a single vnode in the lower file system.

As long as we will only ever find at most one (which I believe to be true=
=20
from this log message), we're fine.

Thanks!

Take care,

Bill

--cPi+lWm09sJ+d57q
Content-Type: application/pgp-signature
Content-Disposition: inline

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

iD8DBQFHoSHOWz+3JHUci9cRAgQHAJ93LpY2IadS02UdNySlabnbO6rrMACeLBPh
+TDmVcXjVGYZHU4yg+hPUHg=
=8Z93
-----END PGP SIGNATURE-----

--cPi+lWm09sJ+d57q--