From owner-svn-src-head@FreeBSD.ORG Fri Nov 14 09:41:38 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 0E23F1065679; Fri, 14 Nov 2008 09:41:38 +0000 (UTC) (envelope-from byshenknet@byshenk.net) Received: from core.byshenk.net (core.byshenk.net [62.58.73.230]) by mx1.freebsd.org (Postfix) with ESMTP id A66338FC13; Fri, 14 Nov 2008 09:41:37 +0000 (UTC) (envelope-from byshenknet@byshenk.net) Received: from core.byshenk.net (localhost.aoes.com [127.0.0.1]) by core.byshenk.net (8.14.3/8.14.3) with ESMTP id mAE9LRoS015815; Fri, 14 Nov 2008 10:21:28 +0100 (CET) (envelope-from byshenknet@core.byshenk.net) Received: (from byshenknet@localhost) by core.byshenk.net (8.14.3/8.14.3/Submit) id mAE9LRQD015814; Fri, 14 Nov 2008 10:21:27 +0100 (CET) (envelope-from byshenknet) Date: Fri, 14 Nov 2008 10:21:27 +0100 From: Greg Byshenk To: Doug Ambrisko Message-ID: <20081114092127.GO907@core.byshenk.net> References: <20081113174740.GV47073@deviant.kiev.zoral.com.ua> <200811131811.mADIBJOj099325@ambrisko.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200811131811.mADIBJOj099325@ambrisko.com> User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-1.4 required=5.0 tests=ALL_TRUSTED autolearn=failed version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on core.byshenk.net Cc: Kostik Belousov , svn-src-head@freebsd.org, 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: Fri, 14 Nov 2008 09:41:38 -0000 On Thu, Nov 13, 2008 at 10:11:19AM -0800, Doug Ambrisko wrote: > 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. FWIW, I've just tested the patch on my system described previously 7.1-PRERELEASE amd64, and my snapshots now work, where they did not without the patch (see below). Of course, a snapshot takes a very long time to complete, but I think that is to be expected with filesystems that large. Unless there is some problem I am unaware of, let me add my vote to the MFC column. -greg /dev/da1p1 96850470 7934486 81167948 9% /export/mail /dev/da1p2 1937058312 974663846 807429802 55% /export/home /dev/da1p3 1937058312 81633990 1700459658 5% /export/misc /dev/da1p4 2598991534 274277272 2116794940 11% /export/spare [...] /dev/md9 96850470 7877014 81225420 9% /export/snapshots/mail /dev/md8 1937058312 973516342 808577306 55% /export/snapshots/home /dev/md10 1937058312 80486486 1701607162 5% /export/snapshots/misc /dev/md11 2598991534 272737736 2118334476 11% /export/snapshots/spare # ls -l /export/*/.snapshots/* -r--r----- 1 root operator 2048000085176 Nov 14 09:44 /export/home/.snapshots/20081113 -r--r----- 1 root operator 102400004296 Nov 14 08:59 /export/mail/.snapshots/20081113 -r--r----- 1 root operator 2048000085176 Nov 14 09:30 /export/misc/.snapshots/20081113 -r--r----- 1 root operator 2747837267536 Nov 14 10:10 /export/spare/.snapshots/20081113 -- greg byshenk - gbyshenk@byshenk.net - Leiden, NL