diff mbox series

btrfs/253: skip on zoned mode as we cannot change the chunk size

Message ID 20220726075759.3155494-1-naohiro.aota@wdc.com (mailing list archive)
State New, archived
Headers show
Series btrfs/253: skip on zoned mode as we cannot change the chunk size | expand

Commit Message

Naohiro Aota July 26, 2022, 7:57 a.m. UTC
On zoned mode, we have a fixed chunk size which is equal to the zone size.
So, we cannot change the chunk size, and running this test results in a
failure with below.

    --- tests/btrfs/253.out     2021-12-10 04:33:53.000000000 +0000
    +++ /host/results/btrfs/253.out.bad 2022-07-26 05:58:10.000000000 +0000
    @@ -2,9 +2,16 @@
     Capture default chunk sizes.
     First allocation.
     Second allocation.
    +./common/rc: line 4670: echo: write error: Invalid argument
    +./common/rc: line 4670: echo: write error: Invalid argument
     Calculate request size so last memory allocation cannot be completely fullfilled.
     Third allocation.
    ...

It is no use to test this feature on zoned mode. So, just skip it.

Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
---
 tests/btrfs/253 | 2 ++
 1 file changed, 2 insertions(+)

Comments

Zorro Lang July 26, 2022, 9:49 a.m. UTC | #1
On Tue, Jul 26, 2022 at 04:57:59PM +0900, Naohiro Aota wrote:
> On zoned mode, we have a fixed chunk size which is equal to the zone size.
> So, we cannot change the chunk size, and running this test results in a
> failure with below.
> 
>     --- tests/btrfs/253.out     2021-12-10 04:33:53.000000000 +0000
>     +++ /host/results/btrfs/253.out.bad 2022-07-26 05:58:10.000000000 +0000
>     @@ -2,9 +2,16 @@
>      Capture default chunk sizes.
>      First allocation.
>      Second allocation.
>     +./common/rc: line 4670: echo: write error: Invalid argument
>     +./common/rc: line 4670: echo: write error: Invalid argument
>      Calculate request size so last memory allocation cannot be completely fullfilled.
>      Third allocation.
>     ...
> 
> It is no use to test this feature on zoned mode. So, just skip it.
> 
> Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
> ---

Make sense to me,

Reviewed-by: Zorro Lang <zlang@redhat.com>

>  tests/btrfs/253 | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/btrfs/253 b/tests/btrfs/253
> index fbbb81fae754..c746f41e9264 100755
> --- a/tests/btrfs/253
> +++ b/tests/btrfs/253
> @@ -81,6 +81,8 @@ alloc_size() {
>  _supported_fs btrfs
>  _require_test
>  _require_scratch
> +# The chunk size on zoned mode is fixed to the zone size
> +_require_non_zoned_device "$SCRATCH_DEV"
>  
>  # Delete log file if it exists.
>  rm -f "${seqres}.full"
> -- 
> 2.35.1
>
diff mbox series

Patch

diff --git a/tests/btrfs/253 b/tests/btrfs/253
index fbbb81fae754..c746f41e9264 100755
--- a/tests/btrfs/253
+++ b/tests/btrfs/253
@@ -81,6 +81,8 @@  alloc_size() {
 _supported_fs btrfs
 _require_test
 _require_scratch
+# The chunk size on zoned mode is fixed to the zone size
+_require_non_zoned_device "$SCRATCH_DEV"
 
 # Delete log file if it exists.
 rm -f "${seqres}.full"