* [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).