From: Karel Zak <kzak@redhat.com>
To: Naohiro Aota <naohiro.aota@wdc.com>
Cc: util-linux@vger.kernel.org
Subject: Re: [PATCH 0/2] add columns for zoned parameters
Date: Tue, 24 Aug 2021 14:47:29 +0200 [thread overview]
Message-ID: <20210824124729.or5nyoadc3fbla2y@ws.net.home> (raw)
In-Reply-To: <20210824011718.300747-1-naohiro.aota@wdc.com>
On Tue, Aug 24, 2021 at 10:17:16AM +0900, Naohiro Aota wrote:
> Several parameters for zoned devices are missing from lsblk's columns. This
> series introduces them as following.
Thanks! I thought about it a few weeks ago ;-)
>
> Column Name Description Sysfs path
> ---------------------------------------------------------------------
> ZONE-SIZE zone size queue/chunk_sectors
> ZONE-WGRAN zone write granularity queue/zone_write_granularity
> ZONE-APPEND zone append max bytes queue/zone_append_max_bytes
> ZONES-NR number of zones queue/nr_zones
> ZONES-OMAX max open zones queue/max_open_zones
> ZONES-AMAX max active zones queue/max_active_zones
>
> Sample output:
>
> $ lsblk -o NAME,ZONED,ZONE-SIZE,ZONE-WGRAN,ZONE-APPEND,ZONES-NR,ZONES-OMAX,ZONES-AMAX -i
What about to add all the columns to "--zoned" output too?
We do not keep backward compatibility for set of columns in outputs
like --zoned, so it's open for changes. I guess we can use --zoned as
a generic option for all zoned stuff and not for a model only.
> NAME ZONED ZONE-SIZE ZONE-WGRAN ZONE-APPEND ZONES-NR ZONES-OMAX ZONES-AMAX
> sda host-managed 256M 4K 672K 55880 128 0
> sdb host-managed 256M 4K 672K 55880 128 0
> zram0 none 0B 0B 0B 0 0 0
> nvme2n1 none 0B 0B 0B 0 0 0
> |-nvme2n1p1 none 0B 0B 0B 0 0 0
> |-nvme2n1p2 none 0B 0B 0B 0 0 0
> `-nvme2n1p3 none 0B 0B 0B 0 0 0
> nvme0n1 none 0B 0B 0B 0 0 0
> nvme1n1 none 0B 0B 0B 0 0 0
> nvme0n2 host-managed 2G 4K 4M 1844 14 14
> nvme1n2 host-managed 2G 4K 4M 1844 14 14
>
> $ lsblk --help|grep -i zone
> -z, --zoned print zone model
> ZONED zone model
> ZONE-SIZE zone size
> ZONE-WGRAN zone write granularity
> ZONE-APPEND zone append max bytes
> ZONES-NR number of zones
> ZONES-OMAX max open zones
> ZONES-AMAX max active zones
It would be nice to have shorter column names, but I do not have any
sane suggestion (Z-SIZE or Z-APPEND seems strange).
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
next prev parent reply other threads:[~2021-08-24 12:47 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-24 1:17 [PATCH 0/2] add columns for zoned parameters Naohiro Aota
2021-08-24 1:17 ` [PATCH 1/2] lsblk: factor out function to read sysfs param as bytes Naohiro Aota
2021-08-24 1:17 ` [PATCH 2/2] lsblk: add columns of zoned parameters Naohiro Aota
2021-08-24 12:47 ` Karel Zak [this message]
2021-08-25 2:12 ` [PATCH 0/2] add columns for " Naohiro Aota
2021-08-25 2:49 ` Damien Le Moal
2021-08-25 10:03 ` Karel Zak
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210824124729.or5nyoadc3fbla2y@ws.net.home \
--to=kzak@redhat.com \
--cc=naohiro.aota@wdc.com \
--cc=util-linux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.