ACK/Cmnt: [PATCH 0/3] Request-Add support for a NVMe-oF-TCP CDC Client - TP 8010
tim.gardner at canonical.com
Wed Jun 22 14:35:57 UTC 2022
No, it was just an FYI and easy enough to figure out from the LP report.
On 6/22/22 08:26, Michael Reed wrote:
> Hi Tim,
> That is an easy fix, Do you need me to submit a second version of the
> patch with that fix?
> On Wed, Jun 22, 2022 at 7:16 AM Tim Gardner <tim.gardner at canonical.com
> <mailto:tim.gardner at canonical.com>> wrote:
> On 6/21/22 15:53, Michael Reed wrote:
> > From: Michael Reed <Michael.Reed at canonical.com
> <mailto:Michael.Reed at canonical.com>>
> > SRU Justification:
> > [Impact]
> > NVMe-oF suffers from a well-known discovery problem that
> fundamentally limits the size of realistic deployments. To address
> this discovery problem, the FMDS working group (within nvme.org
> <http://nvme.org>) is working on two proposals that will allow
> NVMe-oF to be managed via a “network centric” provisioning process
> instead of an “end-node centric” one.
> > TP-8009
> will enable the Automated Discovery of NVMe-oF Discovery Controllers
> in an IP Network and will prevent an end-user from needing to
> manually configure the IP Address of Discovery Controllers.
> > TP-8010 (this launchpad): will define the concept of a
> Centralized Discovery Controller (CDC) and will allow end-users to
> manage connectivity from a single point of management on an IP
> Fabric by IP Fabric basis.
> > Here is code that implements TP8009 and TP8010:
> > https://github.com/martin-belanger/nvme-stas/
> <https://github.com/martin-belanger/nvme-stas/> which now got pulled
> into upstream - https://github.com/linux-nvme/nvme-stas
> > [Fix]
> > 1. Update kernel with TP8010 kernel patches:
> > a.
> > b.
> > c.
> > nvme: add CNTRLTYPE definitions for 'identify controller
> > [Test Case]
> > 1. Compile libnvme and nvme-stas packages from github using the
> > with these patches.
> > 2. Test libnvme and nvme-stas packages
> > [Where problems could occur]
> > Regression risk for these patches are low
> > [Other Info]
> > Hannes Reinecke (1):
> > nvme: add CNTRLTYPE definitions for 'identify controller'
> > Martin Belanger (2):
> > nvme: send uevent on connection up
> > nvme: expose cntrltype and dctype through sysfs
> > drivers/nvme/host/core.c | 48
> > drivers/nvme/host/nvme.h | 3 +++
> > include/linux/nvme.h | 20 +++++++++++++++--
> > 3 files changed, 69 insertions(+), 2 deletions(-)
> Acked-by: Tim Gardner <tim.gardner at canonical.com
> <mailto:tim.gardner at canonical.com>>
> The cover letter subject should mention Jammy.
> Tim Gardner
> Canonical, Inc
More information about the kernel-team