Re: [PATCH 7/9] Set default extN mount options

[Eric Sandeen <sandeen@xxxxxxxxxxx>]

Christoph Hellwig wrote:

>> @@ -585,7 +589,7 @@ _require_scratch()
>>                   _notrun "this test requires a valid \$SCRATCH_DEV"
>>               fi
>>               ;;
>> -    nfs*|ext2|ext3|reiserfs)
>> +    nfs*|ext2|ext3|ext4|reiserfs)
>>               echo $SCRATCH_DEV | grep -q ":" > /dev/null 2>&1
>>               if [ ! -z "$SCRATCH_DEV" -a ! -b "$SCRATCH_DEV" -a "$?" != "0" 
>> ]
> 
> This whole function looks fishy to me, but just treating ext4 the same
> as ext2/ext3 makes sense at least.

I agree, I Need to sort out what's going on here but this was a "make it
work" patch :)

Thanks,

-Eric

> 
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
>