[v3,0/2] btrfs: test active zone tracking

Message

Naohiro Aota Oct. 5, 2022, 5:46 a.m. UTC

This series adds a test for checking if an active zone tracking feature of
btrfs's zoned mode. The first patch introduces _zone_capacity() helper to
get the zone capacity of a specified zone. It rewrites btrfs/237 with the
helper and use the helper in a newly added test btrfs/292.

Changes:
- v3:
  - Fix import of common/zoned in btrfs/237. 
  - Use _fixed_by_kernel_commit.
  - Cleanup background dd processes.
  - Rework error handling.
  - Fix indent.
- v2:
  - Rename to common/zoned.
  - Move _filter_blkzone_report() as well to common/zoned.
  - Drop _require_fio as it was already unnecessary.

Naohiro Aota (2):
  common: introduce zone_capacity() to return a zone capacity
  btrfs: test active zone tracking

 common/filter       |  13 ----
 common/zoned        |  39 ++++++++++++
 tests/btrfs/237     |   8 +--
 tests/btrfs/292     | 143 ++++++++++++++++++++++++++++++++++++++++++++
 tests/btrfs/292.out |   2 +
 5 files changed, 186 insertions(+), 19 deletions(-)
 create mode 100644 common/zoned
 create mode 100755 tests/btrfs/292
 create mode 100644 tests/btrfs/292.out

Comments

Zorro Lang Oct. 7, 2022, 5:02 a.m. UTC | #1

On Wed, Oct 05, 2022 at 02:46:42PM +0900, Naohiro Aota wrote:
> This series adds a test for checking if an active zone tracking feature of
> btrfs's zoned mode. The first patch introduces _zone_capacity() helper to
> get the zone capacity of a specified zone. It rewrites btrfs/237 with the
> helper and use the helper in a newly added test btrfs/292.

I have no more review points on this version now, will push it in next fstests
release, if no objection from others. Thanks!

> 
> Changes:
> - v3:
>   - Fix import of common/zoned in btrfs/237. 
>   - Use _fixed_by_kernel_commit.
>   - Cleanup background dd processes.
>   - Rework error handling.
>   - Fix indent.
> - v2:
>   - Rename to common/zoned.
>   - Move _filter_blkzone_report() as well to common/zoned.
>   - Drop _require_fio as it was already unnecessary.
> 
> Naohiro Aota (2):
>   common: introduce zone_capacity() to return a zone capacity
>   btrfs: test active zone tracking
> 
>  common/filter       |  13 ----
>  common/zoned        |  39 ++++++++++++
>  tests/btrfs/237     |   8 +--
>  tests/btrfs/292     | 143 ++++++++++++++++++++++++++++++++++++++++++++
>  tests/btrfs/292.out |   2 +
>  5 files changed, 186 insertions(+), 19 deletions(-)
>  create mode 100644 common/zoned
>  create mode 100755 tests/btrfs/292
>  create mode 100644 tests/btrfs/292.out
> 
> -- 
> 2.38.0
>