<html>
    <head>
      <base href="https://bugs.llvm.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:chh@google.com" title="Chih-Hung Hsieh <chh@google.com>"> <span class="fn">Chih-Hung Hsieh</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED WONTFIX - clang -O2 BPF netd.o failed to boot on some Android devices"
   href="https://bugs.llvm.org/show_bug.cgi?id=52390">bug 52390</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Resolution</td>
           <td>---
           </td>
           <td>WONTFIX
           </td>
         </tr>

         <tr>
           <td style="text-align:right;">Status</td>
           <td>NEW
           </td>
           <td>RESOLVED
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED WONTFIX - clang -O2 BPF netd.o failed to boot on some Android devices"
   href="https://bugs.llvm.org/show_bug.cgi?id=52390#c13">Comment # 13</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED WONTFIX - clang -O2 BPF netd.o failed to boot on some Android devices"
   href="https://bugs.llvm.org/show_bug.cgi?id=52390">bug 52390</a>
              from <span class="vcard"><a class="email" href="mailto:chh@google.com" title="Chih-Hung Hsieh <chh@google.com>"> <span class="fn">Chih-Hung Hsieh</span></a>
</span></b>
        <pre>Yonghong,

Thanks for looking into the llvm code.

The root failure was found to be in older Android BPF verifier,
which does not track value ranges of variables on the stack.
New llvm output code now returns one variable in netd.c
directly from stack. The BPF verifier cannot be certain
that the return value is always 0 or 1, and aborted the
normal boot process.

Since we need to load new netd.o to existing Adroid devices,
so the solution now is to modify netd.c and make sure
that the return value is always 0 or 1.
There is no need to change clang/llvm for now.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>