Re: [PATCH] Add ext2/3/4-specific _check_extN_filesystem function

[Theodore Tso <tytso@xxxxxxx>]

On Wed, Aug 26, 2009 at 06:12:01PM -0400, Christoph Hellwig wrote:
> On Tue, Aug 18, 2009 at 01:04:56PM -0400, Theodore Tso wrote:
> > > Looks like the only difference between the generic and the extN
> > > check routine is the addition of -f to the fsck command line.  What
> > > about just introducing a _fsck_args similar to _mount_opts where
> > > filesystems can set their additional required mount options?
> > 
> > That would work, yeah, and would be result in far less code
> > duplication in common.rc.  I'll take a whack at it and resubmit.
> 
> Are you still going to look at it realtively soon?  If not I'll just
> put this version in to imrove the test coverage on extN and we can
> clean it up later.
> 

Sorry, work has gotten busy and I'm trying to get some last patches
whipped into shape before the merge window opens.  I probably won't
have time to look at this for at least a week or two.  So if you want
to merge this version in for now, I'll try to clean it up when I have
a chance.

                                 - Ted