<html dir="ltr"><head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style>@font-face {
font-family: Calibri;
}
@font-face {
font-family: Tahoma;
}
@page WordSection1 {margin: 1.0in 1.0in 1.0in 1.0in; }
P.MsoNormal {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Calibri","sans-serif"; FONT-SIZE: 11pt
}
LI.MsoNormal {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Calibri","sans-serif"; FONT-SIZE: 11pt
}
DIV.MsoNormal {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Calibri","sans-serif"; FONT-SIZE: 11pt
}
A:link {
COLOR: blue; TEXT-DECORATION: underline
}
SPAN.MsoHyperlink {
COLOR: blue; TEXT-DECORATION: underline
}
A:visited {
COLOR: purple; TEXT-DECORATION: underline
}
SPAN.MsoHyperlinkFollowed {
COLOR: purple; TEXT-DECORATION: underline
}
P.MsoAcetate {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Tahoma","sans-serif"; FONT-SIZE: 8pt
}
LI.MsoAcetate {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Tahoma","sans-serif"; FONT-SIZE: 8pt
}
DIV.MsoAcetate {
MARGIN: 0in 0in 0pt; FONT-FAMILY: "Tahoma","sans-serif"; FONT-SIZE: 8pt
}
SPAN.BalloonTextChar {
FONT-FAMILY: "Tahoma","sans-serif"
}
SPAN.EmailStyle19 {
FONT-STYLE: normal; FONT-FAMILY: "Times New Roman","serif"; COLOR: #1f497d; FONT-WEIGHT: normal
}
SPAN.EmailStyle20 {
FONT-FAMILY: "Calibri","sans-serif"; COLOR: #1f497d
}
SPAN.EmailStyle21 {
FONT-STYLE: normal; FONT-FAMILY: "Times New Roman","serif"; COLOR: #1f497d; FONT-WEIGHT: normal
}
SPAN.EmailStyle22 {
FONT-FAMILY: "Calibri","sans-serif"; COLOR: #1f497d
}
.MsoChpDefault {
FONT-SIZE: 10pt
}
DIV.WordSection1 {
}
</style>
<meta name="GENERATOR" content="MSHTML 9.00.8112.16447">
<style id="owaTempEditStyle"></style><style title="owaParaStyle"><!--P {
MARGIN-TOP: 0px; MARGIN-BOTTOM: 0px
}
--></style>
</head>
<body lang="EN-US" vlink="purple" link="blue" ocsi="x">
<div style="FONT-FAMILY: Tahoma; DIRECTION: ltr; COLOR: #000000; FONT-SIZE: x-small">
<div>Hi Alex,</div>
<div><font face="tahoma"> We can make that tweak to change how it checks for Windows 8.</font></div>
<div><font face="tahoma"></font> </div>
<div><font face="tahoma"><font face="tahoma"> </font> Also, I discovered one other item that hasn't affected TRIM, but is not actually correct. In the SntiTranslateBLockLimitsPage function that implements the</font></div>
<div><font face="tahoma"><font face="tahoma">Block Limits VPD page, two items weren't being returned correctly. </font> <span>pBLPage</span><span>-></span><span>MaximumCompareAndWriteLength</span><span> and
<span>pBLPage</span><span>-></span><span>MaximumTransferLength were being assigned the value
</span></span></font></div>
<div><font face="tahoma"><span><span><font face="tahoma">of <span> </span><span>pCntrlIdData</span><span>-></span><span>MDTS -- and what needs to be returned there is not this field, but a calculation based on this field and
<span>PAE</span><span>-></span><span>pCtrlRegister</span><span>-></span><span>CAP</span><span>.</span><span>MPSMIN.
</span></span></font></span></span></font></div>
<div><font face="tahoma"><span><span><font face="tahoma"><span><span></span></span></font></span></span></font> </div>
<div><font face="tahoma"><span><span><span><span><font face="tahoma"> And actually, we have the ability to scale that maximum transfer value back based on a registry setting, so rather than doing that calculation, what should probably be
returned there is <font face="Tahoma"><span>pAE</span><span>-></span><span>InitInfo</span><span>.</span><span>MaxTxSize.
</span></font></font></span></span></span></span></font></div>
<div><font face="tahoma"><span><span><span><span><font face="tahoma"><font face="tahoma"><span></span></font></font></span></span></span></span></font> </div>
<div><font face="tahoma"><span><span><span><span><span><font face="tahoma"> Another issue with what was being done is that the value for these fields within the BLock Limits VPD Page needs to be in blocks, so for both of those, we should
be returning <font face="Tahoma"><span>pAE</span><span>-></span><span>InitInfo</span><span>.</span><span>MaxTxSize / block size.</span></font></font></span></span></span></span></span></font></div>
<div><font face="tahoma"><span><span><span><span><span><font face="tahoma"><font face="tahoma"><span></span></font></font></span></span></span></span></span></font> </div>
<div><span><span><span><span><span><span><font face="tahoma"> We can put in those fixes.</font></span></span></span></span></span></span> (Again, these weren't affecting the UNMAP stuff, but should be corrected.)</div>
<div> </div>
<div> </div>
<div> </div>
<div><span><span><span><span><span><span><font face="tahoma"> Thanks,</font></span></span></span></span></span></span></div>
<div><span><span><span><span><span><span><font face="tahoma"></font></span></span></span></span></span></span> </div>
<div><span><span><span><span><span><span> <font face="tahoma"> -Rick</font></span></span></span></span></span></span></div>
<div><span><span><span><span><span><span><font face="tahoma"></font></span></span></span></span></span></span> </div>
<div style="DIRECTION: ltr" id="divRpF670658">
<hr tabindex="-1">
<font color="#000000" size="2" face="Tahoma"><b>From:</b> Chang, Alex [Alex.Chang@idt.com]<br>
<b>Sent:</b> Tuesday, June 11, 2013 3:48 PM<br>
<b>To:</b> Knoblaugh, Rick; Robles, Raymond C; nvmewin@lists.openfabrics.org<br>
<b>Subject:</b> RE: LSI Trim Patch<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">Hi Rick,</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial"></font></span> </div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">Just a quick note that you don't need to define WINDOWS_8 yourself. In a more WDK way, you may just wrap your changes with:</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">#if (_WIN32_WINNT > _WIN32_WINNT_WIN7)</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">#endif</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">Then, it will be compiled without error on both Windows 8 WDK and 7600 WDK.</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial"></font></span> </div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial">Thanks,<br>
Alex</font></span></div>
<div dir="ltr" align="left"><span class="826034422-11062013"><font color="#0000ff" size="2" face="Arial"></font></span> </div>
<br>
<div dir="ltr" lang="en-us" class="OutlookMessageHeader" align="left">
<hr tabindex="-1">
<font size="2" face="Tahoma"><b>From:</b> nvmewin-bounces@lists.openfabrics.org [mailto:nvmewin-bounces@lists.openfabrics.org]
<b>On Behalf Of </b>Knoblaugh, Rick<br>
<b>Sent:</b> Friday, June 07, 2013 6:38 PM<br>
<b>To:</b> Robles, Raymond C; nvmewin@lists.openfabrics.org<br>
<b>Subject:</b> [nvmewin] ***UNCHECKED*** LSI Trim Patch<br>
</font><br>
</div>
<div></div>
<div class="WordSection1">
<p class="MsoNormal"><span style="COLOR: #1f497d">Hi Ray,</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"> Per your request, since we will switch order, moving Intel patch to the number 3 position, I’m sending LSI’s Trim patch.
</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d">Password for the attached file is: lsi1234. Also, I have attached a document here that describes what was changed/added.
</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d"> It would be great if everyone can review and please let me know any feedback.</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d">Thanks.</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d"> -Rick</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></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"> Robles, Raymond C [mailto:raymond.c.robles@intel.com]
<br>
<b>Sent:</b> Monday, June 03, 2013 5:12 PM<br>
<b>To:</b> Knoblaugh, Rick; nvmewin@lists.openfabrics.org<br>
<b>Subject:</b> RE: Sandisk patch delay</span></p>
</div>
</div>
<p class="MsoNormal"> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Hi Rick,</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">That’s great news! LSI will be 3<sup>rd</sup> in line after IDT and Intel. Thanks for the contributions to TRIM.</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Thanks,</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Ray</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></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"> Knoblaugh, Rick [<a href="mailto:Rick.Knoblaugh@lsi.com">mailto:Rick.Knoblaugh@lsi.com</a>]
<br>
<b>Sent:</b> Monday, June 03, 2013 4:57 PM<br>
<b>To:</b> Robles, Raymond C; <a href="mailto:nvmewin@lists.openfabrics.org">nvmewin@lists.openfabrics.org</a><br>
<b>Subject:</b> RE: Sandisk patch delay</span></p>
</div>
</div>
<p class="MsoNormal"> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d">Hi Ray,</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"> We also have the patch for Trim. It is ready to send. Please let me know when you would like me to send out.</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></span> </p>
<p class="MsoNormal"><span style="COLOR: #1f497d">Thanks,</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"> -Rick</span></p>
<p class="MsoNormal"><span style="COLOR: #1f497d"></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">nvmewin-bounces@lists.openfabrics.org</a> [<a href="mailto:nvmewin-bounces@lists.openfabrics.org">mailto:nvmewin-bounces@lists.openfabrics.org</a>]
<b>On Behalf Of </b>Robles, Raymond C<br>
<b>Sent:</b> Monday, June 03, 2013 4:44 PM<br>
<b>To:</b> <a href="mailto:nvmewin@lists.openfabrics.org">nvmewin@lists.openfabrics.org</a><br>
<b>Subject:</b> [nvmewin] Sandisk patch delay</span></p>
</div>
</div>
<p class="MsoNormal"> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Hello,</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">It appears the Sandisk patch for Mode Sense is taking longer than expected. In order to keep things moving along with the OFA driver, I’m going to take
the Sandisk patch offline for now until they can resolve their issues. Once they have worked out the kinks, they can re-submit. In the meantime, Alex from IDT has a patch he’d like to push and I also have a patch I’d like to push.</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Alex, please send your patch out for code review as soon as possible and then I will send out my patch immediately after.</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Thanks,</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Ray</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"></span> </p>
<p class="MsoNormal"><b><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt"><img id="Picture_x0020_1" border="0" alt="cid:image001.png@01CB3870.4BB88E70" src="cid:826034422@11062013-1A67" width="54" height="39"></span></b></p>
<p class="MsoNormal"><b><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Raymond C. Robles</span></b></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">NVM Solutions Group | Internal SSD Engineering</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Technology & Manufacturing Group</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Intel Corporation</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Desk: 480.554.2600</span></p>
<p class="MsoNormal"><span style="FONT-FAMILY: 'Times New Roman','serif'; COLOR: #1f497d; FONT-SIZE: 10pt">Mobile: 480.399.0645</span><span style="COLOR: #1f497d"></span></p>
<p class="MsoNormal"> </p>
</div>
</div>
</div>
</body>
</html>