Date: Thu, 19 Sep 2013 01:59:27 +0000 From: "Teske, Devin" <Devin.Teske@fisglobal.com> To: Kirk McKusick <mckusick@mckusick.com> Cc: freebsd-fs <freebsd-fs@freebsd.org>, "Teske, Devin" <Devin.Teske@fisglobal.com> Subject: Re: kern/182181: [ufs] Leakage of vnode references (race condition?) Message-ID: <13CA24D6AB415D428143D44749F57D720FBD82DC@LTCFISWMSGMB21.FNFIS.com> In-Reply-To: <201309190024.r8J0ORZD097969@chez.mckusick.com> References: <201309190024.r8J0ORZD097969@chez.mckusick.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sep 18, 2013, at 5:24 PM, Kirk McKusick wrote: >> From: "Teske, Devin" <Devin.Teske@fisglobal.com> >> To: Rick Macklem <rmacklem@uoguelph.ca> >> Subject: Re: kern/182181: [ufs] Leakage of vnode references (race condit= ion?) >> Date: Thu, 19 Sep 2013 00:11:03 +0000 >> Cc: freebsd-fs <freebsd-fs@freebsd.org>, >> "Teske, Devin" <Devin.Teske@fisglobal.com> >>=20 >> On Sep 18, 2013, at 5:05 PM, Rick Macklem wrote: >>=20 >>> Devin Teske wrote: >>>>=20 >>>>=20 >>>> Yes, my confusion was... >>>>=20 >>>> 1. The PR headers say 8.4-RELEASE-p3 is affected >>>>=20 >>>> 2. The PR's "How-To-Repeat" starts with "Install a releng/8.4 branch" >>>>=20 >>>> Yet... >>>>=20 >>>> releng/8.4 and even releng/8.3 both use VOP_UNLOCK instead of vput >>>> (read: are patched). >>>>=20 >>> Did you mean "not patched"? The patched version in head has vput() >>> and the unpatched versions have VOP_UNLOCK(), if I read the coed correc= tly. >>>=20 >>=20 >> Well, Kirk's fat-finger made me think that VOP_UNLOCK was the patched- >> state and vput was the unpatched state. (could also be that I'm fighting >> the flu currently). >>=20 >> So everything is copacetic now, except the one outstanding question... >>=20 >> Should we not MFC r253998 to stable/8? >>=20 >> I'm looking to pull this into our own stable/8 kernel, but would >> like to do it by way of svn merge from the stable/8 branch. >> --=20 >> Devin >=20 > Per your request, I have MFC'ed the patch to 8-stable as revision 255681. >=20 Thanks much! Slurping this into our base now. Again, thanks! --=20 Devin _____________ The information contained in this message is proprietary and/or confidentia= l. If you are not the intended recipient, please: (i) delete the message an= d all copies; (ii) do not disclose, distribute or use the message in any ma= nner; and (iii) notify the sender immediately. In addition, please be aware= that any message addressed to our domain is subject to archiving and revie= w by persons other than the intended recipient. Thank you.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?13CA24D6AB415D428143D44749F57D720FBD82DC>