Results 6 comments of majianpeng

@sdpeters Based on your email, I briefly looked at the api of libspdk_nvmf(https://github.com/spdk/spdk/blob/master/lib/nvmf/spdk_nvmf.map).  From the api name, it seems that it can meet the basic needs of discovery: set the...

@sdpeters . Yesterday i based on https://github.com/spdk/spdk/blob/master/examples/nvmf/nvmf/nvmf.c did some tests to verify whether can add subsystem: 1: using spdk_nvmf_transport_create(https://github.com/spdk/spdk/blob/master/include/spdk/nvmf.h#L979) to create a tcp transport 2: using spdk_nvmf_tgt_add_transport to register(https://github.com/spdk/spdk/blob/master/include/spdk/nvmf.h#L1069) 3:...

> "Even with an unauthenticated host ID (so, possibly before https://github.com/ceph/ceph-nvmeof/issues/67) it would be useful if the DS could advertise only the ports each host needs for the namespaces it...

@sdpeters Your reason make sense. Other question is: we choose namespace masking. So why we still need "Host-specific discovery service responses". I mean with https://github.com/PepperJo/spdk/commit/094a9d70f5a8f32313d06b62c0d0f9d3e6ba74ca, we can add all namespace...

@sdpeters, I agree on the reason of this feature. About how to implement host-specific discovery response, in your email, you said "register a user-defined nvmf_generate_discovery_log function". Whether can do more...

@sdpeters For the firstly method, I feel that the current nvmeof/iscsi are all in this way. It need advertise to all host. For second method, one subsystem need all namespace...