xfs
[Top] [All Lists]

Re: [PATCH] xfstests: btrfs/002, another send regression test

To: Josef Bacik <jbacik@xxxxxxxxxxxx>
Subject: Re: [PATCH] xfstests: btrfs/002, another send regression test
From: Rich Johnston <rjohnston@xxxxxxx>
Date: Wed, 14 Aug 2013 15:01:03 -0500
Cc: <linux-btrfs@xxxxxxxxxxxxxxx>, <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1376334802-18960-1-git-send-email-jbacik@xxxxxxxxxxxx>
References: <1376334802-18960-1-git-send-email-jbacik@xxxxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5
On 08/12/2013 02:13 PM, Josef Bacik wrote:
This is a regression test for a problem we had where we'd assume we had created
a directory if it only had subvols inside of it.  This was happening because
subvols would have lower inode numbers than our current send progress because
their inode numbers are based off of a different counter.  Thanks,

Signed-off-by: Josef Bacik <jbacik@xxxxxxxxxxxx>
---
  tests/btrfs/002     |   85 +++++++++++++++++++++++++++++++++++++++++++++++++++
  tests/btrfs/002.out |    2 +
  tests/btrfs/group   |    1 +
  3 files changed, 88 insertions(+), 0 deletions(-)
  create mode 100755 tests/btrfs/002
  create mode 100644 tests/btrfs/002.out

Thanks for the patch Josef, this has been committed:

commit 6ef348f857ee2912b034ab0148bab7cdb814c0e7
Author: Josef Bacik <jbacik@xxxxxxxxxxxx>
Date:   Mon Aug 12 19:13:22 2013 +0000

    xfstests: btrfs/008, another send regression test

This is a regression test for a problem we had where we'd assume we had created a directory if it only had subvols inside of it. This was happening because subvols would have lower inode numbers than our current send progress because
    their inode numbers are based off of a different counter.  Thanks,

    Signed-off-by: Josef Bacik <jbacik@xxxxxxxxxxxx>
    Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>
    Signed-off-by: Rich Johnston <rjohnston@xxxxxxx>

    [rjohnston: renumbered test from 002 to 008]

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