From owner-cvs-all@FreeBSD.ORG  Wed Mar 30 02:59:33 2005
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 51C3916A4CE; Wed, 30 Mar 2005 02:59:33 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 37BE343D2D; Wed, 30 Mar 2005 02:59:33 +0000 (GMT)
	(envelope-from das@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j2U2xXf7092763;
	Wed, 30 Mar 2005 02:59:33 GMT
	(envelope-from das@repoman.freebsd.org)
Received: (from das@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j2U2xXhg092762;
	Wed, 30 Mar 2005 02:59:33 GMT
	(envelope-from das)
Message-Id: <200503300259.j2U2xXhg092762@repoman.freebsd.org>
From: David Schultz <das@FreeBSD.org>
Date: Wed, 30 Mar 2005 02:59:32 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/kern vfs_cache.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 30 Mar 2005 02:59:33 -0000

das         2005-03-30 02:59:32 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             vfs_cache.c 
  Log:
  Merge kern___cwd() and vn_fullpath(), which were virtually identical,
  except for places where people forget to update one of them.  We now
  collect only one set of stats for both of these routines.  Other
  changes in this commit include:
  
  - Start acquiring Giant again in vn_fullpath(), since it is required
    when crossing a mount point.
  
  - Expand the scope of the cache lock to avoid dropping it and
    picking it up again for every pathname component.  This also
    makes it trivial to avoid races in stats collection.
  
  - Assert that nc_dvp == v_dd for directories instead of returning
    an error to userland when this is not true.  AFAIK, it should
    always be true when v_dd is non-null.
  
  - For vn_fullpath(), handle the first (non-directory) vnode
    separately.
  
  Glanced at by:  jeff, phk
  
  Revision  Changes    Path
  1.97      +89 -132   src/sys/kern/vfs_cache.c