From owner-svn-src-head@FreeBSD.ORG Thu Nov 13 18:39:41 2008 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id EB4E0106564A; Thu, 13 Nov 2008 18:39:41 +0000 (UTC) (envelope-from ambrisko@ambrisko.com) Received: from mail.ambrisko.com (mail.ambrisko.com [64.174.51.43]) by mx1.freebsd.org (Postfix) with ESMTP id BEE7B8FC14; Thu, 13 Nov 2008 18:39:41 +0000 (UTC) (envelope-from ambrisko@ambrisko.com) X-Ambrisko-Me: Yes Received: from server2.ambrisko.com (HELO www.ambrisko.com) ([192.168.1.2]) by ironport.ambrisko.com with ESMTP; 13 Nov 2008 10:11:30 -0800 Received: from ambrisko.com (localhost [127.0.0.1]) by www.ambrisko.com (8.14.1/8.14.1) with ESMTP id mADIBJlD099326; Thu, 13 Nov 2008 10:11:19 -0800 (PST) (envelope-from ambrisko@ambrisko.com) Received: (from ambrisko@localhost) by ambrisko.com (8.14.1/8.14.1/Submit) id mADIBJOj099325; Thu, 13 Nov 2008 10:11:19 -0800 (PST) (envelope-from ambrisko) From: Doug Ambrisko Message-Id: <200811131811.mADIBJOj099325@ambrisko.com> In-Reply-To: <20081113174740.GV47073@deviant.kiev.zoral.com.ua> To: Kostik Belousov Date: Thu, 13 Nov 2008 10:11:19 -0800 (PST) X-Mailer: ELM [version 2.4ME+ PL94b (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII Cc: svn-src-head@FreeBSD.org, Doug Ambrisko , svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r184934 - head/sys/ufs/ffs X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 13 Nov 2008 18:39:42 -0000 Kostik Belousov writes: | On Thu, Nov 13, 2008 at 05:40:21PM +0000, Doug Ambrisko wrote: | > Author: ambrisko | > Date: Thu Nov 13 17:40:21 2008 | > New Revision: 184934 | > URL: http://svn.freebsd.org/changeset/base/184934 | > | > Log: | > For now on every 10 cyclinder groups flush the buffer cache to free | > up space. If the buffer cache fills up then the disk systems can | > grind to a halt. Better tuning can be figured out later. | > | > Tested by: Tim, others and work | > Reviewed by: Kostik Belousov | > PR: 128832 | > | > Modified: | > head/sys/ufs/ffs/ffs_snapshot.c | > | > Modified: head/sys/ufs/ffs/ffs_snapshot.c | > ============================================================================== | > --- head/sys/ufs/ffs/ffs_snapshot.c Thu Nov 13 17:13:16 2008 (r184933) | > +++ head/sys/ufs/ffs/ffs_snapshot.c Thu Nov 13 17:40:21 2008 (r184934) | > @@ -351,6 +351,8 @@ restart: | > if (error) | > goto out; | > bawrite(nbp); | > + if (cg % 10 == 0) | > + ffs_syncvnode(vp, MNT_WAIT); | > } | > /* | > * Copy all the cylinder group maps. Although the | > @@ -372,6 +374,8 @@ restart: | > goto out; | > error = cgaccount(cg, vp, nbp, 1); | > bawrite(nbp); | > + if (cg % 10 == 0) | > + ffs_syncvnode(vp, MNT_WAIT); | > if (error) | > goto out; | > } | Thank you ! | | Please, MFC it for 7.1. I'll wait for some of the dust to settle first. Also I want to test a 7.1 machine here without any of my patches, then start adding some back it. I should be able to get that mostly done today unless I get hit with other work stuff. From my perspective I have a couple of low risk patches that should go in but look more radical. Doug A.