Re: [PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2

2013-11-26 Thread Christoph Hellwig
Both tests look good and work fine for me, Reviewed-by: Christoph Hellwig -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2

2013-11-25 Thread Josef Bacik
Btrfs had some issues with fsync()'ing directories and fsync()'ing after renames. These three new tests cover the 3 different issues we were seeing. This breaks out the dmflakey stuff into a common helper to be shared between generic/311 and generic/321. Thanks, Signed-off-by: Josef Bacik --- V1

Re: [PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2

2013-11-25 Thread Josef Bacik
On Mon, Nov 25, 2013 at 08:12:16AM -0800, Christoph Hellwig wrote: > On Fri, Nov 15, 2013 at 01:01:03PM -0500, Josef Bacik wrote: > > Btrfs had some issues with fsync()'ing directories and fsync()'ing after > > renames. These three new tests cover the 3 different issues we were seeing. > > This br

Re: [PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2

2013-11-25 Thread Christoph Hellwig
On Fri, Nov 15, 2013 at 01:01:03PM -0500, Josef Bacik wrote: > Btrfs had some issues with fsync()'ing directories and fsync()'ing after > renames. These three new tests cover the 3 different issues we were seeing. > This breaks out the dmflakey stuff into a common helper to be shared between > gen

[PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2

2013-11-15 Thread Josef Bacik
Btrfs had some issues with fsync()'ing directories and fsync()'ing after renames. These three new tests cover the 3 different issues we were seeing. This breaks out the dmflakey stuff into a common helper to be shared between generic/311 and generic/321. Thanks, Signed-off-by: Josef Bacik --- V1