From: Naohiro Aota <Naohiro.Aota@wdc.com>
To: Karel Zak <kzak@redhat.com>
Cc: "util-linux@vger.kernel.org" <util-linux@vger.kernel.org>,
Damien Le Moal <Damien.LeMoal@wdc.com>
Subject: Re: [PATCH 0/2] add columns for zoned parameters
Date: Wed, 25 Aug 2021 02:12:27 +0000 [thread overview]
Message-ID: <20210825021227.xwt3xl2c3kbmosjb@naota-xeon> (raw)
In-Reply-To: <20210824124729.or5nyoadc3fbla2y@ws.net.home>
On Tue, Aug 24, 2021 at 02:47:29PM +0200, Karel Zak wrote:
> 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.
Nice idea. I'll add a patch for that.
> > 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).
Hmm, considering we already have DISC-{ALN,GRAN,MAX,ZERO}, how about
ZONE-{SZ,GRAN,APP,NR,OMAX,AMAX}?
> Karel
>
> --
> Karel Zak <kzak@redhat.com>
> http://karelzak.blogspot.com
>
next prev parent reply other threads:[~2021-08-25 2:12 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 ` [PATCH 0/2] add columns for " Karel Zak
2021-08-25 2:12 ` Naohiro Aota [this message]
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=20210825021227.xwt3xl2c3kbmosjb@naota-xeon \
--to=naohiro.aota@wdc.com \
--cc=Damien.LeMoal@wdc.com \
--cc=kzak@redhat.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.