<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=iso-8859-1">
<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:0cm;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman",serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0cm;
        mso-margin-bottom-alt:auto;
        margin-left:0cm;
        font-size:12.0pt;
        font-family:"Times New Roman",serif;}
span.EmailStyle20
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:70.85pt 70.85pt 2.0cm 70.85pt;}
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="DE" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">Hi Michihiro and Götz,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">thanks, Götz, for reviewing.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">Michihiro, can I add you as 2<sup>nd</sup> reviewer (no need to be jdk-reviewer for that)? I think you have taken a look and you obviously
 ran tests. So if you’re ok with the change, I can push it next week.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">Our nightly tests look good, too.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">Thanks and best regards,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US">Martin<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Michihiro Horie [mailto:HORIE@jp.ibm.com]
<br>
<b>Sent:</b> Freitag, 2. März 2018 11:55<br>
<b>To:</b> Doerr, Martin <martin.doerr@sap.com><br>
<b>Cc:</b> Lindenmaier, Goetz <goetz.lindenmaier@sap.com>; Gustavo Romero <gromero@linux.vnet.ibm.com>; Hiroshi H Horii (HORII@jp.ibm.com) <HORII@jp.ibm.com>; 'hotspot-compiler-dev@openjdk.java.net' <hotspot-compiler-dev@openjdk.java.net><br>
<b>Subject:</b> Re: RFR(L): 8198894: [PPC64] More generic vector CRC implementation<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p><span style="font-size:10.0pt">Hi Martin,</span><br>
<br>
<span style="font-size:10.0pt">I double checked performance with our micro benchmark.</span><br>
<span style="font-size:10.0pt">This change was 5 times faster. In addition, I did not observe degradation with smaller length but have almost equal performance.</span><br>
<br>
<span style="font-size:10.0pt">Best regards,</span><br>
<span style="font-size:10.0pt">--</span><br>
<span style="font-size:10.0pt">Michihiro,</span><br>
<span style="font-size:10.0pt">IBM Research - Tokyo</span><br>
<br>
<img width="16" height="16" style="width:.1666in;height:.1666in" id="_x0000_i1025" src="cid:image001.gif@01D3B7D2.D108DAF0" alt="Inactive hide details for "Doerr, Martin" ---2018/03/02 00:49:56---Hi, I have implemented a more generic version of the vector "><span style="font-size:10.0pt;color:#424282">"Doerr,
 Martin" ---2018/03/02 00:49:56---Hi, I have implemented a more generic version of the vector instruction based CRC code.</span><br>
<br>
<span style="font-size:10.0pt;color:#5F5F5F">From: </span><span style="font-size:10.0pt">"Doerr, Martin" <<a href="mailto:martin.doerr@sap.com">martin.doerr@sap.com</a>></span><br>
<span style="font-size:10.0pt;color:#5F5F5F">To: </span><span style="font-size:10.0pt">"'hotspot-compiler-dev@openjdk.java.net'" <<a href="mailto:hotspot-compiler-dev@openjdk.java.net">hotspot-compiler-dev@openjdk.java.net</a>></span><br>
<span style="font-size:10.0pt;color:#5F5F5F">Cc: </span><span style="font-size:10.0pt">"Lindenmaier, Goetz" <<a href="mailto:goetz.lindenmaier@sap.com">goetz.lindenmaier@sap.com</a>>, "Hiroshi H Horii (<a href="mailto:HORII@jp.ibm.com">HORII@jp.ibm.com</a>)"
 <<a href="mailto:HORII@jp.ibm.com">HORII@jp.ibm.com</a>>, "Michihiro Horie (<a href="mailto:HORIE@jp.ibm.com">HORIE@jp.ibm.com</a>)" <<a href="mailto:HORIE@jp.ibm.com">HORIE@jp.ibm.com</a>>, Gustavo Romero <<a href="mailto:gromero@linux.vnet.ibm.com">gromero@linux.vnet.ibm.com</a>></span><br>
<span style="font-size:10.0pt;color:#5F5F5F">Date: </span><span style="font-size:10.0pt">2018/03/02 00:49</span><br>
<span style="font-size:10.0pt;color:#5F5F5F">Subject: </span><span style="font-size:10.0pt">RFR(L): 8198894: [PPC64] More generic vector CRC implementation</span><o:p></o:p></p>
<div class="MsoNormal">
<hr size="2" width="100%" noshade="" style="color:#8091A5" align="left">
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
<br>
<br>
<span style="font-family:"Calibri",sans-serif">Hi,</span><br>
<br>
<span style="font-family:"Calibri",sans-serif">I have implemented a more generic version of the vector instruction based CRC code.</span><br>
<span style="font-family:"Calibri",sans-serif">It supports CRC32C and Big Endian, too.</span><br>
<br>
<span style="font-family:"Calibri",sans-serif">The peak performance was even better for large input streams. I got almost 40GB/s.</span><br>
<span style="font-family:"Calibri",sans-serif">Some smaller length may be slower than with the old version.</span><br>
<span style="font-family:"Calibri",sans-serif">Maybe somebody from IBM would like to double-check performance.</span><br>
<br>
<span style="font-family:"Calibri",sans-serif">Please review:</span><br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Emdoerr_8198894-5FPPC64-5FCRC32_webrev.00_&d=DwMFAg&c=jf_iaSHvJObTbx-siA1ZOg&r=oecsIpYF-cifqq2i1JEH0Q&m=XoFQj6pl5KM4yrslTVuMvukjPfiyiw1vyK-FHCveKpc&s=VBCFi423koKhU902olYdRrAWTLdINLCJg_2BNXKfysE&e="><span style="font-family:"Calibri",sans-serif;color:#0563C1">http://cr.openjdk.java.net/~mdoerr/8198894_PPC64_CRC32/webrev.00/</span></a><br>
<span style="font-family:"Calibri",sans-serif">(hoping you like math </span><span style="font-family:Wingdings">J</span><span style="font-family:"Calibri",sans-serif">)</span><br>
<br>
<span style="font-family:"Calibri",sans-serif">Best regards,</span><br>
<span style="font-family:"Calibri",sans-serif">Martin</span><br>
<br>
<o:p></o:p></p>
</div>
</body>
</html>