From: Matias Bjorling <Matias.Bjorling@wdc.com> To: Niklas Cassel <Niklas.Cassel@wdc.com>, Damien Le Moal <Damien.LeMoal@wdc.com> Cc: "axboe@kernel.dk" <axboe@kernel.dk>, "kbusch@kernel.org" <kbusch@kernel.org>, "hch@lst.de" <hch@lst.de>, "sagi@grimberg.me" <sagi@grimberg.me>, "martin.petersen@oracle.com" <martin.petersen@oracle.com>, Hans Holmberg <Hans.Holmberg@wdc.com>, "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>, "linux-nvme@lists.infradead.org" <linux-nvme@lists.infradead.org> Subject: RE: [PATCH 1/2] block: add zone_desc_ext_bytes to sysfs Date: Mon, 29 Jun 2020 09:07:24 +0000 [thread overview] Message-ID: <MN2PR04MB6223CBCEA07D63E53B3AD244F16E0@MN2PR04MB6223.namprd04.prod.outlook.com> (raw) In-Reply-To: <20200629090356.GA521026@localhost.localdomain> > -----Original Message----- > From: Niklas Cassel <Niklas.Cassel@wdc.com> > Sent: Monday, 29 June 2020 11.04 > To: Damien Le Moal <Damien.LeMoal@wdc.com> > Cc: Matias Bjorling <Matias.Bjorling@wdc.com>; axboe@kernel.dk; > kbusch@kernel.org; hch@lst.de; sagi@grimberg.me; > martin.petersen@oracle.com; Hans Holmberg <Hans.Holmberg@wdc.com>; > linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > block@vger.kernel.org; linux-nvme@lists.infradead.org > Subject: Re: [PATCH 1/2] block: add zone_desc_ext_bytes to sysfs > > On Mon, Jun 29, 2020 at 12:52:46AM +0000, Damien Le Moal wrote: > > On 2020/06/29 8:01, Matias Bjorling wrote: > > > The NVMe Zoned Namespace Command Set adds support for associating > > > data to a zone through the Zone Descriptor Extension feature. > > > > > > The Zone Descriptor Extension size is fixed to a multiple of 64 > > > bytes. A value of zero communicates the feature is not available. > > > A value larger than zero communites the feature is available, and > > > the specified Zone Descriptor Extension size in bytes. > > > > > > The Zone Descriptor Extension feature is only available in the NVMe > > > Zoned Namespaces Command Set. Devices that supports ZAC/ZBC > > > therefore reports this value as zero, where as the NVMe device > > > driver reports the Zone Descriptor Extension size from the specific > > > device. > > > > > > Signed-off-by: Matias Bjørling <matias.bjorling@wdc.com> > > > --- > > > Documentation/block/queue-sysfs.rst | 6 ++++++ > > > block/blk-sysfs.c | 15 ++++++++++++++- > > > drivers/nvme/host/zns.c | 1 + > > > drivers/scsi/sd_zbc.c | 1 + > > > include/linux/blkdev.h | 22 ++++++++++++++++++++++ > > > 5 files changed, 44 insertions(+), 1 deletion(-) > > > > > > diff --git a/Documentation/block/queue-sysfs.rst > > > b/Documentation/block/queue-sysfs.rst > > > index f261a5c84170..c4fa195c87b4 100644 > > (snip) > > > > static struct queue_sysfs_entry queue_nomerges_entry = { > > > .attr = {.name = "nomerges", .mode = 0644 }, > > > .show = queue_nomerges_show, > > > @@ -787,6 +798,7 @@ static struct attribute *queue_attrs[] = { > > > &queue_nr_zones_entry.attr, > > > &queue_max_open_zones_entry.attr, > > > &queue_max_active_zones_entry.attr, > > > > Which tree is this patch based on ? Not I have seen any patch > > introducing max active zones. > > The cover letter forgot to mention this patch series' dependencies. > I assume that it is based on the following patch series: > https://lore.kernel.org/linux-block/20200622162530.1287650-1- > kbusch@kernel.org/ > https://lore.kernel.org/linux-block/20200616102546.491961-1- > niklas.cassel@wdc.com/ > Very true. Thanks, Niklas. I'll rebase this work when the above patchsets have been picked up. > > Kind regards, > Niklas
WARNING: multiple messages have this Message-ID (diff)
From: Matias Bjorling <Matias.Bjorling@wdc.com> To: Niklas Cassel <Niklas.Cassel@wdc.com>, Damien Le Moal <Damien.LeMoal@wdc.com> Cc: "axboe@kernel.dk" <axboe@kernel.dk>, "sagi@grimberg.me" <sagi@grimberg.me>, "martin.petersen@oracle.com" <martin.petersen@oracle.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-nvme@lists.infradead.org" <linux-nvme@lists.infradead.org>, "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>, Hans Holmberg <Hans.Holmberg@wdc.com>, "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>, "kbusch@kernel.org" <kbusch@kernel.org>, "hch@lst.de" <hch@lst.de> Subject: RE: [PATCH 1/2] block: add zone_desc_ext_bytes to sysfs Date: Mon, 29 Jun 2020 09:07:24 +0000 [thread overview] Message-ID: <MN2PR04MB6223CBCEA07D63E53B3AD244F16E0@MN2PR04MB6223.namprd04.prod.outlook.com> (raw) In-Reply-To: <20200629090356.GA521026@localhost.localdomain> > -----Original Message----- > From: Niklas Cassel <Niklas.Cassel@wdc.com> > Sent: Monday, 29 June 2020 11.04 > To: Damien Le Moal <Damien.LeMoal@wdc.com> > Cc: Matias Bjorling <Matias.Bjorling@wdc.com>; axboe@kernel.dk; > kbusch@kernel.org; hch@lst.de; sagi@grimberg.me; > martin.petersen@oracle.com; Hans Holmberg <Hans.Holmberg@wdc.com>; > linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > block@vger.kernel.org; linux-nvme@lists.infradead.org > Subject: Re: [PATCH 1/2] block: add zone_desc_ext_bytes to sysfs > > On Mon, Jun 29, 2020 at 12:52:46AM +0000, Damien Le Moal wrote: > > On 2020/06/29 8:01, Matias Bjorling wrote: > > > The NVMe Zoned Namespace Command Set adds support for associating > > > data to a zone through the Zone Descriptor Extension feature. > > > > > > The Zone Descriptor Extension size is fixed to a multiple of 64 > > > bytes. A value of zero communicates the feature is not available. > > > A value larger than zero communites the feature is available, and > > > the specified Zone Descriptor Extension size in bytes. > > > > > > The Zone Descriptor Extension feature is only available in the NVMe > > > Zoned Namespaces Command Set. Devices that supports ZAC/ZBC > > > therefore reports this value as zero, where as the NVMe device > > > driver reports the Zone Descriptor Extension size from the specific > > > device. > > > > > > Signed-off-by: Matias Bjørling <matias.bjorling@wdc.com> > > > --- > > > Documentation/block/queue-sysfs.rst | 6 ++++++ > > > block/blk-sysfs.c | 15 ++++++++++++++- > > > drivers/nvme/host/zns.c | 1 + > > > drivers/scsi/sd_zbc.c | 1 + > > > include/linux/blkdev.h | 22 ++++++++++++++++++++++ > > > 5 files changed, 44 insertions(+), 1 deletion(-) > > > > > > diff --git a/Documentation/block/queue-sysfs.rst > > > b/Documentation/block/queue-sysfs.rst > > > index f261a5c84170..c4fa195c87b4 100644 > > (snip) > > > > static struct queue_sysfs_entry queue_nomerges_entry = { > > > .attr = {.name = "nomerges", .mode = 0644 }, > > > .show = queue_nomerges_show, > > > @@ -787,6 +798,7 @@ static struct attribute *queue_attrs[] = { > > > &queue_nr_zones_entry.attr, > > > &queue_max_open_zones_entry.attr, > > > &queue_max_active_zones_entry.attr, > > > > Which tree is this patch based on ? Not I have seen any patch > > introducing max active zones. > > The cover letter forgot to mention this patch series' dependencies. > I assume that it is based on the following patch series: > https://lore.kernel.org/linux-block/20200622162530.1287650-1- > kbusch@kernel.org/ > https://lore.kernel.org/linux-block/20200616102546.491961-1- > niklas.cassel@wdc.com/ > Very true. Thanks, Niklas. I'll rebase this work when the above patchsets have been picked up. > > Kind regards, > Niklas _______________________________________________ Linux-nvme mailing list Linux-nvme@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-nvme
next prev parent reply other threads:[~2020-06-29 21:16 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-06-28 23:01 [PATCH 0/2] Zone Descriptor Extension for Zoned Block Devices Matias Bjørling 2020-06-28 23:01 ` Matias Bjørling 2020-06-28 23:01 ` [PATCH 1/2] block: add zone_desc_ext_bytes to sysfs Matias Bjørling 2020-06-28 23:01 ` Matias Bjørling 2020-06-29 0:52 ` Damien Le Moal 2020-06-29 0:52 ` Damien Le Moal 2020-06-29 9:03 ` Niklas Cassel 2020-06-29 9:03 ` Niklas Cassel 2020-06-29 9:07 ` Matias Bjorling [this message] 2020-06-29 9:07 ` Matias Bjorling 2020-06-28 23:01 ` [PATCH 2/2] block: add BLKSETDESCZONE ioctl for Zoned Block Devices Matias Bjørling 2020-06-28 23:01 ` Matias Bjørling 2020-06-29 1:00 ` Damien Le Moal 2020-06-29 1:00 ` Damien Le Moal 2020-06-29 19:39 ` Javier González 2020-06-29 19:39 ` Javier González 2020-07-03 9:44 ` Matias Bjorling 2020-07-03 9:44 ` Matias Bjorling 2020-07-07 8:43 ` Javier González 2020-07-07 8:43 ` Javier González 2020-07-07 16:03 ` Matias Bjorling 2020-07-07 16:03 ` Matias Bjorling 2020-06-29 1:36 ` Bart Van Assche 2020-06-29 1:36 ` Bart Van Assche 2020-06-30 13:28 ` Matias Bjorling 2020-06-30 13:28 ` Matias Bjorling
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=MN2PR04MB6223CBCEA07D63E53B3AD244F16E0@MN2PR04MB6223.namprd04.prod.outlook.com \ --to=matias.bjorling@wdc.com \ --cc=Damien.LeMoal@wdc.com \ --cc=Hans.Holmberg@wdc.com \ --cc=Niklas.Cassel@wdc.com \ --cc=axboe@kernel.dk \ --cc=hch@lst.de \ --cc=kbusch@kernel.org \ --cc=linux-block@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-nvme@lists.infradead.org \ --cc=linux-scsi@vger.kernel.org \ --cc=martin.petersen@oracle.com \ --cc=sagi@grimberg.me \ /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: linkBe 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.