[nvmewin] binary for the pending release

[Luse, Paul E]

Cool, do you believe we need to hold the release for these - if so can you explain the impact of not doing so?

Thx
Paul

From: Chang, Alex [mailto:Alex.Chang at idt.com]
Sent: Thursday, April 26, 2012 6:14 PM
To: Luse, Paul E; nvmewin at lists.openfabrics.org
Subject: RE: binary for the pending release

I see. I will prepare a new patch then.

Thanks,
Alex
________________________________
From: Luse, Paul E [paul.e.luse at intel.com]
Sent: Thursday, April 26, 2012 4:55 PM
To: Chang, Alex; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: binary for the pending release
The patch has already been applied so go ahead and start a new one based on the latest, make the changes, test and send to the list for review.

Thx
Paul

From: Chang, Alex [mailto:Alex.Chang at idt.com]<mailto:[mailto:Alex.Chang at idt.com]>
Sent: Thursday, April 26, 2012 4:46 PM
To: Luse, Paul E; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: binary for the pending release

Hi Paul,

I had fixed a race condition issue when using INTx interrupt and several errors found in nvme.h and nvmereg.h. I wonder if it's okay to add the changes to your patch. Please let me know.

Thanks,
Alex

________________________________
From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org]<mailto:[mailto:nvmewin-bounces at lists.openfabrics.org]> On Behalf Of Luse, Paul E
Sent: Wednesday, April 25, 2012 8:55 AM
To: nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: [nvmewin] ***UNCHECKED*** binary for the pending release
pw is intel123.  A few quick notes:


-         Both x86/64 fre and chk builds are included here

-         Source patch for these will be applied shortly

-         3 addt'l changes since our review

o   Our ID ctrl stuct was missing 2 fields, IEEE and MAC

o   The struct for the COAL feature cmd had two fields in the wrong order

o   There was a typo in the INF for the x86 catalog file addition

-         I've run tests on the 64 bit versions so far, will look at 32 bit soon.

-         The attached are signed for the convenience of testing on the 64 bit.  I actually don't have an issue posting these with my signature, again, for convenience only.  Any issues with that from everyone?

Also, we'll be covering the attached 2 slides at the NVME meeting this week, please review real quick and send any feedback by EOD

Thx
Paul

____________________________________
Paul Luse
Sr. Staff Engineer
PCG Server Software Engineering
Desk: 480.554.3688, Mobile: 480.334.4630

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20120427/b961936a/attachment.html>