On Mar 15, 2013, at 9:23 AM, Rich Johnston <rjohnston@xxxxxxx> wrote: > On 03/15/2013 08:46 AM, Eric Sandeen wrote: >> Yep sorry. I figure its always a race in numbers do I don't bother with fixing them up. Should I? > > Eric, > I was going to revert and resubmit as test 306, is that OK with you? > Sure, of course. Thanks, Eric >> On Mar 15, 2013, at 5:16 AM, Dave Chinner <david@xxxxxxxxxxxxx> wrote: >> >>> On Wed, Mar 13, 2013 at 02:01:58PM -0500, Eric Sandeen wrote: >>>> Run basic btrfs information commands in various ways, performing >>>> sanity tests of: filesystem show, label, sync, and device stats >>>> (sync is included just because it's simple). These are mostly >>>> just smoke tests, although for example show by label & UUID >>>> should verify that the correct fs was shown. >>>> >>>> This also add2730c49a3s quite a few new filters to accommodate the output >>>> of the new commands. >>>> >>>> Cc: linux-btrfs@xxxxxxxxxxxxxxx >>>> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> >>>> --- >>>> >>>> V2: sort & uniq device stats output to account for variable >>>> nrs. of devices. Optionally check that the proper nr. of >>>> devices was returned, as well. >>>> >>>> V3: _filter_uuid needs to pay attention to $1, not hope >>>> that the caller has set $UUID. Oops. >>>> >>>> V4: fix stats filter to set UNIQ_OPT in both cases, ugh. >>> >>> Something I just noticed in this commit: >>> >>>> +# get standard environment, filters and checks >>>> +. ./common.rc >>>> +. ./btrfs.filter >>> >>> That should probably have been named "common.filter.btrfs". > > Dave, > Do you want me to rename btrfs.filter to common.filter.btrfs when I resubmit this patch? > > Regards > --Rich > >>> I found this updating my patchset that moves all the common.* files >>> into a common/ directory.... >>> >>> I'll rename it to common/filter.btrfs.... >>> >>> Cheers, >>> >>> Dave. >>> -- >>> Dave Chinner >>> david@xxxxxxxxxxxxx >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in >>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> _______________________________________________ >> xfs mailing list >> xfs@xxxxxxxxxxx >> http://oss.sgi.com/mailman/listinfo/xfs > > _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
