[Top] [All Lists]

Re: [PATCH 2/3] xfstests 265: add a prealloc and reserve test

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 2/3] xfstests 265: add a prealloc and reserve test
From: WuBo <wu.bo@xxxxxxxxxxxxxx>
Date: Wed, 16 Nov 2011 09:30:32 +0800
Cc: Ben Myers <bpm@xxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, Linux Btrfs <linux-btrfs@xxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <20111115190821.GA15331@xxxxxxxxxxxxx>
References: <4EB20505.7070005@xxxxxxxxxxxxxx> <4EB205CC.6010509@xxxxxxxxxxxxxx> <20111115182113.GD29840@xxxxxxx> <20111115190821.GA15331@xxxxxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20100921 Fedora/3.1.4-1.fc14 Thunderbird/3.1.4
On 11/16/2011 03:08 AM, Christoph Hellwig wrote:
> On Tue, Nov 15, 2011 at 12:21:13PM -0600, Ben Myers wrote:
>> Hi Wu Bo,
>> On Thu, Nov 03, 2011 at 11:09:00AM +0800, WuBo wrote:
>>> This test is for preallocation test. If the disk is full, just with a 
>>> prealloc
>>> file has some free space that prealloc early. We need to check whether the 
>>> write
>>> to the free space is success or not.
>>> Signed-off-by: Wu Bo <Wu.Bo@xxxxxxxxxxxxxx>
>> This test is failing for me because I don't have fallocate installed.  I
>> suggest the test could to be changed to check for binaries it uses,
>> possibly the version of those binaries, and then not run unless the
>> right ones are installed.  But the best I can do right now is make a
>> note of it.
> It might be even better to just use the xfs_io falloc command as we
> generally expect an uptodate xfs_io for use with xfstests.

Got it.


> --
> 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

<Prev in Thread] Current Thread [Next in Thread>