[nvmewin] New Patch - Mode Select

[Dharani Kotte]

In the function NVMeInitCallback() case NVMeWaitOnLearnMapping: has to be fixed to take care of the condition where the Reads can fail during learning process but still the MSI-x vector is vaild. Hence the NUMA mapping as apart of learning should not be failed.

In the else case where we are failing the learning we should set pMMT->Shared = TRUE otherwise the interupt will not get serviced.

Thanks,
Dharani.

From: nvmewin-bounces at lists.openfabrics.org [mailto:nvmewin-bounces at lists.openfabrics.org] On Behalf Of Chang, Alex
Sent: Friday, February 08, 2013 9:08 AM
To: Kong, Kwok; nvmewin at lists.openfabrics.org
Subject: Re: [nvmewin] New Patch - Mode Select

Yes, you're right, Kwok.
Paul/Ray, Rick, please let us know if you approve it.

Thanks,
Alex

________________________________
From: Kong, Kwok
Sent: Friday, February 08, 2013 9:02 AM
To: Chang, Alex; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: New Patch - Mode Select
Alex,

We cannot check it in yet. We have to wait for the formal approval from intel and LSI.

Intel and LSI, please approve the patch.

Thanks

-Kwok

From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org] On Behalf Of Chang, Alex
Sent: Thursday, February 07, 2013 6:21 PM
To: nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: [nvmewin] ***UNCHECKED*** RE: New Patch - Mode Select

Hi all,

It's been two weeks and I did not receive more feedbacks. Based on the feedbacks from Dharani and Ray, I modified the patch and re-tested it. Please find the attached patch and review it (password: idt123). Since the changes are pretty straightforward, I'd like to ask Ray's favor to add it to the base if no more feedbacks received by the end of tomorrow. Thanks a lot, Ray.
Here is the summary of the changes:
In function SntiTranslateModeData (nvmesnti.c):
- Fixed some typos.
- Fixed miscalculation of the pointer to Mode Parameter Descriptor Block.
- Fixed DWORD11's value assignment in case of MODE_PAGE_CACHING.
In function NVMeInitCallBack (nvmeinit.c):
- Fixed a typo on Line#1796.

Thanks,
Alex


________________________________
From: Robles, Raymond C [mailto:raymond.c.robles at intel.com]
Sent: Friday, January 25, 2013 5:13 PM
To: Chang, Alex; Dharani Kotte; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: New Patch - Mode Select
Hi Alex,

Yes this is a copy and paste error.  Dharani is correct... it should read pModeHeader10->BlockDescriptorLength;

Did you want to change this in your patch since it is directly related to Mode Select?

Thanks,
Ray

From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org] On Behalf Of Chang, Alex
Sent: Friday, January 25, 2013 4:57 PM
To: Dharani Kotte; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: Re: [nvmewin] New Patch - Mode Select

Good catch, Dharani.

Hi Ray,

Is this just a typo?

Thanks,
Alex
________________________________
From: Dharani Kotte [mailto:Dharani.Kotte at sandisk.com]
Sent: Friday, January 25, 2013 3:46 PM
To: Chang, Alex; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: New Patch - Mode Select
    Hi Alex,

In the file nvmeSnti.c and function SntiTranslateModeData()

/* Mode Select 10 */
        pModeHeader10 = (PMODE_PARAMETER_HEADER10)(GET_DATA_BUFFER(pSrb));

#pragma prefast(suppress:6011,"This pointer is not NULL")
        blockDescLength = pModeHeader6->BlockDescriptorLength;  // BUG -  pModeHeader6 is not initialized probably it should be pModeHeader10

thanks,
Dharani.

From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org] On Behalf Of Chang, Alex
Sent: Friday, January 25, 2013 3:27 PM
To: nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: [WARNING - ENCRYPTED ATTACHMENT NOT VIRUS SCANNED] [nvmewin] ***UNCHECKED*** New Patch - Mode Select

Hi all,

I am adding a patch to fix a bug related to Mode Select command when users enable/disable Volatile Write Cache. The changes I made are within SntiTranslateModeData routine of nvmeSnti.c to correctly locate the page code and program Dword 11 of NVMe submission entry before issuing Set Features command. The password is idt123. Please review it and provide feedbacks in next couple weeks.

Thanks,
Alex

________________________________

PLEASE NOTE: The information contained in this electronic mail message is intended only for the use of the designated recipient(s) named above. If the reader of this message is not the intended recipient, you are hereby notified that you have received this message in error and that any review, dissemination, distribution, or copying of this message is strictly prohibited. If you have received this communication in error, please notify the sender by telephone or e-mail (as shown above) immediately and destroy any and all copies of this message in your possession (whether hard copies or electronically stored copies).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20130208/38ed11ff/attachment.html>