On Mon, Mar 04, 2013 at 11:42:29AM +1100, Dave Chinner wrote:
> On Sun, Mar 03, 2013 at 04:57:55PM -0600, Eric Sandeen wrote:
> > On 3/3/13 4:53 PM, Richard Weinberger wrote:
> > >> Anyway, what if you did something more along the lines of [pseudocode]
> > >>
> > >> ocfs2)
> > >> if mounted.ocfs2 -f $TEST-DEV | frob_as_necessary
> > >> ;
> > >> else
> > >> fsck.ocfs2 $TEST-DEV
> > >> fi
> > >> ;;
> > >>
> > >> so that *if* it's mounted on some other node, the fsck won't run.
> > >> That has downsides as Dave mentioned, but for the case where the
> > >> xfstests node is the only one with it in use, it'll still do the
> > >> beneficial consistency check.
> > >>
> > >> Just tweaking the fsck action bsed on *if* it's mounted (or,
> > >> maybe, if the node is in a cluster?) might be a more generic solution
> > >> that is widely applicable to all ocfs2 test environments.
> > >
> > > Good point. mounted.ocfs2 really makes sense. I'll implement this on my
> > > test suite and submit a new patch.
> > Sounds good to me.
> > It'd be most preferable to do a cluster-wide unmount and fsck,
> Which makes no sense to me, because ./check will then do a cluster
> wide unmount as it runs the test harness initialisation. Hence all
> the subsequent tests will run with the filesystem only mounted on
> the local node....
> Really, xfstests is not designed for testing cluster filesystems in
> clustered environments. If we really want to support clustered
> filesystems and cluster wide operations, then we need to think
> about how to architect multi-host support into xfstests sanely.
> Clustered filesystems are not the only people that could make use of
> such functionality (NFS and CIFS come to mind).... ;)
I'm much happier with the "check for other cluster mounts" approach
rather than avoiding fsck entirely. We like xfstests in the local or
single-node cases to behave exactly as one would expect.
We do run xfstests on multiple nodes in a cluster, but by hand and with
our own cleanup. Obviously support for automating that would be awesome
> Dave Chinner
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
"Born under a bad sign.
I been down since I began to crawl.
If it wasn't for bad luck,
I wouldn't have no luck at all."