Tobias Grosser via llvm-dev
2016-Jan-12 20:35 UTC
[llvm-dev] greendragon build noisy due to mmap_stress.cc
Hi, one of the greendragon bots is flaky due to mmap_stress.cc in compiler-rt. In the last two days this bot failed three times (and recovered) due to thhe mmap_stress.cc test in compiler-rt: http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/testReport/ http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9551/testReport/ http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9532/testReport/ Dmitry Vyukov, you seem to have committed this test case back in March. Could you possibly have a look? CCed Michael who owns this bot (afaiu). Best, Tobias ---------- Forwarded message ---------- From: ** <llvm.greendragon at gmail.com <mailto:llvm.greendragon at gmail.com>> Date: 2016-01-13 0:00 GMT+05:00 Subject: Clang Stage 1: cmake, RA, using system compiler (Check) - Build # 9557 - Failure! To: Roman Gareev <gareevroman at gmail.com <mailto:gareevroman at gmail.com>>, Simon Atanasyan <simon at atanasyan.com <mailto:simon at atanasyan.com>>, Teresa Johnson <tejohnson at google.com <mailto:tejohnson at google.com>>, Sanjay Patel <spatel at rotateright.com <mailto:spatel at rotateright.com>> Cc: Mikhail Zolotukhin <mzolotukhin at apple.com <mailto:mzolotukhin at apple.com>> __ [FAILURE] clang-stage1-cmake-RA_check [#9557] Build URL: http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/ Project: clang-stage1-cmake-RA_check Date of build: Tue, 12 Jan 2016 10:37:00 -0800 Build duration: 23 min Identified problems: * Regression test failed: This build failed because a regression test in the test suite FAILed. See the test report <http://testReport/> for details. o Indication 1 <http://lab.llvm.org:8080/green//job/clang-stage1-cmake-RA_check/9557/consoleFull#-2455948548254eaf0-7326-4999-85b0-388101f2d404> * Ninja target failed: Below is a link to the first failed ninja target. o Indication 1 <http://lab.llvm.org:8080/green//job/clang-stage1-cmake-RA_check/9557/consoleFull#-676809979a1ca8a51-895e-46c6-af87-ce24fa4cd561> Tests: Name: *AddressSanitizer-Unit* Failed: 0 test(s), Passed: 486 test(s), Total: 486 test(s) Name: *AddressSanitizer-i386-darwin* Failed: 0 test(s), Passed: 345 test(s), Total: 345 test(s) Name: *AddressSanitizer-x86_64-darwin* Failed: 0 test(s), Passed: 345 test(s), Total: 345 test(s) Name: *Clang* Failed: 0 test(s), Passed: 7496 test(s), Total: 7496 test(s) Name: *Clang Tools* Failed: 0 test(s), Passed: 156 test(s), Total: 156 test(s) Name: *Clang-Unit* Failed: 0 test(s), Passed: 1385 test(s), Total: 1385 test(s) Name: *Extra Tools Unit Tests* Failed: 0 test(s), Passed: 73 test(s), Total: 73 test(s) Name: *LLVM* Failed: 0 test(s), Passed: 14153 test(s), Total: 14153 test(s) Name: *LLVM-Unit* Failed: 0 test(s), Passed: 1428 test(s), Total: 1428 test(s) Name: *Profile* Failed: 0 test(s), Passed: 25 test(s), Total: 25 test(s) Name: *SafeStack* Failed: 0 test(s), Passed: 7 test(s), Total: 7 test(s) Name: *SanitizerCommon-Unit* Failed: 0 test(s), Passed: 225 test(s), Total: 225 test(s) Name: *SanitizerCommon-asan* Failed: 0 test(s), Passed: 68 test(s), Total: 68 test(s) Name: *ThreadSanitizer* Failed: 1 test(s), Passed: 212 test(s), Total: 213 test(s) * * Failed: ThreadSanitizer.ThreadSanitizer.mmap_stress.cc <http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/testReport/junit/ThreadSanitizer/ThreadSanitizer/mmap_stress_cc> * Name: *UBSan-ASan-i386* Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) Name: *UBSan-ASan-x86_64* Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) Name: *UBSan-Standalone-i386* Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) Name: *UBSan-Standalone-x86_64* Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) Name: *UBSan-TSan-x86_64* Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) Name: *cfi* Failed: 0 test(s), Passed: 17 test(s), Total: 17 test(s) Name: *libc++* Failed: 0 test(s), Passed: 5023 test(s), Total: 5023 test(s) Changes * Commit *257497* by *spatel:* function names start with a lower case letter ; NFC o *edit*: llvm-revision.src/cfe/trunk/lib/CodeGen/CodeGenFunction.cpp * Commit *257496* by *spatel:* function names start with a lower case letter ; NFC o *edit*: llvm-revision.src/llvm/trunk/include/llvm/IR/IRBuilder.h o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/InstCombine/InstCombineSelect.cpp o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/Utils/LoopUtils.cpp o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp o *edit*: llvm-revision.src/llvm/trunk/unittests/IR/IRBuilderTest.cpp * Commit *257495* by *romangareev:* We do not need to schedule another loop interchange pass after Polly, as Polly should perform loop interchanges itself. This also fixes a bug we see due to the "loop-interchange" pass producing incorrect IR when compiling linpack-pc.c from the LLVM test-suite with "-polly-position=before-vectorizer". Reviewed-by: Tobias Grosser ____ o *edit*: llvm-revision.src/polly/trunk/lib/CodeGen/CodegenCleanup.cpp * Commit *257494* by *tejohnson:* Fix bot failure from r257493: remove extraneous temp file read This was left from an earlier version of the test. o *edit*: llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/funcimport_alias.ll * Commit *257493* by *tejohnson:* [ThinLTO] Handle an external call from an import to an alias in dest The findExternalCalls routine ignores calls to functions already defined in the dest module. This was not handling the case where the definition in the current module is actually an alias to a function call. o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp o *add*: llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/Inputs/funcimport_alias.ll o *add*: llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/funcimport_alias.ll * Commit *257492* by *atanasyan:* [ELF][MIPS] Do not create dynamic relocations against _gp_disp symbol MIPS _gp_disp designates offset between start of function and gp pointer into GOT therefore any relocations against it do not require dynamic relocation. o *edit*: llvm-revision.src/lld/trunk/ELF/Writer.cpp o *edit*: llvm-revision.src/lld/trunk/test/ELF/mips-gp-disp.s * Commit *257491* by *spatel:* [LibCallSimplifier] use instruction-level fast-math-flags to transform pow(exp(x)) calls See also: http://reviews.llvm.org/rL255555 http://reviews.llvm.org/rL256871 http://reviews.llvm.org/rL256964 http://reviews.llvm.org/rL257400 http://reviews.llvm.org/rL257404 http://reviews.llvm.org/rL257414 o *edit*: llvm-revision.src/llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp o *edit*: llvm-revision.src/llvm/trunk/test/Transforms/InstCombine/pow-exp.ll
Kostya Serebryany via llvm-dev
2016-Jan-12 21:26 UTC
[llvm-dev] greendragon build noisy due to mmap_stress.cc
Hi Tobias, What machine is that? We have seen this and similar tests be flaky on older Linux kernels due to kernel bug(s). May I ask you to run the same test (just built with clang) on the same machine for ~100000 times and see if it ever crashes? clang++ ~/llvm/projects/compiler-rt/test/tsan/mmap_stress.cc -lpthread while true; do for((i=0;i<10;i++)); do ./a.out || echo ===============FAILED================ & done ; wait; done On Tue, Jan 12, 2016 at 12:35 PM, Tobias Grosser via llvm-dev < llvm-dev at lists.llvm.org> wrote:> Hi, > > one of the greendragon bots is flaky due to mmap_stress.cc in compiler-rt. > > In the last two days this bot failed three times (and recovered) due to > thhe mmap_stress.cc test in compiler-rt: > > > http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/testReport/ > > http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9551/testReport/ > > http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9532/testReport/ > > > Dmitry Vyukov, you seem to have committed this test case back in March. > Could you possibly have a look? > > CCed Michael who owns this bot (afaiu). > > Best, > Tobias > > ---------- Forwarded message ---------- > From: ** <llvm.greendragon at gmail.com <mailto:llvm.greendragon at gmail.com>> > Date: 2016-01-13 0:00 GMT+05:00 > Subject: Clang Stage 1: cmake, RA, using system compiler (Check) - Build > # 9557 - Failure! > To: Roman Gareev <gareevroman at gmail.com <mailto:gareevroman at gmail.com>>, > Simon Atanasyan <simon at atanasyan.com <mailto:simon at atanasyan.com>>, > Teresa Johnson <tejohnson at google.com <mailto:tejohnson at google.com>>, > Sanjay Patel <spatel at rotateright.com <mailto:spatel at rotateright.com>> > Cc: Mikhail Zolotukhin <mzolotukhin at apple.com > <mailto:mzolotukhin at apple.com>> > > > __ > > > [FAILURE] clang-stage1-cmake-RA_check [#9557] > > Build URL: > http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/ > Project: clang-stage1-cmake-RA_check > Date of build: Tue, 12 Jan 2016 10:37:00 -0800 > Build duration: 23 min > > > Identified problems: > > * Regression test failed: This build failed because a regression test > in the test suite FAILed. See the test report <http://testReport/> > for details. > o Indication 1 > > < > http://lab.llvm.org:8080/green//job/clang-stage1-cmake-RA_check/9557/consoleFull#-2455948548254eaf0-7326-4999-85b0-388101f2d404 > > > * Ninja target failed: Below is a link to the first failed ninja target. > o Indication 1 > > < > http://lab.llvm.org:8080/green//job/clang-stage1-cmake-RA_check/9557/consoleFull#-676809979a1ca8a51-895e-46c6-af87-ce24fa4cd561 > > > > > Tests: > > Name: *AddressSanitizer-Unit* > Failed: 0 test(s), Passed: 486 test(s), Total: 486 test(s) > Name: *AddressSanitizer-i386-darwin* > Failed: 0 test(s), Passed: 345 test(s), Total: 345 test(s) > Name: *AddressSanitizer-x86_64-darwin* > Failed: 0 test(s), Passed: 345 test(s), Total: 345 test(s) > Name: *Clang* > Failed: 0 test(s), Passed: 7496 test(s), Total: 7496 test(s) > Name: *Clang Tools* > Failed: 0 test(s), Passed: 156 test(s), Total: 156 test(s) > Name: *Clang-Unit* > Failed: 0 test(s), Passed: 1385 test(s), Total: 1385 test(s) > Name: *Extra Tools Unit Tests* > Failed: 0 test(s), Passed: 73 test(s), Total: 73 test(s) > Name: *LLVM* > Failed: 0 test(s), Passed: 14153 test(s), Total: 14153 test(s) > Name: *LLVM-Unit* > Failed: 0 test(s), Passed: 1428 test(s), Total: 1428 test(s) > Name: *Profile* > Failed: 0 test(s), Passed: 25 test(s), Total: 25 test(s) > Name: *SafeStack* > Failed: 0 test(s), Passed: 7 test(s), Total: 7 test(s) > Name: *SanitizerCommon-Unit* > Failed: 0 test(s), Passed: 225 test(s), Total: 225 test(s) > Name: *SanitizerCommon-asan* > Failed: 0 test(s), Passed: 68 test(s), Total: 68 test(s) > Name: *ThreadSanitizer* > Failed: 1 test(s), Passed: 212 test(s), Total: 213 test(s) > * > > * Failed: ThreadSanitizer.ThreadSanitizer.mmap_stress.cc > > < > http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/9557/testReport/junit/ThreadSanitizer/ThreadSanitizer/mmap_stress_cc > > > > > * > Name: *UBSan-ASan-i386* > Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) > Name: *UBSan-ASan-x86_64* > Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) > Name: *UBSan-Standalone-i386* > Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) > Name: *UBSan-Standalone-x86_64* > Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) > Name: *UBSan-TSan-x86_64* > Failed: 0 test(s), Passed: 34 test(s), Total: 34 test(s) > Name: *cfi* > Failed: 0 test(s), Passed: 17 test(s), Total: 17 test(s) > Name: *libc++* > Failed: 0 test(s), Passed: 5023 test(s), Total: 5023 test(s) > > > Changes > > * Commit *257497* by *spatel:* > > function names start with a lower case letter ; NFC > > o *edit*: llvm-revision.src/cfe/trunk/lib/CodeGen/CodeGenFunction.cpp > > * Commit *257496* by *spatel:* > > function names start with a lower case letter ; NFC > > o *edit*: llvm-revision.src/llvm/trunk/include/llvm/IR/IRBuilder.h > o *edit*: > > > llvm-revision.src/llvm/trunk/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp > o *edit*: > > > llvm-revision.src/llvm/trunk/lib/Transforms/InstCombine/InstCombineSelect.cpp > o *edit*: > llvm-revision.src/llvm/trunk/lib/Transforms/Utils/LoopUtils.cpp > o *edit*: > > llvm-revision.src/llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp > o *edit*: > > llvm-revision.src/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp > o *edit*: llvm-revision.src/llvm/trunk/unittests/IR/IRBuilderTest.cpp > > * Commit *257495* by *romangareev:* > > We do not need to schedule another loop interchange pass after > Polly, as Polly should perform loop interchanges itself. This also > fixes a bug we see due to the "loop-interchange" pass producing > incorrect IR when compiling linpack-pc.c from the LLVM test-suite > with "-polly-position=before-vectorizer". Reviewed-by: Tobias > Grosser ____ > > o *edit*: > llvm-revision.src/polly/trunk/lib/CodeGen/CodegenCleanup.cpp > > * Commit *257494* by *tejohnson:* > > Fix bot failure from r257493: remove extraneous temp file read This > was left from an earlier version of the test. > > o *edit*: > > > llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/funcimport_alias.ll > > * Commit *257493* by *tejohnson:* > > [ThinLTO] Handle an external call from an import to an alias in dest > The findExternalCalls routine ignores calls to functions already > defined in the dest module. This was not handling the case where the > definition in the current module is actually an alias to a function > call. > > o *edit*: > llvm-revision.src/llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp > o *add*: > > > llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/Inputs/funcimport_alias.ll > o *add*: > > > llvm-revision.src/llvm/trunk/test/Transforms/FunctionImport/funcimport_alias.ll > > * Commit *257492* by *atanasyan:* > > [ELF][MIPS] Do not create dynamic relocations against _gp_disp > symbol MIPS _gp_disp designates offset between start of function and > gp pointer into GOT therefore any relocations against it do not > require dynamic relocation. > > o *edit*: llvm-revision.src/lld/trunk/ELF/Writer.cpp > o *edit*: llvm-revision.src/lld/trunk/test/ELF/mips-gp-disp.s > > * Commit *257491* by *spatel:* > > [LibCallSimplifier] use instruction-level fast-math-flags to > transform pow(exp(x)) calls See also: > http://reviews.llvm.org/rL255555 http://reviews.llvm.org/rL256871 > http://reviews.llvm.org/rL256964 http://reviews.llvm.org/rL257400 > http://reviews.llvm.org/rL257404 http://reviews.llvm.org/rL257414 > > o *edit*: > > llvm-revision.src/llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp > o *edit*: > llvm-revision.src/llvm/trunk/test/Transforms/InstCombine/pow-exp.ll > > > > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160112/9c5e2a28/attachment.html>
Tobias Grosser via llvm-dev
2016-Jan-12 21:28 UTC
[llvm-dev] greendragon build noisy due to mmap_stress.cc
On 01/12/2016 10:26 PM, Kostya Serebryany wrote:> Hi Tobias, > > What machine is that? > We have seen this and similar tests be flaky on older Linux kernels due > to kernel bug(s). > May I ask you to run the same test (just built with clang) on the same > machine for ~100000 times and see if it ever crashes? > > clang++ ~/llvm/projects/compiler-rt/test/tsan/mmap_stress.cc -lpthread > while true; do for((i=0;i<10;i++)); do ./a.out || echo > ===============FAILED================ & done ; wait; doneAdd Michael again (who might have access to this Machine). Tobias