xfs
[Top] [All Lists]

Re: [PATCH] xfstests: add test for ctimes updates in futimens

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfstests: add test for ctimes updates in futimens
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Tue, 12 Jan 2010 23:07:36 +1100
Cc: xfs@xxxxxxxxxxx, Eric Blake <ebb9@xxxxxxx>
In-reply-to: <20100110215227.GA15354@xxxxxxxxxxxxx>
References: <20100110215227.GA15354@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17)
On Sun, Jan 10, 2010 at 04:52:27PM -0500, Christoph Hellwig wrote:
> This adds Erik's testcase for the ctime update in futimes that I fixed
> a while ago.
> 
> Erik, can you double check the attribution and licensing for your
> testcase?

Looks good. We've both got test "221" patches outstanding now, so
one othem them will have to change number ;)

> +# real QA test starts here
> +_supported_fs xfs
> +_supported_os Linux

FWIW, I'd consider this a generic test - it should work correctly
on NFS, ext3/4, etc...

Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

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