<div>Hi Ray,</div>
<div> </div>
<div>I <span style="LINE-HEIGHT:115%;FONT-FAMILY:'Calibri','sans-serif';FONT-SIZE:11pt">apologize </span> as I am looking for old trunk which returns the "status" received by NVMePowerMgm() parameter directly. In new update it always returns the "TRUE". Thanks for your notice.</div>
<div> </div>
<div>Regards,</div>
<div>Ravi Kumar<br><br></div>
<div class="gmail_quote">On Thu, May 3, 2012 at 12:20 AM, Robles, Raymond C <span dir="ltr"><<a href="mailto:raymond.c.robles@intel.com" target="_blank">raymond.c.robles@intel.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div lang="EN-US" vlink="purple" link="blue">
<div>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt">I must admit that I didn’t quite understand the original problem stated and I should have asked for clarification sooner. Ravi, can you please explain the problem that you see in the SRB_FUNCTION_POWER case by using line numbers from the source code so that we can track where the potential issue before submitting a code patch?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt">Are you inferring that the StartIo function will return FALSE even when the power request has been successfully completed? As Alex mentioned, the StartIo function will return TRUE in the case of handling a SRB_FUNCTION_POWER request. However, please note that we call StorPortNotification with the correct SrbStatus immediately after the call to NVMePowerControl. This will complete the command appropriately back to Storport (with the correct status). The StartIo will always return TRUE indicating the command was successfully started.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt">Thanks,<br>Ray<u></u><u></u></span></p>
<p class="MsoNormal"><span style="COLOR:#1f497d;FONT-SIZE:10pt"><u></u> <u></u></span></p>
<div>
<div style="BORDER-BOTTOM:medium none;BORDER-LEFT:medium none;PADDING-BOTTOM:0in;PADDING-LEFT:0in;PADDING-RIGHT:0in;BORDER-TOP:#b5c4df 1pt solid;BORDER-RIGHT:medium none;PADDING-TOP:3pt">
<p class="MsoNormal"><b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt">From:</span></b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt"> <a href="mailto:nvmewin-bounces@lists.openfabrics.org" target="_blank">nvmewin-bounces@lists.openfabrics.org</a> [mailto:<a href="mailto:nvmewin-bounces@lists.openfabrics.org" target="_blank">nvmewin-bounces@lists.openfabrics.org</a>] <b>On Behalf Of </b>Chang, Alex<br>
<b>Sent:</b> Wednesday, May 02, 2012 9:23 AM<br><b>To:</b> Ravi Kumar; Luse, Paul E
<div>
<div class="h5"><br><b>Cc:</b> <a href="mailto:nvmewin@lists.openfabrics.org" target="_blank">nvmewin@lists.openfabrics.org</a><br><b>Subject:</b> Re: [nvmewin] StartIo Function Return value for NVMePowerControl<u></u><u></u></div>
</div></span>
<p></p></p></div></div>
<div>
<div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><span style="FONT-FAMILY:'Arial','sans-serif';COLOR:blue;FONT-SIZE:10pt">Hi Ravi,</span><u></u><u></u></p>
<p class="MsoNormal"> <u></u><u></u></p>
<p class="MsoNormal"><span style="FONT-FAMILY:'Arial','sans-serif';COLOR:blue;FONT-SIZE:10pt">For SRB_FUNCTION_POWER requests, NVMeStartIo always returns TRUE (Line 1205 in nvmestd.c) to indicate the request had been successfully initiated/processed. I am not sure why you mentioned it returns FALSE to Storport driver?</span><u></u><u></u></p>
<p class="MsoNormal"> <u></u><u></u></p>
<p class="MsoNormal"><span style="FONT-FAMILY:'Arial','sans-serif';COLOR:blue;FONT-SIZE:10pt">Thanks,</span><u></u><u></u></p>
<p class="MsoNormal"><span style="FONT-FAMILY:'Arial','sans-serif';COLOR:blue;FONT-SIZE:10pt">Alex</span><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="TEXT-ALIGN:center" class="MsoNormal" align="center">
<hr align="center" size="2" width="100%">
</div>
<p style="MARGIN-BOTTOM:12pt" class="MsoNormal"><b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt">From:</span></b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt"> <a href="mailto:nvmewin-bounces@lists.openfabrics.org" target="_blank">nvmewin-bounces@lists.openfabrics.org</a> <a href="mailto:[mailto:nvmewin-bounces@lists.openfabrics.org]" target="_blank">[mailto:nvmewin-bounces@lists.openfabrics.org]</a> <b>On Behalf Of </b>Ravi Kumar<br>
<b>Sent:</b> Tuesday, May 01, 2012 8:42 PM<br><b>To:</b> Luse, Paul E<br><b>Cc:</b> <a href="mailto:nvmewin@lists.openfabrics.org" target="_blank">nvmewin@lists.openfabrics.org</a><br><b>Subject:</b> Re: [nvmewin] StartIo Function Return value for NVMePowerControl</span><u></u><u></u></p>
<div>
<p class="MsoNormal">Hi Paul,<u></u><u></u></p></div>
<div>
<p class="MsoNormal"> <u></u><u></u></p></div>
<div>
<p class="MsoNormal">I'll put the patch with proper fix in a day and update the same.<u></u><u></u></p></div>
<div>
<p class="MsoNormal"> <u></u><u></u></p></div>
<div>
<p class="MsoNormal">Regards,<u></u><u></u></p></div>
<div>
<p style="MARGIN-BOTTOM:12pt" class="MsoNormal">Ravi Kumar<u></u><u></u></p></div>
<div>
<p class="MsoNormal">On Wed, May 2, 2012 at 1:14 AM, Luse, Paul E <<a href="mailto:paul.e.luse@intel.com" target="_blank">paul.e.luse@intel.com</a>> wrote:<u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"><span style="FONT-FAMILY:'Calibri','sans-serif';COLOR:#1f497d;FONT-SIZE:10pt">Thanks for the note Ravi. Can you please either put a patch together with the fix or simply enter a bug at <a href="https://www.openfabrics.org/resources/developer-tools/nvme-windows-development.html" target="_blank">https://www.openfabrics.org/resources/developer-tools/nvme-windows-development.html</a>. Using the list to simply report something is a good way to make sure it gets lost </span><span style="FONT-FAMILY:Wingdings;COLOR:#1f497d;FONT-SIZE:10pt">J</span><u></u><u></u></p>
<p class="MsoNormal"> <u></u><u></u></p>
<p class="MsoNormal"><b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt">From:</span></b><span style="FONT-FAMILY:'Tahoma','sans-serif';FONT-SIZE:10pt"> <a href="mailto:nvmewin-bounces@lists.openfabrics.org" target="_blank">nvmewin-bounces@lists.openfabrics.org</a> [mailto:<a href="mailto:nvmewin-bounces@lists.openfabrics.org" target="_blank">nvmewin-bounces@lists.openfabrics.org</a>] <b>On Behalf Of </b>Ravi Kumar<br>
<b>Sent:</b> Sunday, April 29, 2012 10:32 PM<br><b>To:</b> <a href="mailto:nvmewin@lists.openfabrics.org" target="_blank">nvmewin@lists.openfabrics.org</a><br><b>Subject:</b> [nvmewin] StartIo Function Return value for NVMePowerControl</span><u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
<div>
<p class="MsoNormal">Hi,<u></u><u></u></p></div>
<div>
<p class="MsoNormal"> <u></u><u></u></p></div>
<div>
<p class="MsoNormal">As per observation on Windows the function NVMeStarIo(NVMeStd.c) for Power management, in case NVMePowerControl()<u></u><u></u></p></div>
<div>
<p class="MsoNormal">returns false it is still completed the request in NVMeStartIo and returns FALSE to StorPort driver. Whereas in case of StartIo returns FALSE StorPort driver treats IO has some issue and try to complete its own in AdapterControl call which create the BSOD to system (Multiple_IO_Request_Complete error).<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"> Don't we required to put the check in case failure of handle the power management system need not to be complete the Io as PortDriver makes it complete its own.<u></u><u></u></p></div>
<div>
<p class="MsoNormal"> <u></u><u></u></p></div>
<div>
<p class="MsoNormal"> <u></u><u></u></p></div>
<div>
<p class="MsoNormal">Regards,<u></u><u></u></p></div>
<div>
<p class="MsoNormal">Ravi Kumar<u></u><u></u></p></div></div></div></div></div></div>
<p class="MsoNormal"><br><br clear="all"><br>-- <u></u><u></u></p>
<div>
<p class="MsoNormal">Regards,<u></u><u></u></p></div>
<div>
<p class="MsoNormal">Ravi<u></u><u></u></p></div>
<p class="MsoNormal"><u></u> <u></u></p></div></div></div></div></blockquote></div><br><br clear="all"><br>-- <br>
<div>Regards,</div>
<div>Ravi</div><br>