[nvmewin] Issue with the latest OFA driver

Mon Nov 14 13:14:02 PST 2016

Thanks Suman/Judy.

I’ll go ahead and pull the fix into the current trunk. Thanks for catching this Tom.

Thanks…
Ray

From: SUMAN PRAKASH B [mailto:suman.p at samsung.com]
Sent: Monday, November 14, 2016 8:06 AM
To: Judy Brock <judy.brock at samsung.com>; Robles, Raymond C <raymond.c.robles at intel.com>; Tom Freeman <thomas.freeman at wdc.com>; nvmewin <nvmewin at lists.openfabrics.org>
Subject: RE: RE: Issue with the latest OFA driver

Ray, Judy,

Yes, it makes good sense to pull in this patch for the upcoming release.

Thanks,

Suman

--------- Original Message ---------

Sender : Judy Brock <judy.brock at samsung.com<mailto:judy.brock at samsung.com>> Engineer Staff/America Office(DS)-R&D-MSL - Enterprise Product Development - R&D/Samsung Electronics

Date : 2016-11-12 04:58 (GMT+5:30)

Title : RE: Issue with the latest OFA driver

Personally, I think it makes good sense. Suman?

From: nvmewin [mailto:nvmewin-bounces at lists.openfabrics.org] On Behalf Of Robles, Raymond C
Sent: Friday, November 11, 2016 10:51 AM
To: Tom Freeman; nvmewin
Subject: Re: [nvmewin] Issue with the latest OFA driver

Thanks Tom.

Samsung (Judy/Suman), do you have any opinion on pulling in this patch for the upcoming release?

Thanks…
Ray

From: Tom Freeman [mailto:thomas.freeman at wdc.com]<mailto:[mailto:thomas.freeman at wdc.com]>
Sent: Friday, November 11, 2016 7:41 AM
To: Robles, Raymond C <raymond.c.robles at intel.com<mailto:raymond.c.robles at intel.com>>; nvmewin <nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>>
Subject: RE: Issue with the latest OFA driver

Ray,
In case you decide to include this fix in the latest release, here are the details:

The attached zip file, Ident-CNTIDPatch.zip (password = nvmehgst) contains:

1.       Source subdirectory containing the modified code based on revision 139.

2.       ControllerIdFix.patch containing the SVN patch to revision 139.

3.       README.txt – description of the problem and the fix.

Tom Freeman
Software Engineer, Device Manager and Driver Development

HGST, a Western Digital brand
Email:  Thomas.freeman at wdc.com<mailto:Thomas.freeman at wdc.com>
Office:  +1-507-322-2311

From: Robles, Raymond C [mailto:raymond.c.robles at intel.com]
Sent: Tuesday, November 1, 2016 1:27 PM
To: Tom Freeman <thomas.freeman at wdc.com<mailto:thomas.freeman at wdc.com>>; nvmewin <nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>>
Subject: RE: Issue with the latest OFA driver

Hi Tom,

Good find!

I’m of the opinion that it makes sense to get this patch in for the release we are about to announce. This is a minor fix for the Identify command that is isolated to two scenarios. Does anyone else have any objections or feel we should wait for this patch?

Depending on the feedback, we can go ahead and push a patch. I would say for now, please get the patch ready and then let’s hold for feedback from the community reviewers… and go from there.

Thank you…
Ray

From: Tom Freeman [mailto:thomas.freeman at wdc.com]
Sent: Tuesday, November 01, 2016 9:08 AM
To: Robles, Raymond C <raymond.c.robles at intel.com<mailto:raymond.c.robles at intel.com>>; nvmewin <nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>>
Subject: Issue with the latest OFA driver

Ray,
We’ve run into the following issue with the latest OFA driver.
During initialization, the driver request for NVME Identify Namespace fails with a status of “invalid field in command”. Because of this, initialization fails and the device does not show up.
The underlying cause is that the driver puts the controller ID value into CDW10.CNTID. According to the NVMe spec, that is only valid when CNS = 0x12 or 0x13. For all other cases, “the host shall clear this field to 0h”

I’d like to make a patch for this. The fix would remove controller ID from any of the instance of NVMe Identify command (the driver does not use CNS = 0x12 or 0x13). How would you like me to proceed to coordinate with the current 1.5 testing?
Thanks,

Tom Freeman
Software Engineer, Device Manager and Driver Development

HGST, a Western Digital brand
Email:  Thomas.freeman at wdc.com<mailto:Thomas.freeman at wdc.com>
Office:  +1-507-322-2311

Western Digital Corporation (and its subsidiaries) E-mail Confidentiality Notice & Disclaimer:
This e-mail and any files transmitted with it may contain confidential or legally privileged information of WDC and/or its affiliates, and are intended solely for the use of the individual or entity to which they are addressed. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited. If you have received this e-mail in error, please notify the sender immediately and delete the e-mail in its entirety from your system.
Western Digital Corporation (and its subsidiaries) E-mail Confidentiality Notice & Disclaimer:
This e-mail and any files transmitted with it may contain confidential or legally privileged information of WDC and/or its affiliates, and are intended solely for the use of the individual or entity to which they are addressed. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited. If you have received this e-mail in error, please notify the sender immediately and delete the e-mail in its entirety from your system.

CAUTION: Please confirm that the password protected
.zip attachment which contains the file(s) of type
  image001.gif, Ident-CNTIDPatch.zip
is legitimate prior to opening.  To make sure this
message is not infected with a virus, it is important to
verify that you are expecting the message or else
confirm its legitimacy with the sender.

[cid:image002.png at 01D23E7F.B04ADA70]

[http://ext.samsung.net/mail/ext/v1/external/status/update?userid=suman.p&do=bWFpbElEPTIwMTYxMTE0MTUwNTMxZXBjbXM1cDZkY2E5M2RiNzgwMGEzNTU1M2ZhMWIyM2Y3ZDA1ZTc4MyZyZWNpcGllbnRBZGRyZXNzPXJheW1vbmQuYy5yb2JsZXNAaW50ZWwuY29t]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20161114/24cbb0e1/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.gif
Type: image/gif
Size: 1117 bytes
Desc: image001.gif
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20161114/24cbb0e1/attachment.gif>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image002.png
Type: image/png
Size: 33527 bytes
Desc: image002.png
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20161114/24cbb0e1/attachment.png>