From owner-cvs-all@FreeBSD.ORG  Mon Mar 31 07:55:45 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
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 86770106566B;
	Mon, 31 Mar 2008 07:55:45 +0000 (UTC)
	(envelope-from jeff@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 8ACDB8FC12;
	Mon, 31 Mar 2008 07:55:45 +0000 (UTC)
	(envelope-from jeff@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id m2V7tj4K078640;
	Mon, 31 Mar 2008 07:55:45 GMT
	(envelope-from jeff@repoman.freebsd.org)
Received: (from jeff@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m2V7tjKX078639;
	Mon, 31 Mar 2008 07:55:45 GMT (envelope-from jeff)
Message-Id: <200803310755.m2V7tjKX078639@repoman.freebsd.org>
From: Jeff Roberson <jeff@FreeBSD.org>
Date: Mon, 31 Mar 2008 07:55:45 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/ufs/ffs ffs_vnops.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
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: Mon, 31 Mar 2008 07:55:45 -0000

jeff        2008-03-31 07:55:45 UTC

  FreeBSD src repository

  Modified files:
    sys/ufs/ffs          ffs_vnops.c 
  Log:
   - Since rev 1.142 of ffs_snapshot.c the interlock has not been required
     to protect the v_lock pointer.  Removing the interlock acquisition
     here allows vn_lock() to proceed without requiring the interlock
     at all.
   - If the lock mutated while we were sleeping on it the interlock has
     been dropped.  It is conceivable that the upper layer code was
     relying on the interlock and LK_NOWAIT to protect the identity or
     state of the vnode while acquiring the lock.  In this case return
     EBUSY rather than trying the new lock to prevent potential races.
  
  Reviewed by:    tegge
  
  Revision  Changes    Path
  1.182     +4 -11     src/sys/ufs/ffs/ffs_vnops.c