Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/31/2017 11:46 AM, Christoph Hellwig wrote: > On Tue, May 30, 2017 at 11:45:25AM +0200, Johannes Thumshirn wrote: >> Mostly consistency. The current nvme host code has the EUI sprinkled all >> around. Sure I can drop it, but then what's the point in evaluating it >> on the host side? Other

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/31/2017 11:46 AM, Christoph Hellwig wrote: > On Tue, May 30, 2017 at 11:45:25AM +0200, Johannes Thumshirn wrote: >> Mostly consistency. The current nvme host code has the EUI sprinkled all >> around. Sure I can drop it, but then what's the point in evaluating it >> on the host side? Other

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On Tue, May 30, 2017 at 11:45:25AM +0200, Johannes Thumshirn wrote: > Mostly consistency. The current nvme host code has the EUI sprinkled all > around. Sure I can drop it, but then what's the point in evaluating it > on the host side? Other targets may send it, so we need in on the host > and do

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On Tue, May 30, 2017 at 11:45:25AM +0200, Johannes Thumshirn wrote: > Mostly consistency. The current nvme host code has the EUI sprinkled all > around. Sure I can drop it, but then what's the point in evaluating it > on the host side? Other targets may send it, so we need in on the host > and do

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/30/2017 11:25 AM, Christoph Hellwig wrote: > On Tue, May 30, 2017 at 10:08:18AM +0200, Johannes Thumshirn wrote: >> Add the EUI-64 field from the NVMe Namespace Identification Descriptor >> to the nvmet_ns structure and allow it's population via configfs. > > Is there any good use case for

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/30/2017 11:25 AM, Christoph Hellwig wrote: > On Tue, May 30, 2017 at 10:08:18AM +0200, Johannes Thumshirn wrote: >> Add the EUI-64 field from the NVMe Namespace Identification Descriptor >> to the nvmet_ns structure and allow it's population via configfs. > > Is there any good use case for

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On Tue, May 30, 2017 at 10:08:18AM +0200, Johannes Thumshirn wrote: > Add the EUI-64 field from the NVMe Namespace Identification Descriptor > to the nvmet_ns structure and allow it's population via configfs. Is there any good use case for bothering with this identifier that's too short to

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On Tue, May 30, 2017 at 10:08:18AM +0200, Johannes Thumshirn wrote: > Add the EUI-64 field from the NVMe Namespace Identification Descriptor > to the nvmet_ns structure and allow it's population via configfs. Is there any good use case for bothering with this identifier that's too short to

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/30/2017 10:08 AM, Johannes Thumshirn wrote: > Add the EUI-64 field from the NVMe Namespace Identification Descriptor > to the nvmet_ns structure and allow it's population via configfs. > > Signed-off-by: Johannes Thumshirn > --- > drivers/nvme/target/configfs.c | 48 >

Re: [PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

On 05/30/2017 10:08 AM, Johannes Thumshirn wrote: > Add the EUI-64 field from the NVMe Namespace Identification Descriptor > to the nvmet_ns structure and allow it's population via configfs. > > Signed-off-by: Johannes Thumshirn > --- > drivers/nvme/target/configfs.c | 48 >

[PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

Add the EUI-64 field from the NVMe Namespace Identification Descriptor to the nvmet_ns structure and allow it's population via configfs. Signed-off-by: Johannes Thumshirn --- drivers/nvme/target/configfs.c | 48 ++

[PATCH 3/7] nvmet: add eui64 field to nvme_ns and populate via configfs

Add the EUI-64 field from the NVMe Namespace Identification Descriptor to the nvmet_ns structure and allow it's population via configfs. Signed-off-by: Johannes Thumshirn --- drivers/nvme/target/configfs.c | 48 ++ drivers/nvme/target/nvmet.h| 1 + 2