From owner-cvs-all@FreeBSD.ORG  Thu May 15 14:12:09 2003
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 6725737B401; Thu, 15 May 2003 14:12:09 -0700 (PDT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 034B543F3F; Thu, 15 May 2003 14:12:09 -0700 (PDT)
	(envelope-from rwatson@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.6/8.12.6) with ESMTP id h4FLC80U072681;
	Thu, 15 May 2003 14:12:08 -0700 (PDT)
	(envelope-from rwatson@repoman.freebsd.org)
Received: (from rwatson@localhost)
	by repoman.freebsd.org (8.12.6/8.12.6/Submit) id h4FLC8MF072680;
	Thu, 15 May 2003 14:12:08 -0700 (PDT)
Message-Id: <200305152112.h4FLC8MF072680@repoman.freebsd.org>
From: Robert Watson <rwatson@FreeBSD.org>
Date: Thu, 15 May 2003 14:12:08 -0700 (PDT)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/nfsclient nfs_bio.c nfs_vnops.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: Thu, 15 May 2003 21:12:09 -0000

rwatson     2003/05/15 14:12:08 PDT

  FreeBSD src repository

  Modified files:
    sys/nfsclient        nfs_bio.c nfs_vnops.c 
  Log:
  This change grabs the vnode lock for NFS client vnodes when calling
  VOP_SETATTR() or VOP_GETATTR(); without these locks (a) VFS_DEBUG_LOCKS
  will panic, and (b) it may be possible to corrupt entries in the cached
  vnode attributes in the nfsnode, since nfsnode attribute cache data is
  also protected by the vnode lock.
  
  Approved by:    re (jhb)
  Pointed out by: VFS_DEBUG_LOCKS
  
  Revision  Changes    Path
  1.118     +3 -1      src/sys/nfsclient/nfs_bio.c
  1.205     +4 -0      src/sys/nfsclient/nfs_vnops.c