<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
        {font-family:Wingdings;
        panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:24.0pt;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Calibri",sans-serif;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal0;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
p.msochpdefault, li.msochpdefault, div.msochpdefault
        {mso-style-name:msochpdefault;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:10.0pt;
        font-family:"Times New Roman",serif;}
span.emailstyle19
        {mso-style-name:emailstyle19;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.emailstyle20
        {mso-style-name:emailstyle20;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.emailstyle21
        {mso-style-name:emailstyle21;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.emailstyle22
        {mso-style-name:emailstyle22;
        font-family:"Calibri",sans-serif;
        color:#002060;
        font-weight:normal;
        font-style:normal;
        text-decoration:none none;}
span.emailstyle23
        {mso-style-name:emailstyle23;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.emailstyle24
        {mso-style-name:emailstyle24;
        font-family:"Calibri",sans-serif;
        color:#002060;
        font-weight:normal;
        font-style:normal;}
span.EmailStyle26
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">Hi Tom,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">No worries… thank you very much.
<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">Thanks…<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">Ray<o:p></o:p></span></p>
<p class="MsoNormal"><a name="_MailEndCompose"><span style="font-size:11.0pt;color:#1F497D"><o:p> </o:p></span></a></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt">From:</span></b><span style="font-size:11.0pt"> Tom Freeman [mailto:thomas.freeman@wdc.com]
<br>
<b>Sent:</b> Friday, June 16, 2017 12:47 PM<br>
<b>To:</b> Robles, Raymond C <raymond.c.robles@intel.com>; Justina Lai <justina_lai@phison.com>; nvmewin@lists.openfabrics.org<br>
<b>Cc:</b> umaparepalli@gmail.com; Larry Li <larry_li@phison.com><br>
<b>Subject:</b> Re: [nvmewin] Compatibility issue with 1.5 version nvme driver<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Ray, I just discovered this email - it was routed to my junk folder. I'll review early next week<o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span style="font-family:"Times New Roman",serif"><o:p> </o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Tom Freeman<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Software Engineer, Device Manager and Driver Development<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Western Digital<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Email: <a href="mailto:Thomas.Freeman@wdc.com">
Thomas.Freeman@wdc.com</a><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Office:+1-507-322-2311<o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span style="font-family:"Times New Roman",serif"><o:p> </o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">-------- Original Message --------<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Subject: Re: [nvmewin] Compatibility issue with 1.5 version nvme driver<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">From: "Robles, Raymond C" <<a href="mailto:raymond.c.robles@intel.com">raymond.c.robles@intel.com</a>><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Date: Jun 13, 2017, 3:31 PM<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">To: Justina Lai <<a href="mailto:justina_lai@phison.com">justina_lai@phison.com</a>>,nvmewin@lists.openfabrics.org<o:p></o:p></span></p>
</div>
<div>
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Hi Justina,</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">I’m still waiting for feedback from Samsung and HGST on your patch.
</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><b><span style="font-size:11.0pt;color:#002060;background:yellow">Tom/Judy/Suman</span></b><span style="font-size:11.0pt;color:#002060;background:yellow">,
</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060;background:yellow"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060;background:yellow">Can you please provide feedback on Phison’s patch for the BSOD fix for grabbing the same lock twice?</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Thanks…</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Ray</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt">From:</span></b><span style="font-size:11.0pt"> Justina Lai [<a href="mailto:justina_lai@phison.com">mailto:justina_lai@phison.com</a>]
<br>
<b>Sent:</b> Thursday, May 25, 2017 7:08 PM<br>
<b>To:</b> Robles, Raymond C <<a href="mailto:raymond.c.robles@intel.com">raymond.c.robles@intel.com</a>>;
<a href="mailto:nvmewin@lists.openfabrics.org">nvmewin@lists.openfabrics.org</a><br>
<b>Cc:</b> Larry Li <<a href="mailto:larry_li@phison.com">larry_li@phison.com</a>>; 'umaparepalli@gmail.com' <<a href="mailto:umaparepalli@gmail.com">umaparepalli@gmail.com</a>><br>
<b>Subject:</b> RE: Compatibility issue with 1.5 version nvme driver</span><o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Hi Raymond,<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks for reply.<o:p></o:p></p>
<p class="MsoNormal">Please find attachment.<o:p></o:p></p>
<p class="MsoNormal">It’s modified based on 4/8 157 version.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks!<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<div>
<p class="MsoNormal">Best Regards,<o:p></o:p></p>
<p class="MsoNormal">Justina Lai<o:p></o:p></p>
<p class="MsoNormal">#5707<o:p></o:p></p>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt">From:</span></b><span style="font-size:11.0pt"> Robles, Raymond C [<a href="mailto:raymond.c.robles@intel.com">mailto:raymond.c.robles@intel.com</a>]
<br>
<b>Sent:</b> Friday, May 26, 2017 1:36 AM<br>
<b>To:</b> Justina Lai <<a href="mailto:justina_lai@phison.com">justina_lai@phison.com</a>>;
<a href="mailto:nvmewin@lists.openfabrics.org">nvmewin@lists.openfabrics.org</a><br>
<b>Cc:</b> Larry Li <<a href="mailto:larry_li@phison.com">larry_li@phison.com</a>>; 'umaparepalli@gmail.com' <<a href="mailto:umaparepalli@gmail.com">umaparepalli@gmail.com</a>><br>
<b>Subject:</b> RE: Compatibility issue with 1.5 version nvme driver</span><o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Hi Justina,</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">The issue you mention below is a known issue. I thought there was a patch pushed to fix this, but if not, then my recommendation is to provide a patch that resolves this issue.
</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">The main issue is for commands that require 2 child commands, the lock is acquired a second time for the second time on the completions side of the first child command. It should be a very simple
 fix. </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Normally, I request that community members who find the issue submit patches to resolve this issue. This is our model. Could you please provide the patch for the fix?</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><b><span style="font-size:11.0pt;color:#002060">NOTE: Uma Parepalli is the new OFA chair and will handle this issue moving forward.
</span></b><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060">Thanks…<br>
Ray</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#002060"> </span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt">From:</span></b><span style="font-size:11.0pt"> nvmewin [<a href="mailto:nvmewin-bounces@lists.openfabrics.org">mailto:nvmewin-bounces@lists.openfabrics.org</a>]
<b>On Behalf Of </b>Justina Lai<br>
<b>Sent:</b> Thursday, May 25, 2017 1:03 AM<br>
<b>To:</b> <a href="mailto:nvmewin@lists.openfabrics.org">nvmewin@lists.openfabrics.org</a><br>
<b>Cc:</b> Larry Li <<a href="mailto:larry_li@phison.com">larry_li@phison.com</a>><br>
<b>Subject:</b> Re: [nvmewin] Compatibility issue with 1.5 version nvme driver</span><o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Dear Sir/Madam,<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">We are facing compatibility issue on our PS5007 NVMe device with recent 1.5 version nvme driver.<o:p></o:p></p>
<p class="MsoNormal">Please check below issue description.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Phison PS5007 supports 7 IO queues, and if we use PS5007 with OFA driver on the platform core number>7, ex: 8-core or 12-core PC, we will see PC hang up and cannot enter OS like below:<o:p></o:p></p>
<p class="MsoNormal"><img border="0" width="538" height="302" id="_x0000_i1025" src="cid:image001.jpg@01D2E6AF.FAA0E780"><o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">After debugging on our side, we found the fail is caused by below flow:<o:p></o:p></p>
<p class="MsoNormal">---------------------------------------------------------------------------------------------------------------------------------------------------------------------<o:p></o:p></p>
<p class="MsoNormal">……….<o:p></o:p></p>
<p class="MsoNormal" style="margin-left:.25in;text-indent:-.25in">1.<span style="font-size:7.0pt;font-family:"Times New Roman",serif">     
</span>SntiTranslateModeSense()<o:p></o:p></p>
<p class="MsoListParagraph" style="margin-left:.25in">……….<o:p></o:p></p>
<p class="MsoNormal">2.IoCompletionRoutine()<o:p></o:p></p>
<p class="MsoNormal">    if (pDpc != NULL) {<o:p></o:p></p>
<p class="MsoNormal">        ASSERT(pAE->ntldrDump == FALSE);<o:p></o:p></p>
<p class="MsoNormal">        if (<span style="color:red">pAE->MultipleCoresToSingleQueueFlag</span>) {<o:p></o:p></p>
<p class="MsoNormal">            <span style="color:red">StorPortAcquireSpinLock(pAE, StartIoLock, NULL, &StartLockHandle); --------------------</span><span style="font-family:Wingdings;color:red">à</span><span style="color:red"> execute lock</span><o:p></o:p></p>
<p class="MsoNormal">        } else {<o:p></o:p></p>
<p class="MsoNormal">            StorPortAcquireSpinLock(pAE, DpcLock, pDpc, &DpcLockhandle);<o:p></o:p></p>
<p class="MsoNormal">        }<o:p></o:p></p>
<p class="MsoNormal">    }<o:p></o:p></p>
<p class="MsoNormal">………<o:p></o:p></p>
<p class="MsoNormal">callStorportNotification = pSrbExtension->pNvmeCompletionRoutine(pAE, (PVOID)pSrbExtension)<o:p></o:p></p>
<p class="MsoNormal">                            && (pSrbExtension->pSrb != NULL);<o:p></o:p></p>
<p class="MsoNormal">…….<o:p></o:p></p>
<p class="MsoNormal">3.SntiTranslateModeSenseResponse()<o:p></o:p></p>
<p class="MsoListParagraph" style="margin-left:.25in">…….<o:p></o:p></p>
<p class="MsoNormal">case MODE_SENSE_RETURN_ALL:<o:p></o:p></p>
<p class="MsoNormal">…….<o:p></o:p></p>
<p class="MsoNormal">if (supportsVwc == TRUE) {<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">                                                          pSrbExt->pNvmeCompletionRoutine = SntiCompletionCallbackRoutine;<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">                                                          /* Finally, make sure we issue the GET FEATURES command */<o:p></o:p></p>
<p class="MsoNormal">                                                          SntiBuildGetFeaturesCmd(pSrbExt, VOLATILE_WRITE_CACHE);<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">                                                          ioStarted =
<span style="color:red">ProcessIo</span>(pSrbExt->pNvmeDevExt,<o:p></o:p></p>
<p class="MsoNormal">                                                                   pSrbExt,<o:p></o:p></p>
<p class="MsoNormal">                                                                   NVME_QUEUE_TYPE_ADMIN,<o:p></o:p></p>
<p class="MsoNormal">                                                                  
<span style="color:red">TRUE</span>);<o:p></o:p></p>
<p class="MsoNormal">                            <o:p></o:p></p>
<p class="MsoNormal">                            <o:p></o:p></p>
<p class="MsoNormal">4.ProcessIo()   <o:p></o:p></p>
<p class="MsoNormal">        ……..<o:p></o:p></p>
<p class="MsoNormal">        if (<span style="color:red">AcquireLock == TRUE</span>) {<o:p></o:p></p>
<p class="MsoNormal">            <span style="color:red">StorPortAcquireSpinLock</span>(pAdapterExtension,<o:p></o:p></p>
<p class="MsoNormal">                                 StartIoLock,<o:p></o:p></p>
<p class="MsoNormal">                                 NULL,<o:p></o:p></p>
<p class="MsoNormal">                                 &hStartIoLock);  <span style="color:red">
----------------------</span><span style="font-family:Wingdings;color:red">à</span><span style="color:red"> double execute lock and cause (Assertion failure - code c0000420)</span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:red"> </span><o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">        }        <o:p></o:p></p>
<p class="MsoNormal">---------------------------------------------------------------------------------------------------------------------------------------------------------------------<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Since many users are now using our PS5007 device with 8-core or 12-core PC, they are facing this issue right now. Could you please help to modify driver to solve this problem asap?<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Any unclear point, please kindly let us know.<o:p></o:p></p>
<p class="MsoNormal">Thank you very much for the help!<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Best Regards,<o:p></o:p></p>
<p class="MsoNormal">Justina Lai<o:p></o:p></p>
<p class="MsoNormal">#5707<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif"><br>
</span><span style="font-family:"Arial",sans-serif;color:gray"><br>
This message and any attachments are confidential and may be legally privileged. Any unauthorized review, use or distribution by anyone other than the intended recipient is strictly prohibited. If you are not the intended recipient, please immediately notify
 the sender, completely delete the message and any attachments, and destroy all copies. Your cooperation will be highly appreciated.</span><o:p></o:p></p>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif"><br>
</span><span style="font-family:"Arial",sans-serif;color:gray"><br>
This message and any attachments are confidential and may be legally privileged. Any unauthorized review, use or distribution by anyone other than the intended recipient is strictly prohibited. If you are not the intended recipient, please immediately notify
 the sender, completely delete the message and any attachments, and destroy all copies. Your cooperation will be highly appreciated.</span><o:p></o:p></p>
</div>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">Western Digital Corporation (and its subsidiaries) E-mail Confidentiality Notice & Disclaimer:<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-family:"Times New Roman",serif">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.<o:p></o:p></span></p>
</div>
</div>
</div>
</body>
</html>