Date: Sat, 26 Apr 1997 19:06:50 +0100 (BST) From: Doug Rabson <dfr@nlsystems.com> To: KATO Takenori <kato@FreeBSD.org> Cc: CVS-committers@FreeBSD.org, cvs-all@FreeBSD.org, cvs-sys@FreeBSD.org Subject: Re: cvs commit: src/sys/miscfs/union union_vnops.c Message-ID: <Pine.BSF.3.95q.970426190247.346D-100000@herring.nlsystems.com> In-Reply-To: <199704261343.GAA20133@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 26 Apr 1997, KATO Takenori wrote: > kato 97/04/26 06:43:27 > > Modified: sys/miscfs/union union_vnops.c > Log: > Do nothing instead of adjusting un_flags when (uppervp is locked) && > (UN_ULOCK is not set) in union_lock. This condition may indicate > race. DIAGNOSTIC kernel still panic here. I don't think this is right. I just had a lockmgr panic using the 1.28 version of this file (the one which played with the flags). It turned out that update had the uppervp locked and the union node should have locked the uppervp itself which would have waited until update had finished before continuing. In the same situation for 1.29 we would not lock uppervp at all which just seems wrong. -- Doug Rabson Mail: dfr@nlsystems.com Nonlinear Systems Ltd. Phone: +44 181 951 1891
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.3.95q.970426190247.346D-100000>