<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=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"MS Mincho";
        panose-1:2 2 6 9 4 2 5 8 3 4;}
@font-face
        {font-family:宋体;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:"\@MS Mincho";
        panose-1:2 2 6 9 4 2 5 8 3 4;}
@font-face
        {font-family:"\@宋体";
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 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.MsoPlainText, li.MsoPlainText, div.MsoPlainText
        {mso-style-priority:99;
        mso-style-link:"纯文本 Char";
        margin:0cm;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"MS Mincho";}
span.EmailStyle17
        {mso-style-type:personal-reply;
        font-family:"Arial","sans-serif";
        color:#1F497D;}
span.Char
        {mso-style-name:"纯文本 Char";
        mso-style-priority:99;
        mso-style-link:纯文本;
        font-family:"MS Mincho";}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 90.0pt 72.0pt 90.0pt;}
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="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"MS Mincho"">Hi, Alex<o:p></o:p></span></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"MS Mincho"">I refered to the previous patches. Should the Upstream-Status be “Submitted” ?</span><span style="font-size:11.0pt;font-family:"Arial","sans-serif";color:#1F497D"><o:p></o:p></span></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">Best regards <o:p></o:p></p>
<p class="MsoPlainText">Lei<o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Arial","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Arial","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Alexander Kanavin [mailto:alex.kanavin@gmail.com]
<br>
<b>Sent:</b> Thursday, June 13, 2019 8:45 PM<br>
<b>To:</b> Burton, Ross<br>
<b>Cc:</b> Lei, Maohui/</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:宋体">雷</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">
</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:宋体">茂慧</span><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">; Hong Liu; OE-core<br>
<b>Subject:</b> Re: [OE-core] [PATCH] openssl: Fix a build bug on aarch64BE.<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">Also if the patch is already submitted, the status should not be Pending.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Alex<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">On Thu, 13 Jun 2019 at 11:23, Burton, Ross <<a href="mailto:ross.burton@intel.com">ross.burton@intel.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<p class="MsoNormal">On Thu, 13 Jun 2019 at 09:48, Lei Maohui <<a href="mailto:leimaohui@cn.fujitsu.com" target="_blank">leimaohui@cn.fujitsu.com</a>> wrote:<br>
><br>
> From: Hong Liu <<a href="mailto:hongl.fnst@cn.fujitsu.com" target="_blank">hongl.fnst@cn.fujitsu.com</a>><br>
><br>
> Fix bug as following on aarch64BE:<br>
><br>
> Error: operand 1 must be an integer register -- `rev v31.16b,v31.16b'<br>
><br>
> Upstream-Status: Pending [<a href="https://github.com/openssl/openssl/pull/9151" target="_blank">https://github.com/openssl/openssl/pull/9151</a>]<br>
<br>
This line ^^^<br>
<br>
> Signed-off-by: Lei Maohui <<a href="mailto:leimaohui@cn.fujitsu.com" target="_blank">leimaohui@cn.fujitsu.com</a>><br>
> ---<br>
> ...01-Fix-build-error-for-aarch64-big-endian.patch | 41 ++++++++++++++++++++++<br>
> .../recipes-connectivity/openssl/<a href="http://openssl_1.1.1c.bb" target="_blank">openssl_1.1.1c.bb</a> | 1 +<br>
> 2 files changed, 42 insertions(+)<br>
> create mode 100644 meta/recipes-connectivity/openssl/openssl/0001-Fix-build-error-for-aarch64-big-endian.patch<br>
><br>
> diff --git a/meta/recipes-connectivity/openssl/openssl/0001-Fix-build-error-for-aarch64-big-endian.patch b/meta/recipes-connectivity/openssl/openssl/0001-Fix-build-error-for-aarch64-big-endian.patch<br>
> new file mode 100644<br>
> index 0000000..65ca02c<br>
> --- /dev/null<br>
> +++ b/meta/recipes-connectivity/openssl/openssl/0001-Fix-build-error-for-aarch64-big-endian.patch<br>
> @@ -0,0 +1,41 @@<br>
> +From 1f8c0f2feea5cdcae0bcd9dfc78198d9e2c4cf09 Mon Sep 17 00:00:00 2001<br>
> +From: Lei Maohui <<a href="mailto:leimaohui@cn.fujitsu.com" target="_blank">leimaohui@cn.fujitsu.com</a>><br>
> +Date: Thu, 13 Jun 2019 12:17:30 +0900<br>
> +Subject: [PATCH] Fix build error for aarch64 big endian.<br>
> +<br>
> +Modified rev to rev64, because rev only takes integer registers.<br>
> +<a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90827" target="_blank">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90827</a><br>
> +Otherwise, the following error will occur.<br>
> +<br>
> +Error: operand 1 must be an integer register -- `rev v31.16b,v31.16b'<br>
> +<br>
<br>
^^ goes here.<br>
<br>
The upstream status is a property of the patch you're adding, not the<br>
commit message.<br>
<br>
> +Signed-off-by: Lei Maohui <<a href="mailto:leimaohui@cn.fujitsu.com" target="_blank">leimaohui@cn.fujitsu.com</a>><br>
> +---<br>
> + crypto/sha/asm/<a href="http://keccak1600-armv8.pl" target="_blank">keccak1600-armv8.pl</a> | 4 ++--<br>
> + 1 file changed, 2 insertions(+), 2 deletions(-)<br>
> +<br>
> +diff --git a/crypto/sha/asm/<a href="http://keccak1600-armv8.pl" target="_blank">keccak1600-armv8.pl</a> b/crypto/sha/asm/<a href="http://keccak1600-armv8.pl" target="_blank">keccak1600-armv8.pl</a><br>
> +index dc72f18..6620690 100755<br>
> +--- a/crypto/sha/asm/<a href="http://keccak1600-armv8.pl" target="_blank">keccak1600-armv8.pl</a><br>
> ++++ b/crypto/sha/asm/<a href="http://keccak1600-armv8.pl" target="_blank">keccak1600-armv8.pl</a><br>
> +@@ -731,7 +731,7 @@ $code.=<<___;<br>
> + blo .Lprocess_block_ce<br>
> + ldr d31,[$inp],#8 // *inp++<br>
> + #ifdef __AARCH64EB__<br>
> +- rev v31.16b,v31.16b<br>
> ++ rev64 v31.16b,v31.16b<br>
> + #endif<br>
> + eor $A[$j/5][$j%5],$A[$j/5][$j%5],v31.16b<br>
> + beq .Lprocess_block_ce<br>
> +@@ -740,7 +740,7 @@ ___<br>
> + $code.=<<___;<br>
> + ldr d31,[$inp],#8 // *inp++<br>
> + #ifdef __AARCH64EB__<br>
> +- rev v31.16b,v31.16b<br>
> ++ rev64 v31.16b,v31.16b<br>
> + #endif<br>
> + eor $A[4][4],$A[4][4],v31.16b<br>
> +<br>
> +--<br>
> +2.7.4<br>
> +<br>
> diff --git a/meta/recipes-connectivity/openssl/<a href="http://openssl_1.1.1c.bb" target="_blank">openssl_1.1.1c.bb</a> b/meta/recipes-connectivity/openssl/<a href="http://openssl_1.1.1c.bb" target="_blank">openssl_1.1.1c.bb</a><br>
> index 669b1a1..ec85ccd 100644<br>
> --- a/meta/recipes-connectivity/openssl/<a href="http://openssl_1.1.1c.bb" target="_blank">openssl_1.1.1c.bb</a><br>
> +++ b/meta/recipes-connectivity/openssl/<a href="http://openssl_1.1.1c.bb" target="_blank">openssl_1.1.1c.bb</a><br>
> @@ -16,6 +16,7 @@ SRC_URI = "<a href="http://www.openssl.org/source/openssl-$%7BPV%7D.tar.gz" target="_blank">http://www.openssl.org/source/openssl-${PV}.tar.gz</a> \<br>
> file://0001-skip-test_symbol_presence.patch \<br>
> file://0001-buildinfo-strip-sysroot-and-debug-prefix-map-from-co.patch \<br>
> file://afalg.patch \<br>
> + file://0001-Fix-build-error-for-aarch64-big-endian.patch \<br>
> "<br>
><br>
> SRC_URI_append_class-nativesdk = " \<br>
> --<br>
> 2.7.4<br>
><br>
><br>
><br>
> --<br>
> _______________________________________________<br>
> Openembedded-core mailing list<br>
> <a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
> <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" target="_blank">
http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
-- <br>
_______________________________________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><o:p></o:p></p>
</blockquote>
</div>
</div>
</div>
</body>
</html>