<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 7, 2017 at 6:41 AM, Bill Seurer <span dir="ltr"><<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Actually there was a range of revision a bit further on where it worked, too.<br>
<br>
r279571 (and previous) - works<br>
r279572 - fails<br>
. . . fails<br>
r279642 - fails<br>
r279643 - works<br>
. . . works<br>
r279663 - works<br>
r279664 - fails<br>
(and fails from here on)<br>
<br>
IIRC this is the same range or revisions that affected some of of the other multithread test cases that are flaky on powerpcle and which are currently disabled.<br>
<br>
I wonder if some of these multithread tests should be run multiple times when the bots run them?</blockquote><div><br></div><div>No. We simply need to figure out what's broken and fix that. </div><div>This will be one of the Aleksey's tasks for the nearest month or two. </div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-"><br>
<br>
On 03/06/2017 06:42 PM, Kostya Serebryany wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
Good to know, thanks!<br>
Aleksey is going to start digging around this code very soon.<br>
<br>
On Mon, Mar 6, 2017 at 9:10 AM, Bill Seurer <<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.com</a><br></span><span class="gmail-">
<mailto:<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.<wbr>com</a>>> wrote:<br>
<br>
    I ran a script that dug back through LOTS of revisions and traced<br>
    the start of this test cases failing to r279572.<br>
<br>
    ------------------------------<wbr>------------------------------<wbr>------------<br>
    r279572 | kcc | 2016-08-23 16:19:47 -0500 (Tue, 23 Aug 2016) | 1 line<br>
<br>
    [sanitizer] change the 64-bit allocator to use a single array for<br>
    free-d chunks instead of a lock-free linked list of tranfer batches.<br>
    This change simplifies the code, makes th<br>
    e allocator more 'hardened', and will allow simpler code to release<br>
    RAM to OS. This may also slowdown malloc stress tests due to lock<br>
    contension, but I did not observe noticeab<br>
    le slowdown on various real multi-threaded benchmarks.<br>
<br>
<br>
    BTW, I had to set the repeat count of<br>
    projects/compiler-rt/lib/asan/<wbr>tests/default/Asan-powerpc64le<wbr>-with-calls-Test<br>
    up to 500 in order to get it to consistently fail.<br>
<br>
<br>
    On 03/01/2017 06:24 PM, Kostya Serebryany wrote:<br>
<br>
        + more folks<br>
<br>
        On Wed, Mar 1, 2017 at 4:23 PM, Matthias Braun <<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a><br>
        <mailto:<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>><br></span><span class="gmail-">
        <mailto:<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a> <mailto:<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>>>> wrote:<br>
<br>
            We have another flaky sanitizer test on the ppc64 bot. Just<br>
        look at<br>
            the recent history and you see<br>
            AddressSanitizer-Unit::Address<wbr>Sanitizer.ManyThreadsTest<br>
        failing a<br>
            lot on unrelated commits:<br>
<br>
<br>
        <a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage?numbuilds=50" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/build<wbr>ers/clang-ppc64le-linux-multis<wbr>tage?numbuilds=50</a><br>
        <<a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage?numbuilds=50" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/buil<wbr>ders/clang-ppc64le-linux-multi<wbr>stage?numbuilds=50</a>><br>
<br>
        <<a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage?numbuilds=50" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/buil<wbr>ders/clang-ppc64le-linux-multi<wbr>stage?numbuilds=50</a><br>
        <<a href="http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage?numbuilds=50" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/buil<wbr>ders/clang-ppc64le-linux-multi<wbr>stage?numbuilds=50</a>>><br>
<br>
            Please fix or disable this test.<br>
<br>
            - Matthias<br>
<br>
<br>
<br>
<br>
    --<br>
<br>
    -Bill Seurer<br>
<br>
<br>
</span></blockquote><span class="gmail-HOEnZb"><font color="#888888">
<br>
<br>
-- <br>
<br>
-Bill Seurer<br>
<br>
</font></span></blockquote></div><br></div></div>