Received: with ECARTIS (v1.0.0; list xfs); Thu, 26 Jun 2008 06:24:21 -0700 (PDT) X-Spam-Checker-Version: SpamAssassin 3.3.0-r574664 (2007-09-11) on oss.sgi.com X-Spam-Level: X-Spam-Status: No, score=-2.2 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.0-r574664 Received: from cuda.sgi.com (cuda2.sgi.com [192.48.168.29]) by oss.sgi.com (8.12.11.20060308/8.12.11/SuSE Linux 0.7) with ESMTP id m5QDODSE027022 for ; Thu, 26 Jun 2008 06:24:13 -0700 X-ASG-Debug-ID: 1214486713-1a1300bb0000-NocioJ X-Barracuda-URL: http://cuda.sgi.com:80/cgi-bin/mark.cgi Received: from sandeen.net (localhost [127.0.0.1]) by cuda.sgi.com (Spam Firewall) with ESMTP id 6570028795A for ; Thu, 26 Jun 2008 06:25:13 -0700 (PDT) Received: from sandeen.net (sandeen.net [209.173.210.139]) by cuda.sgi.com with ESMTP id 5Mz2HyP0xXDkdcv4 for ; Thu, 26 Jun 2008 06:25:13 -0700 (PDT) Received: from liberator.sandeen.net (liberator.sandeen.net [10.0.0.4]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by sandeen.net (Postfix) with ESMTP id CBCA8A84093; Thu, 26 Jun 2008 08:25:12 -0500 (CDT) Message-ID: <486398B7.50306@sandeen.net> Date: Thu, 26 Jun 2008 08:25:11 -0500 From: Eric Sandeen User-Agent: Thunderbird 2.0.0.14 (Macintosh/20080421) MIME-Version: 1.0 To: Timothy Shimmin CC: xfs-oss , LinuxRaid , NeilBrown , jeremy@sgi.comwe X-ASG-Orig-Subj: Re: [PATCH] disable queue flag test in barrier check Subject: Re: [PATCH] disable queue flag test in barrier check References: <486307EA.7080007@sandeen.net> <48635284.3060001@sgi.com> In-Reply-To: <48635284.3060001@sgi.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Barracuda-Connect: sandeen.net[209.173.210.139] X-Barracuda-Start-Time: 1214486714 X-Barracuda-Bayes: INNOCENT GLOBAL 0.0000 1.0000 -2.0210 X-Barracuda-Virus-Scanned: by cuda.sgi.com at sgi.com X-Barracuda-Spam-Score: -2.02 X-Barracuda-Spam-Status: No, SCORE=-2.02 using per-user scores of TAG_LEVEL=2.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=2.1 tests= X-Barracuda-Spam-Report: Code version 3.1, rules version 3.1.54393 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-Virus-Scanned: ClamAV 0.91.2/6021/Wed Feb 27 15:55:48 2008 on oss.sgi.com X-Virus-Status: Clean X-archive-position: 16578 X-ecartis-version: Ecartis v1.0.0 Sender: xfs-bounce@oss.sgi.com Errors-to: xfs-bounce@oss.sgi.com X-original-sender: sandeen@sandeen.net Precedence: bulk X-list: xfs Timothy Shimmin wrote: > Also from memory, I believe Neil checked this removal into the SLES10sp1 tree > and some sgi boxes started having slow downs > (looking at Dave's email below - we were not wanting to tell them > to use nobarrier but needed it to work by default - I forget now). But that's an admin issue. The way it is now, for example a home user of md raid1 (me!) can't run barriers even if they wanted to. Until there is a way to know if a write cache is non-volatile the only safe option is to enable barriers when possible. > 6. >> Date: Wed, 25 Jun 2008 08:57:24 +1000 >> From: Dave Chinner >> To: Eric Sandeen >> Cc: LinuxRaid , xfs-oss >> Subject: Re: md raid1 passes barriers, but xfs doesn't use them? >> >> Yeah, the problem was that last time this check was removed was >> that a bunch of existing hardware had barriers enabled on them when >> not necessary (e.g. had NVRAM) and they went 5x slower on MD raid1 >> devices. Having to change the root drive config on a wide install >> base was considered much more of support pain than leaving the >> check there. I guess that was more of a distro upgrade issue than >> a mainline problem, but that's the history. Hence I think we >> should probably do whatever everyone else is doing here.... >> >> Cheers, >> >> Dave. > > So I guess my question is whether there are cases where we are > going to be in trouble again. > Jeremy, do you see some problems? FWIW, the problem *I* foresee is that some people are going to slow down when using the defaults, yes, because barriers will start working again. But I don't see any other safe way around it. Education would be in order, I suppose. :) -Eric > --Tim > > >