[PATCH 1/4] xfstests: fix fsx fpunch test to actually test for fpunch

Allison Henderson achender at linux.vnet.ibm.com
Mon Jun 27 13:15:26 CDT 2011


On 06/26/2011 10:48 PM, Dave Chinner wrote:
> From: Dave Chinner<dchinner at redhat.com>
>
> The operation flags parameter to fallocate is the second parameter,
> not the last. Hence the fpunch test is actually testing for falloc
> support, not fpunch. Somebody needs a brown paper bag.
>
> Also, add a ftruncate call whenthe fpunch succeeds just in case the
> file was not already zero sized. Failing to ensure we start with a
> zero length file can cause read ops to fail size checks if they
> occur before the file is written to be the main test loop.
>
> While there, observe the quiet flag the same as the falloc test
> does and have them both emit the warning at the same error level.

Hi there,

Sorry about that, I think this bug was mine.  I have tried your patch 
set on my box, and it appears to run with out problems for me. Thx!

Allison Henderson




More information about the xfs mailing list