<br><br>On Monday, February 17, 2014, Dave Chinner <<a href="mailto:david@fromorbit.com">david@fromorbit.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Feb 17, 2014 at 12:20:38AM +0000, Filipe David Borba Manana wrote:<br>
> Test for a btrfs incremental send issue where we end up sending a<br>
> wrong section of data from a file extent if the corresponding file<br>
> extent is compressed and the respective file extent item has a non<br>
> zero data offset.<br>
><br>
> Fixed by the following linux kernel btrfs patch:<br>
><br>
>    Btrfs: use right clone root offset for compressed extents<br>
><br>
> Signed-off-by: Filipe David Borba Manana <<a href="javascript:;" onclick="_e(event, 'cvml', 'fdmanana@gmail.com')">fdmanana@gmail.com</a>><br>
> ---<br>
><br>
> V2: Made the test more reliable. Now it doesn't depend anymore of btrfs'<br>
>     hole punch implementation leaving hole file extent items when we punch<br>
>     beyond the file's current size.<br>
> V3: Filter xfs_io output and make less use of the run_check function, as<br>
>     suggested by Dave Chinner.<br>
<br>
Awesome. Thanks for the quick turn around.<br>
<br>
>  common/rc           |    5 +++<br>
>  tests/btrfs/040     |  119 +++++++++++++++++++++++++++++++++++++++++++++++++++<br>
>  tests/btrfs/040.out |    9 ++++<br>
>  tests/btrfs/group   |    1 +<br>
>  4 files changed, 134 insertions(+)<br>
>  create mode 100755 tests/btrfs/040<br>
>  create mode 100644 tests/btrfs/040.out<br>
><br>
> diff --git a/common/rc b/common/rc<br>
> index e91568b..27be009 100644<br>
> --- a/common/rc<br>
> +++ b/common/rc<br>
> @@ -2207,6 +2207,11 @@ run_check()<br>
>       "$@" >> $seqres.full 2>&1 || _fail "failed: '$@'"<br>
>  }<br>
><br>
> +_run_btrfs_util_prog()<br>
> +{<br>
> +     run_check $BTRFS_UTIL_PROG $*<br>
> +}<br>
<br>
Can you do a cleanup of all the other btrfs tests that can use this?</blockquote><div><br></div><div>Ok. I just did that for all the test cases not yet merged, as you probably noticed already.</div><div>For the ones already in the repository, I'll see if I can do it soon this coming week.</div>
<div><br></div><div>Thanks</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
<br>
Dave.<br>
--<br>
Dave Chinner<br>
<a href="javascript:;" onclick="_e(event, 'cvml', 'david@fromorbit.com')">david@fromorbit.com</a><br>
</blockquote><br><br>-- <br>Filipe David Manana,<br><br>"Reasonable men adapt themselves to the world.<br> Unreasonable men adapt the world to themselves.<br> That's why all progress depends on unreasonable men."<br>
<br>