From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tao Ma Subject: Re: [PATCH 00/15] replace barriers with explicit flush / FUA usage Date: Fri, 20 Aug 2010 14:50:34 +0800 Message-ID: <4C6E25BA.2020009@oracle.com> References: <20100818093432.646633424@bombadil.infradead.org> <20100818103542.GB9873@mail.oracle.com> <20100818105307.GA31992@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: mfasheh@suse.com, Joel Becker , linux-fsdevel@vger.kernel.org To: Christoph Hellwig Return-path: Received: from rcsinet10.oracle.com ([148.87.113.121]:59811 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751504Ab0HTGvn (ORCPT ); Fri, 20 Aug 2010 02:51:43 -0400 In-Reply-To: <20100818105307.GA31992@infradead.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On 08/18/2010 06:53 PM, Christoph Hellwig wrote: > On Wed, Aug 18, 2010 at 03:35:42AM -0700, Joel Becker wrote: >> On Wed, Aug 18, 2010 at 05:29:08AM -0400, Christoph Hellwig wrote: >>> This series converts over all filesystems to the new WRITE_FLUSH_FUA >>> primitive that Tejun added. XFS, btrfs, gfs2, reiserfs, ext3 and ext4 >>> have passed extensive xfstests coverage with this, while ocfs2, nilfs2 >>> and fat are unsupposed by xfstests and thus untested in this patch. >> >> What does it take to get ocfs2 supported by xfstests? > > Basically just a few filesystem-specific paramters need to be added > to the switch table in common.rc: > > - mkfs paramters if the default mkfs.$FSTYPE invocation doesn't do the > right thing, or ask for confirmatio when doing things like > overwriting existing filesystems or working on whole disks (not sure > if anyone but extN came up with that last weird thing) > - mount paramters for a successfull single node mount with acls and > xattrs enabled > - fsck options to actually force a real check even if the filesystem > is mounted clean (and is nessecary to not make it wait for user > input) > - optionally an entry to create a filesystem with a given size for > ENOSPC tests. oh, I didn't noticed that you have described in detail here. Sorry for the noise. Regards, Tao