<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 10/28/2017 01:40 AM, Khem Raj wrote:<br>
    </div>
    <blockquote
cite="mid:CAMKF1sqQ6E6vKD8rtpF4MXR=6ceOWKnmU9h5gLMWQFc87xSjZw@mail.gmail.com"
      type="cite">
      <pre wrap="">On Fri, Oct 27, 2017 at 2:43 AM, Chen Qi <a class="moz-txt-link-rfc2396E" href="mailto:Qi.Chen@windriver.com">&lt;Qi.Chen@windriver.com&gt;</a> wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">We've observed strange behaviour of `systemctl status &lt;xxx&gt; on qemumips64.
The output of the command is like `systemctl show &lt;xxx&gt;', which is incorrect.

This patch is from gcc bugzilla's attachment.
<a class="moz-txt-link-freetext" href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81803">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81803</a>

The patch hasn't been merged into gcc. But it does solve the above problem.

</pre>
      </blockquote>
      <pre wrap="">
This looks ok. Can you help by providing your testing results to gcc bugzilla
so the patch can then be submitted to gcc ml.
</pre>
    </blockquote>
    <br>
    Some updates in this issue.<br>
    <br>
    1) I've provided information in gcc bugzilla. The author says that
    he's going to submit the patch to gcc mailing list.<br>
    <br>
    2) I've update remote branch to fix a typo ('mips4' --&gt;
    'mips64').
    <pre wrap="">  git://git.pokylinux.org/poky-contrib ChenQi/bug12266
  <a class="moz-txt-link-freetext" href="http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/bug12266">http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/bug12266</a></pre>
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    <title></title>
    <meta name="generator" content="LibreOffice 4.2.8.2 (Linux)">
    <style type="text/css">
        <!--
                @page { margin: 0.79in }
                p { margin-bottom: 0.1in; direction: ltr; color: #000000; line-height: 120%; widows: 2; orphans: 2 }
                p.western { font-family: "Times New Roman", serif; font-size: 12pt; so-language: en-US }
                p.cjk { font-family: "Times New Roman", serif; font-size: 12pt; so-language: zh-CN }
                p.ctl { font-family: "Times New Roman", serif; font-size: 12pt; so-language: ar-SA }
                a:link { color: #000000 }
        -->
        </style><br>
    3) 'core-image-sato + testimage' has been tested on other qemu
    machines including qemux86-64, qemux86, qemumips, qemuppc,
    qemuarm64.<br>
    <br>
    Best Regards,<br>
    Chen Qi<br>
    <br>
    <blockquote
cite="mid:CAMKF1sqQ6E6vKD8rtpF4MXR=6ceOWKnmU9h5gLMWQFc87xSjZw@mail.gmail.com"
      type="cite">
      <pre wrap="">
</pre>
      <blockquote type="cite">
        <pre wrap="">Signed-off-by: Chen Qi <a class="moz-txt-link-rfc2396E" href="mailto:Qi.Chen@windriver.com">&lt;Qi.Chen@windriver.com&gt;</a>
---
 meta/recipes-devtools/gcc/gcc-7.2.inc              |  1 +
 .../fix-miscompilation-for-O1-on-mips64el.patch    | 35 ++++++++++++++++++++++
 2 files changed, 36 insertions(+)
 create mode 100644 meta/recipes-devtools/gcc/gcc-7.2/fix-miscompilation-for-O1-on-mips64el.patch

diff --git a/meta/recipes-devtools/gcc/gcc-7.2.inc b/meta/recipes-devtools/gcc/gcc-7.2.inc
index 5883bc6..04edad7 100644
--- a/meta/recipes-devtools/gcc/gcc-7.2.inc
+++ b/meta/recipes-devtools/gcc/gcc-7.2.inc
@@ -75,6 +75,7 @@ SRC_URI = "\
            <a class="moz-txt-link-freetext" href="file://0048-gcc-Enable-static-PIE.patch">file://0048-gcc-Enable-static-PIE.patch</a> \
            <a class="moz-txt-link-freetext" href="file://fix-segmentation-fault-precompiled-hdr.patch">file://fix-segmentation-fault-precompiled-hdr.patch</a> \
            <a class="moz-txt-link-freetext" href="file://0050-RISC-V-Handle-non-legitimate-address-in-riscv_legiti.patch">file://0050-RISC-V-Handle-non-legitimate-address-in-riscv_legiti.patch</a> \
+           <a class="moz-txt-link-freetext" href="file://fix-miscompilation-for-O1-on-mips64el.patch">file://fix-miscompilation-for-O1-on-mips64el.patch</a> \
            ${BACKPORTS} \
 "
 BACKPORTS = "\
diff --git a/meta/recipes-devtools/gcc/gcc-7.2/fix-miscompilation-for-O1-on-mips64el.patch b/meta/recipes-devtools/gcc/gcc-7.2/fix-miscompilation-for-O1-on-mips64el.patch
new file mode 100644
index 0000000..28020b0
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-7.2/fix-miscompilation-for-O1-on-mips64el.patch
@@ -0,0 +1,35 @@
+This patch comes from <a class="moz-txt-link-freetext" href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81803">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81803</a>
+
+Author: mpf &lt;mpf@138bc75d-0d04-0410-961f-82ee72b054a4&gt;
+
+This patch hasn't been merged into gcc. But it does solve the problem of
+strange behaviour of `systemd status &lt;xxx&gt;' on qemumips64.
+
+Upstream-Status: Pending [Taken from gcc bugzilla's attachment]
+
+Signed-off-by: Chen Qi <a class="moz-txt-link-rfc2396E" href="mailto:Qi.Chen@windriver.com">&lt;Qi.Chen@windriver.com&gt;</a>
+---
+ gcc/lra-constraints.c | 7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c
+index c8bc9b9a66f..6d319662b09 100644
+--- a/gcc/lra-constraints.c
++++ b/gcc/lra-constraints.c
+@@ -4235,7 +4235,12 @@ curr_insn_transform (bool check_only_p)
+                         &amp;&amp; (goal_alt[i] == NO_REGS
+                             || (simplify_subreg_regno
+                                 (ira_class_hard_regs[goal_alt[i]][0],
+-                                 GET_MODE (reg), byte, mode) &gt;= 0)))))
++                                 GET_MODE (reg), byte, mode) &gt;= 0))))
++                    || (type != OP_IN
++                        &amp;&amp; GET_MODE_PRECISION (mode)
++                        &lt; GET_MODE_PRECISION (GET_MODE (reg))
++                        &amp;&amp; GET_MODE_SIZE (GET_MODE (reg)) &lt;= UNITS_PER_WORD
++                        &amp;&amp; WORD_REGISTER_OPERATIONS))
+               {
+                 /* An OP_INOUT is required when reloading a subreg of a
+                    mode wider than a word to ensure that data beyond the
+--
+2.13.0
+
--
1.9.1

--
_______________________________________________
Openembedded-core mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a>
<a class="moz-txt-link-freetext" href="http://lists.openembedded.org/mailman/listinfo/openembedded-core">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a>
</pre>
      </blockquote>
      <pre wrap="">
</pre>
    </blockquote>
    <p><br>
    </p>
  </body>
</html>