From owner-cvs-all@FreeBSD.ORG Tue Apr 22 12:45:22 2008 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DD78C1065670 for ; Tue, 22 Apr 2008 12:45:22 +0000 (UTC) (envelope-from jroberson@jroberson.net) Received: from gv-out-0910.google.com (gv-out-0910.google.com [216.239.58.189]) by mx1.freebsd.org (Postfix) with ESMTP id 42BAC8FC15 for ; Tue, 22 Apr 2008 12:45:22 +0000 (UTC) (envelope-from jroberson@jroberson.net) Received: by gv-out-0910.google.com with SMTP id n40so201955gve.39 for ; Tue, 22 Apr 2008 05:45:20 -0700 (PDT) Received: by 10.150.225.14 with SMTP id x14mr140331ybg.153.1208868320002; Tue, 22 Apr 2008 05:45:20 -0700 (PDT) Received: from ?10.0.1.199? ( [24.94.72.120]) by mx.google.com with ESMTPS id i10sm209332wxd.22.2008.04.22.05.45.18 (version=SSLv3 cipher=OTHER); Tue, 22 Apr 2008 05:45:19 -0700 (PDT) Date: Tue, 22 Apr 2008 02:46:01 -1000 (HST) From: Jeff Roberson X-X-Sender: jroberson@desktop To: Jeff Roberson In-Reply-To: <200804221234.m3MCYGx9036614@repoman.freebsd.org> Message-ID: <20080422024505.V942@desktop> References: <200804221234.m3MCYGx9036614@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed 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 X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Apr 2008 12:45:23 -0000 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 >