From owner-cvs-src@FreeBSD.ORG  Wed Jul 23 14:32:52 2008
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id A1D101065750;
	Wed, 23 Jul 2008 14:32:50 +0000 (UTC) (envelope-from kib@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id CE8DD8FC12;
	Wed, 23 Jul 2008 14:32:50 +0000 (UTC) (envelope-from kib@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 m6NEWo2n000749;
	Wed, 23 Jul 2008 14:32:50 GMT (envelope-from kib@repoman.freebsd.org)
Received: (from svn2cvs@localhost)
	by repoman.freebsd.org (8.14.2/8.14.1/Submit) id m6NEWoTo000748;
	Wed, 23 Jul 2008 14:32:50 GMT (envelope-from kib@repoman.freebsd.org)
Message-Id: <200807231432.m6NEWoTo000748@repoman.freebsd.org>
X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to
	kib@repoman.freebsd.org using -f
From: Konstantin Belousov <kib@FreeBSD.org>
Date: Wed, 23 Jul 2008 14:32:44 +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_balloc.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 23 Jul 2008 14:32:52 -0000

kib         2008-07-23 14:32:44 UTC

  FreeBSD src repository

  Modified files:
    sys/ufs/ffs          ffs_balloc.c 
  Log:
  SVN rev 180758 on 2008-07-23 14:32:44Z by kib
  
  The ffs_balloc_ufs{1,2} functions call bdwrite() while having several
  vnode buffers locked at once. In particular, there are indirect buffers
  among locked ones. The bdwrite() may start the flushing to keep dirty
  buffer list at the bounds. If any buffer on the dirty list requires
  translation from logical to physical block number, code may ends up
  trying to lock an indirect buffer already locked in ffs_balloc_ufsX.
  
  Prevent the bdflush() activity when several buffers are locked at once
  by setting the TDP_INBDFUSH for the problematic code blocks.
  
  Reported and tested by: pho, Josef Buchsteiner at Juniper
  In collaboration with:  kan
  MFC after:      1 month
  
  Revision  Changes    Path
  1.54      +22 -2     src/sys/ufs/ffs/ffs_balloc.c