Re: [PATCH] xfstests: generic/315: make sure the test file exists and is zero length

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[cc fstests@xxxxxxxxxxxxxxx]

On Wed, Dec 10, 2014 at 10:13:43PM -0500, Theodore Ts'o wrote:
> Before we determine how much space is available to be fallocated, and
> before trying to fallocate the file, we need to make sure testfile.315
> exists and is zero-length.  Otherwise the fallocate program will fail
> the first time generic/315 is run on a freshly mkfs'ed file system:
>
> generic/315 0s ...	[06:31:43] [06:31:44] [failed, exit status 1] - output mismatch (see /results/results-ext3conv/generic/315.out.bad)
>     --- tests/generic/315.out	2014-10-31 10:13:04.000000000 -0400
>     +++ /results/results-ext3conv/generic/315.out.bad	2014-11-24 06:31:44.111279060 -0500
>     @@ -1,2 +1,6 @@
>      QA output created by 315
>      Slience is golden
>     +ls: cannot access /vdd/testfile.315: No such file or directory
>     +./tests/generic/315: line 69: [: : integer expression expected
>     +File size is changed to ( Bytes)
>     +(see /results/results-ext3conv/generic/315.full for details)

Already fixed by commit ca28228 ("generic: use xfs_io falloc, not
fallocate").

Yet another demonstration of why we have a "use xfs_io whereever
possible" rule for new tests: we don't have to hack around semantic
oddities of random CLI tools. ;)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx
--
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




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux