xfs
[Top] [All Lists]

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

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 1/2] xfstests: add generic/321 to test fsync() on directories V2
From: Josef Bacik <jbacik@xxxxxxxxxxxx>
Date: Mon, 25 Nov 2013 16:37:51 -0500
Cc: Josef Bacik <jbacik@xxxxxxxxxxxx>, <xfs@xxxxxxxxxxx>, <linux-btrfs@xxxxxxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/simple; d=fusionio.com; s=default; t=1385415474; bh=7N2fyeOz7NcWfIpvePVocDMuorfIVKopDkuK2YerDU0=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=eEQiT0GJ3O559Ghbfr1+wXar+EglhIVzd//YIyvhU6sLzq+J5xV04rIs+9c8M/TPl OcM9005ciT4N3W1RldHQQA2VOxnkAW2aBV/P+3DAcr6G2KgflhsDfMGJf4Ex14DskD LRDcIBuu6uWsxPJXLNzFXMlia/lqgrjXea+Igm58=
In-reply-to: <20131125161216.GA5085@xxxxxxxxxxxxx>
References: <1384538464-7384-1-git-send-email-jbacik@xxxxxxxxxxxx> <20131125161216.GA5085@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.22 (2013-10-16)
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 breaks out the dmflakey stuff into a common helper to be shared between
> > generic/311 and generic/321. Thanks,
> > 
> > Signed-off-by: Josef Bacik <jbacik@xxxxxxxxxxxx>
> 
> This doesn't seem to actually create the common/dmflakey file.

Eek sorry about that, I'll resend.

Josef

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