Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 29 Jan 2009 20:15:23 +0200
From:      Kostik Belousov <kostikbel@gmail.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Edward Tomasz Napierala <trasz@freebsd.org>
Subject:   Re: svn commit: r187894 - head/sys/ufs/ffs
Message-ID:  <20090129181523.GA2471@deviant.kiev.zoral.com.ua>
In-Reply-To: <200901291209.14313.jhb@freebsd.org>
References:  <200901291647.n0TGlFHZ058776@svn.freebsd.org> <200901291209.14313.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--OgqxwSJOaUobr8KG
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Jan 29, 2009 at 12:09:13PM -0500, John Baldwin wrote:
> On Thursday 29 January 2009 11:47:15 am Edward Tomasz Napierala wrote:
> > Author: trasz
> > Date: Thu Jan 29 16:47:15 2009
> > New Revision: 187894
> > URL: http://svn.freebsd.org/changeset/base/187894
> >=20
> > Log:
> >   Make sure the cdev doesn't go away while the filesystem is still moun=
ted.
> >   Otherwise dev2udev() could return garbage.
> >  =20
> >   Reviewed by:	kib
> >   Approved by:	rwatson (mentor)
> >   Sponsored by:	FreeBSD Foundation
>=20
> Is this applicable to all filesystems?  I'm curious why the VREF() on the=
=20
> vnode associated with the cdev entry (um_devvp) is not sufficient to prev=
ent=20
> this?  I would have thought that the vnode would have held a reference on=
 the=20
> cdev.

The point of this commit is that devvp vnode may be reclaimed.

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

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (FreeBSD)

iEYEARECAAYFAkmB8jkACgkQC3+MBN1Mb4jNegCfTz/8Vr2bHIQNhF+2/FLG75/s
PMQAoKMsGUmxwoV/3p8tB0AdzcBZ9Ecd
=OZ25
-----END PGP SIGNATURE-----

--OgqxwSJOaUobr8KG--



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