Date: Tue, 22 Apr 2008 02:46:01 -1000 (HST) From: Jeff Roberson <jroberson@jroberson.net> To: Jeff Roberson <jeff@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/ufs/ufs inode.h ufs_lookup.c Message-ID: <20080422024505.V942@desktop> In-Reply-To: <200804221234.m3MCYGx9036614@repoman.freebsd.org> References: <200804221234.m3MCYGx9036614@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Forgot to mention; I intentionally removed some asserts that were not valid. Shared lookups are still not enabled on the mountpoint because we're still able to corrupt a filesystem using them. Hopefully it's close to working though. Thanks, Jeff On Tue, 22 Apr 2008, Jeff Roberson wrote: > jeff 2008-04-22 12:34:16 UTC > > FreeBSD src repository > > Modified files: > sys/ufs/ufs inode.h ufs_lookup.c > Log: > - Use a local variable for i_ino in ufs_lookup. It is only used to > communicate between two parts of this one function. This was causing > problems with shared lookups as each would trash the ino value in the > inode. > - Remove the unused i_ino field from the inode structure. > > Revision Changes Path > 1.53 +0 -1 src/sys/ufs/ufs/inode.h > 1.88 +10 -13 src/sys/ufs/ufs/ufs_lookup.c >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080422024505.V942>