From: Anand Jain <anand.jain@oracle.com>
To: Filipe Manana <fdmanana@kernel.org>, Johannes Thumshirn <jth@kernel.org>
Cc: linux-btrfs@vger.kernel.org, Zorro Lang <zlang@redhat.com>,
fstests@vger.kernel.org,
Johannes Thumshirn <johannes.thumshirn@wdc.com>
Subject: Re: [PATCH] btrfs: update golden output of RST test cases
Date: Mon, 1 Jul 2024 17:48:31 +0800 [thread overview]
Message-ID: <818b51a6-6ac3-4a2f-afa7-42568227db65@oracle.com> (raw)
In-Reply-To: <CAL3q7H5coRxND9EwUkzOyLSp2cLKWirbEueiuATegytfBChRaQ@mail.gmail.com>
On 6/29/24 18:50, Filipe Manana wrote:
> On Thu, Jun 27, 2024 at 10:47 AM Johannes Thumshirn <jth@kernel.org> wrote:
>>
>> From: Johannes Thumshirn <johannes.thumshirn@wdc.com>
>>
>> Update the golden output of the RAID stripe tree test cases after the
>> on-disk format and print format changes.
>
> Better to mention here that the on-disk format changes happened:
>
> 1) For btrfs-progs 6.9.1, with commit 7c549b5f7cc0 ("btrfs-progs:
> remove raid stripe encoding";
>
> 2) For the kernel, with the patch "btrfs: remove raid-stripe-tree
> encoding field from stripe_extent", which is not yet in Linus' tree;
>
> And that we don't need to ensure compatibility with different
> btrfs-progs and kernel versions because it's an experimental feature,
> subject to changes until stabilized.
>
> Other than that it looks good, it makes the tests pass with the
> for-next btrfs kernel branch and btrfs-progs 6.9.1+.
>
Also, please rebase on the latest fstests for-next.
Thanks, Anand
> Thanks.
>
>>
>> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
>> ---
>> tests/btrfs/304.out | 9 +++------
>> tests/btrfs/305.out | 24 ++++++++----------------
>> tests/btrfs/306.out | 18 ++++++------------
>> tests/btrfs/307.out | 15 +++++----------
>> tests/btrfs/308.out | 39 +++++++++++++--------------------------
>> 5 files changed, 35 insertions(+), 70 deletions(-)
>>
>> diff --git a/tests/btrfs/304.out b/tests/btrfs/304.out
>> index 39f56f32274d..97ec27455b01 100644
>> --- a/tests/btrfs/304.out
>> +++ b/tests/btrfs/304.out
>> @@ -12,8 +12,7 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> bytes used XXXXXX
>> @@ -30,8 +29,7 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> @@ -49,8 +47,7 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 0 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> diff --git a/tests/btrfs/305.out b/tests/btrfs/305.out
>> index 7090626c3036..02642c904b1e 100644
>> --- a/tests/btrfs/305.out
>> +++ b/tests/btrfs/305.out
>> @@ -14,14 +14,11 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 2 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 2 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 16
>> stripe 0 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> bytes used XXXXXX
>> @@ -40,12 +37,10 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 1 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> @@ -65,16 +60,13 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 0 key (XXXXXX RAID_STRIPE 61440) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 2 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 2 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 3 physical XXXXXXXXX
>> stripe 1 devid 4 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> diff --git a/tests/btrfs/306.out b/tests/btrfs/306.out
>> index 25065674c77b..954567db7623 100644
>> --- a/tests/btrfs/306.out
>> +++ b/tests/btrfs/306.out
>> @@ -14,11 +14,9 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 16
>> stripe 0 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> bytes used XXXXXX
>> @@ -37,12 +35,10 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> @@ -62,12 +58,10 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 1 key (XXXXXX RAID_STRIPE 4096) itemoff XXXXX itemsize 32
>> stripe 0 devid 3 physical XXXXXXXXX
>> stripe 1 devid 4 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> diff --git a/tests/btrfs/307.out b/tests/btrfs/307.out
>> index 2815d17d7f03..e2f1d3d84a68 100644
>> --- a/tests/btrfs/307.out
>> +++ b/tests/btrfs/307.out
>> @@ -12,11 +12,9 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 1 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 16
>> stripe 0 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> bytes used XXXXXX
>> @@ -33,8 +31,7 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 131072) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 0 key (XXXXXX RAID_STRIPE 131072) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> @@ -52,12 +49,10 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 0 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 1 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 32
>> stripe 0 devid 3 physical XXXXXXXXX
>> stripe 1 devid 4 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> diff --git a/tests/btrfs/308.out b/tests/btrfs/308.out
>> index 23b31dd32959..75e010d54252 100644
>> --- a/tests/btrfs/308.out
>> +++ b/tests/btrfs/308.out
>> @@ -16,20 +16,15 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 1 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 2 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 2 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 16
>> stripe 0 devid 2 physical XXXXXXXXX
>> - item 3 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 3 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> - item 4 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 24
>> - encoding: RAID0
>> + item 4 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 16
>> stripe 0 devid 1 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> bytes used XXXXXX
>> @@ -50,16 +45,13 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 131072) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 1 key (XXXXXX RAID_STRIPE 131072) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 2 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 40
>> - encoding: RAID1
>> + item 2 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> @@ -81,24 +73,19 @@ checksum stored <CHECKSUM>
>> checksum calced <CHECKSUM>
>> fs uuid <UUID>
>> chunk uuid <UUID>
>> - item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 0 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 1 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 1 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 2 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 2 key (XXXXXX RAID_STRIPE 65536) itemoff XXXXX itemsize 32
>> stripe 0 devid 3 physical XXXXXXXXX
>> stripe 1 devid 4 physical XXXXXXXXX
>> - item 3 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 3 key (XXXXXX RAID_STRIPE 32768) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> - item 4 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 40
>> - encoding: RAID10
>> + item 4 key (XXXXXX RAID_STRIPE 8192) itemoff XXXXX itemsize 32
>> stripe 0 devid 1 physical XXXXXXXXX
>> stripe 1 devid 2 physical XXXXXXXXX
>> total bytes XXXXXXXX
>> --
>> 2.43.0
>>
>>
next prev parent reply other threads:[~2024-07-01 9:48 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-27 9:47 [PATCH] btrfs: update golden output of RST test cases Johannes Thumshirn
2024-06-29 10:50 ` Filipe Manana
2024-07-01 9:48 ` Anand Jain [this message]
2024-07-01 9:53 ` Anand Jain
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=818b51a6-6ac3-4a2f-afa7-42568227db65@oracle.com \
--to=anand.jain@oracle.com \
--cc=fdmanana@kernel.org \
--cc=fstests@vger.kernel.org \
--cc=johannes.thumshirn@wdc.com \
--cc=jth@kernel.org \
--cc=linux-btrfs@vger.kernel.org \
--cc=zlang@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox