* [PATCH v2] btrfs/315: update filter to match mount cmd @ 2024-09-23 7:57 An Long 2024-09-23 11:29 ` Filipe Manana 2024-09-23 13:30 ` Zorro Lang 0 siblings, 2 replies; 6+ messages in thread From: An Long @ 2024-09-23 7:57 UTC (permalink / raw) To: fstests, linux-btrfs; +Cc: lan Mount error info changed since util-linux v2.40 (91ea38e libmount: report failed syscall name). So update _filter_mount_error() to match it. Signed-off-by: An Long <lan@suse.com> --- tests/btrfs/315 | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/btrfs/315 b/tests/btrfs/315 index 5852afad..5101a9a3 100755 --- a/tests/btrfs/315 +++ b/tests/btrfs/315 @@ -39,7 +39,11 @@ _filter_mount_error() # mount: <mnt-point>: fsconfig system call failed: File exists. # dmesg(1) may have more information after failed mount system call. - grep -v dmesg | _filter_test_dir | sed -e "s/mount(2)\|fsconfig//g" + # For util-linux v2.4 and later: + # mount: <mountpoint>: mount system call failed: File exists. + + grep -v dmesg | _filter_test_dir | sed -e "s/mount(2)\|fsconfig//g" | \ + sed -E "s/mount( system call failed:)/\1/" } seed_device_must_fail() -- 2.43.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] btrfs/315: update filter to match mount cmd 2024-09-23 7:57 [PATCH v2] btrfs/315: update filter to match mount cmd An Long @ 2024-09-23 11:29 ` Filipe Manana 2024-09-23 13:30 ` Zorro Lang 1 sibling, 0 replies; 6+ messages in thread From: Filipe Manana @ 2024-09-23 11:29 UTC (permalink / raw) To: An Long; +Cc: fstests, linux-btrfs On Mon, Sep 23, 2024 at 8:57 AM An Long <lan@suse.com> wrote: > > Mount error info changed since util-linux v2.40 > (91ea38e libmount: report failed syscall name). > So update _filter_mount_error() to match it. > > Signed-off-by: An Long <lan@suse.com> Reviewed-by: Filipe Manana <fdmanana@suse.com> Looks good, thanks. > --- > tests/btrfs/315 | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tests/btrfs/315 b/tests/btrfs/315 > index 5852afad..5101a9a3 100755 > --- a/tests/btrfs/315 > +++ b/tests/btrfs/315 > @@ -39,7 +39,11 @@ _filter_mount_error() > # mount: <mnt-point>: fsconfig system call failed: File exists. > # dmesg(1) may have more information after failed mount system > call. > > - grep -v dmesg | _filter_test_dir | sed -e > "s/mount(2)\|fsconfig//g" > + # For util-linux v2.4 and later: > + # mount: <mountpoint>: mount system call failed: File exists. > + > + grep -v dmesg | _filter_test_dir | sed -e > "s/mount(2)\|fsconfig//g" | \ > + sed -E "s/mount( system call failed:)/\1/" > } > > seed_device_must_fail() > -- > 2.43.0 > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] btrfs/315: update filter to match mount cmd 2024-09-23 7:57 [PATCH v2] btrfs/315: update filter to match mount cmd An Long 2024-09-23 11:29 ` Filipe Manana @ 2024-09-23 13:30 ` Zorro Lang 2024-10-07 12:11 ` Filipe Manana 1 sibling, 1 reply; 6+ messages in thread From: Zorro Lang @ 2024-09-23 13:30 UTC (permalink / raw) To: An Long; +Cc: fstests, linux-btrfs On Mon, Sep 23, 2024 at 03:57:13PM +0800, An Long wrote: > Mount error info changed since util-linux v2.40 > (91ea38e libmount: report failed syscall name). > So update _filter_mount_error() to match it. > > Signed-off-by: An Long <lan@suse.com> > --- > tests/btrfs/315 | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tests/btrfs/315 b/tests/btrfs/315 > index 5852afad..5101a9a3 100755 > --- a/tests/btrfs/315 > +++ b/tests/btrfs/315 > @@ -39,7 +39,11 @@ _filter_mount_error() > # mount: <mnt-point>: fsconfig system call failed: File exists. > # dmesg(1) may have more information after failed mount system > call. > > - grep -v dmesg | _filter_test_dir | sed -e > "s/mount(2)\|fsconfig//g" > + # For util-linux v2.4 and later: > + # mount: <mountpoint>: mount system call failed: File exists. > + > + grep -v dmesg | _filter_test_dir | sed -e > "s/mount(2)\|fsconfig//g" | \ > + sed -E "s/mount( system call failed:)/\1/" Oh, there's a local _filter_mount_error() in btrfs/315. I thought you can change the common helper _filter_error_mount() in common/filter. So maybe you can merge this _filter_mount_error into that common helper in another patch, then other test cases can use it. Thanks, Zorro > } > > seed_device_must_fail() > -- > 2.43.0 > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] btrfs/315: update filter to match mount cmd 2024-09-23 13:30 ` Zorro Lang @ 2024-10-07 12:11 ` Filipe Manana 2024-10-11 6:03 ` Zorro Lang 0 siblings, 1 reply; 6+ messages in thread From: Filipe Manana @ 2024-10-07 12:11 UTC (permalink / raw) To: Zorro Lang; +Cc: An Long, fstests, linux-btrfs On Mon, Sep 23, 2024 at 2:30 PM Zorro Lang <zlang@redhat.com> wrote: > > On Mon, Sep 23, 2024 at 03:57:13PM +0800, An Long wrote: > > Mount error info changed since util-linux v2.40 > > (91ea38e libmount: report failed syscall name). > > So update _filter_mount_error() to match it. > > > > Signed-off-by: An Long <lan@suse.com> > > --- > > tests/btrfs/315 | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/tests/btrfs/315 b/tests/btrfs/315 > > index 5852afad..5101a9a3 100755 > > --- a/tests/btrfs/315 > > +++ b/tests/btrfs/315 > > @@ -39,7 +39,11 @@ _filter_mount_error() > > # mount: <mnt-point>: fsconfig system call failed: File exists. > > # dmesg(1) may have more information after failed mount system > > call. > > > > - grep -v dmesg | _filter_test_dir | sed -e > > "s/mount(2)\|fsconfig//g" > > + # For util-linux v2.4 and later: > > + # mount: <mountpoint>: mount system call failed: File exists. > > + > > + grep -v dmesg | _filter_test_dir | sed -e > > "s/mount(2)\|fsconfig//g" | \ > > + sed -E "s/mount( system call failed:)/\1/" > > Oh, there's a local _filter_mount_error() in btrfs/315. I thought you can > change the common helper _filter_error_mount() in common/filter. So maybe > you can merge this _filter_mount_error into that common helper in another > patch, then other test cases can use it. This patch missed the last update. Was the cleanup to use the filter from common/filter required before merging this? Thanks. > > Thanks, > Zorro > > > > } > > > > seed_device_must_fail() > > -- > > 2.43.0 > > > > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] btrfs/315: update filter to match mount cmd 2024-10-07 12:11 ` Filipe Manana @ 2024-10-11 6:03 ` Zorro Lang 2024-10-14 3:51 ` An Long 0 siblings, 1 reply; 6+ messages in thread From: Zorro Lang @ 2024-10-11 6:03 UTC (permalink / raw) To: Filipe Manana; +Cc: An Long, fstests, linux-btrfs On Mon, Oct 07, 2024 at 01:11:35PM +0100, Filipe Manana wrote: > On Mon, Sep 23, 2024 at 2:30 PM Zorro Lang <zlang@redhat.com> wrote: > > > > On Mon, Sep 23, 2024 at 03:57:13PM +0800, An Long wrote: > > > Mount error info changed since util-linux v2.40 > > > (91ea38e libmount: report failed syscall name). > > > So update _filter_mount_error() to match it. > > > > > > Signed-off-by: An Long <lan@suse.com> > > > --- > > > tests/btrfs/315 | 6 +++++- > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/tests/btrfs/315 b/tests/btrfs/315 > > > index 5852afad..5101a9a3 100755 > > > --- a/tests/btrfs/315 > > > +++ b/tests/btrfs/315 > > > @@ -39,7 +39,11 @@ _filter_mount_error() > > > # mount: <mnt-point>: fsconfig system call failed: File exists. > > > # dmesg(1) may have more information after failed mount system > > > call. > > > > > > - grep -v dmesg | _filter_test_dir | sed -e > > > "s/mount(2)\|fsconfig//g" > > > + # For util-linux v2.4 and later: > > > + # mount: <mountpoint>: mount system call failed: File exists. > > > + > > > + grep -v dmesg | _filter_test_dir | sed -e > > > "s/mount(2)\|fsconfig//g" | \ > > > + sed -E "s/mount( system call failed:)/\1/" > > > > Oh, there's a local _filter_mount_error() in btrfs/315. I thought you can > > change the common helper _filter_error_mount() in common/filter. So maybe > > you can merge this _filter_mount_error into that common helper in another > > patch, then other test cases can use it. > > This patch missed the last update. > Was the cleanup to use the filter from common/filter required before > merging this? Thanks Filipe, I'll merge this patch at first. Hi An, this patch has same issue with the other one you sent recently: $ git am -s ./v2_20240923_lan_btrfs_315_update_filter_to_match_mount_cmd.mbx Applying: btrfs/315: update filter to match mount cmd error: corrupt patch at line 12 Patch failed at 0001 btrfs/315: update filter to match mount cmd ... They all have "corrupt" data, maybe you can check the way you generate and send patches, better to not change the .patch file manually before sending. I'll merge this patch manually to catch up the release of this week, so don't need one more version :) Reviewed-by: Zorro Lang <zlang@redhat.com> > > Thanks. > > > > > Thanks, > > Zorro > > > > > > > } > > > > > > seed_device_must_fail() > > > -- > > > 2.43.0 > > > > > > > > > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] btrfs/315: update filter to match mount cmd 2024-10-11 6:03 ` Zorro Lang @ 2024-10-14 3:51 ` An Long 0 siblings, 0 replies; 6+ messages in thread From: An Long @ 2024-10-14 3:51 UTC (permalink / raw) To: Zorro Lang, Filipe Manana; +Cc: fstests, linux-btrfs On Fri, 2024-10-11 at 14:03 +0800, Zorro Lang wrote: > On Mon, Oct 07, 2024 at 01:11:35PM +0100, Filipe Manana wrote: > > On Mon, Sep 23, 2024 at 2:30 PM Zorro Lang <zlang@redhat.com> > > wrote: > > > > > > On Mon, Sep 23, 2024 at 03:57:13PM +0800, An Long wrote: > > > > Mount error info changed since util-linux v2.40 > > > > (91ea38e libmount: report failed syscall name). > > > > So update _filter_mount_error() to match it. > > > > > > > > Signed-off-by: An Long <lan@suse.com> > > > > --- > > > > tests/btrfs/315 | 6 +++++- > > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/tests/btrfs/315 b/tests/btrfs/315 > > > > index 5852afad..5101a9a3 100755 > > > > --- a/tests/btrfs/315 > > > > +++ b/tests/btrfs/315 > > > > @@ -39,7 +39,11 @@ _filter_mount_error() > > > > # mount: <mnt-point>: fsconfig system call failed: File > > > > exists. > > > > # dmesg(1) may have more information after failed mount > > > > system > > > > call. > > > > > > > > - grep -v dmesg | _filter_test_dir | sed -e > > > > "s/mount(2)\|fsconfig//g" > > > > + # For util-linux v2.4 and later: > > > > + # mount: <mountpoint>: mount system call failed: File > > > > exists. > > > > + > > > > + grep -v dmesg | _filter_test_dir | sed -e > > > > "s/mount(2)\|fsconfig//g" | \ > > > > + sed -E "s/mount( system call failed:)/\1/" > > > > > > Oh, there's a local _filter_mount_error() in btrfs/315. I thought > > > you can > > > change the common helper _filter_error_mount() in common/filter. > > > So maybe > > > you can merge this _filter_mount_error into that common helper in > > > another > > > patch, then other test cases can use it. > > > > This patch missed the last update. > > Was the cleanup to use the filter from common/filter required > > before > > merging this? > > Thanks Filipe, I'll merge this patch at first. > > Hi An, this patch has same issue with the other one you sent > recently: > > $ git am -s > ./v2_20240923_lan_btrfs_315_update_filter_to_match_mount_cmd.mbx > Applying: btrfs/315: update filter to match mount cmd > error: corrupt patch at line 12 > Patch failed at 0001 btrfs/315: update filter to match mount cmd > ... > > They all have "corrupt" data, maybe you can check the way you > generate and > send patches, better to not change the .patch file manually before > sending. > I'll merge this patch manually to catch up the release of this week, > so > don't need one more version :) Thank you so much! I've fixed this problem and it won't happen again ^_^ > > Reviewed-by: Zorro Lang <zlang@redhat.com> > > > > > Thanks. > > > > > > > > Thanks, > > > Zorro > > > > > > > > > > } > > > > > > > > seed_device_must_fail() > > > > -- > > > > 2.43.0 > > > > > > > > > > > > > > > > > -- An Long <lan@suse.com> SUSE QE LSG, QE 2, Beijing ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-10-14 3:51 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-09-23 7:57 [PATCH v2] btrfs/315: update filter to match mount cmd An Long 2024-09-23 11:29 ` Filipe Manana 2024-09-23 13:30 ` Zorro Lang 2024-10-07 12:11 ` Filipe Manana 2024-10-11 6:03 ` Zorro Lang 2024-10-14 3:51 ` An Long
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).