From: jsmart2021@gmail.com (James Smart)
Subject: [PATCH 0/5] nvmet/nvmet_fc: add events for discovery controller rescan
Date: Wed, 1 Nov 2017 09:03:37 -0700 [thread overview]
Message-ID: <7b7521f3-5412-4969-41dc-c5d87eacd72f@gmail.com> (raw)
In-Reply-To: <20171101153817.GB13553@infradead.org>
On 11/1/2017 8:38 AM, Christoph Hellwig wrote:
> On Sun, Oct 29, 2017@09:43:14PM -0700, James Smart wrote:
>> Currently, the FC-NVME standard strongly suggests a discovery controller on
>> each target.
>
> Yikes. This is a new requirement not backed by anything in NVMe or
> NVMeoF itself. Please have discussion in the technical working group
> on this behavior first.
>
> It's not that I'm against it - but I really want this sort of optional
> transport behavior clearly documented in the actual NVMe spec first.
>
See my last email.
The "strongly suggest" is laymans language for the word "should" that
was used in the standard. Should was used so that discovery of the
available discovery controllers on FC could be done in an automatic and
dynamic way without apriori knowledge. Any FC target device is free to
choose if and where they implement discovery controllers, just as on any
other transport in nvmf.
We have had discussions in the technical working group and with nvme
leadership on this and as I said, its well within what a transport can
do. It's doing nothing illegal or odd. It doesn't require explicit
documentation in a NVMe spec. It is in a NVME fabrics-based transport
specification.
-- james
next prev parent reply other threads:[~2017-11-01 16:03 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-28 17:21 [PATCH 0/5] nvmet/nvmet_fc: add events for discovery controller rescan James Smart
2017-10-28 17:21 ` [PATCH 1/5] nvmet: call transport on subsystem add and delete James Smart
2017-10-28 17:21 ` [PATCH 2/5] nvmet_fc: support transport subsystem events James Smart
2017-10-28 17:21 ` [PATCH 3/5] lpfc: Add support to generate RSCN events for nport James Smart
2017-10-28 17:21 ` [PATCH 4/5] lpfc: Add NVME rescan support via RSCNs James Smart
2017-10-28 17:21 ` [PATCH 5/5] lpfc: Add sysfs interface to post NVME RSCN James Smart
2017-11-02 20:09 ` Ewan D. Milne
2017-11-02 22:02 ` James Smart
2017-10-29 16:11 ` [PATCH 0/5] nvmet/nvmet_fc: add events for discovery controller rescan Sagi Grimberg
2017-10-30 4:43 ` James Smart
2017-11-01 15:38 ` Christoph Hellwig
2017-11-01 16:03 ` James Smart [this message]
2017-11-01 16:28 ` Christoph Hellwig
2017-11-01 15:36 ` Christoph Hellwig
2017-11-01 15:55 ` James Smart
2017-11-01 15:57 ` Christoph Hellwig
2017-11-01 16:12 ` James Smart
2017-11-01 16:29 ` Christoph Hellwig
[not found] <20171029011456.12885-1-jsmart2021@gmail.com>
2017-10-31 12:22 ` Martin K. Petersen
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=7b7521f3-5412-4969-41dc-c5d87eacd72f@gmail.com \
--to=jsmart2021@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).