Re: [PATCH] btrfs/284: list a couple btrfs-progs git commits

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Wed, Mar 08, 2023 at 04:03:57PM +0800, Anand Jain wrote:
> On 07/03/2023 19:38, fdmanana@xxxxxxxxxx wrote:
> > From: Filipe Manana <fdmanana@xxxxxxxx>
> > 
> > This test may often fail when running with btrfs-progs versions not very
> > recent. The corresponding git commits in btrfs-progs that fix issues
> > uncovered by this test are:
> > 
> > 1) 6f4a51886b37 ("btrfs-progs: receive: fix silent data loss after fall back from encoded write")
> >     Introduced in btrfs-progs v6.0.2;
> > 
> > 2) e3209f8792f4 ("btrfs-progs: receive: fix a corruption when decompressing zstd extents"")
> >     Introduced in btrfs-progs v6.2.
> > 
> > So add the corresponding _fixed_by_git_commit calls to the test.
> > 
> > Signed-off-by: Filipe Manana <fdmanana@xxxxxxxx>
> > ---
> >   tests/btrfs/284 | 5 +++++
> >   1 file changed, 5 insertions(+)
> > 
> > diff --git a/tests/btrfs/284 b/tests/btrfs/284
> > index 0d31e5d9..c6692668 100755
> > --- a/tests/btrfs/284
> > +++ b/tests/btrfs/284
> > @@ -20,6 +20,11 @@ _require_test
> >   _require_scratch_size $(($LOAD_FACTOR * 1 * 1024 * 1024))
> >   _require_fssum
> > +_fixed_by_git_commit btrfs-progs e3209f8792f4 \
> > +	"btrfs-progs: receive: fix a corruption when decompressing zstd extents"
> > +_fixed_by_git_commit btrfs-progs 6f4a51886b37 \
> > +	"btrfs-progs: receive: fix silent data loss after fall back from encoded write"
> > +
> >   send_files_dir=$TEST_DIR/btrfs-test-$seq
> >   rm -fr $send_files_dir
> 
> 
> Along with this, why not check the btrfs-progs version using
> 'btrfs --version' and call _not_run()?

Does this case expose some known bugs, right? Or the failures due to some
features missing?

If this case uncovers some known issues, then the failure is expected on unfixed
version. We should let the failure exposing, not hide it by _notrun.

And the package version is not a good way to jundge if a issue is fixed or a
feature is merged. Due to many downstream packages might merge some upstream
patches independently.

Thanks,
Zorro

> 
> Thanks, Anand
> 




[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux