<div dir="ltr">Thanks for the heads up.  I suspect the issue is the $PID process is exiting before wait runs.  I've sent r361326 as a speculative fix.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 21, 2019 at 3:37 PM <<a href="mailto:douglas.yung@sony.com" target="_blank">douglas.yung@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Matt,<br>
<br>
This change is causing the test you re-enabled to fail on our internal linux bot. The test output is not terribly informative though and I am seeing if I can figure out where it is failing, but wanted to give you a heads up.<br>
<br>
Douglas Yung<br>
<br>
-----Original Message-----<br>
From: llvm-commits <<a href="mailto:llvm-commits-bounces@lists.llvm.org" target="_blank">llvm-commits-bounces@lists.llvm.org</a>> On Behalf Of Matt Morehouse via llvm-commits<br>
Sent: Tuesday, May 21, 2019 14:16<br>
To: <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
Subject: [compiler-rt] r361313 - [libFuzzer] Reduce flakiness of merge-sigusr.test.<br>
<br>
Author: morehouse<br>
Date: Tue May 21 14:15:51 2019<br>
New Revision: 361313<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=361313&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=361313&view=rev</a><br>
Log:<br>
[libFuzzer] Reduce flakiness of merge-sigusr.test.<br>
<br>
Double the number of files to merge, and use wait instead of sleep.<br>
<br>
Modified:<br>
    compiler-rt/trunk/test/fuzzer/merge-sigusr.test<br>
<br>
Modified: compiler-rt/trunk/test/fuzzer/merge-sigusr.test<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/fuzzer/merge-sigusr.test?rev=361313&r1=361312&r2=361313&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/fuzzer/merge-sigusr.test?rev=361313&r1=361312&r2=361313&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/fuzzer/merge-sigusr.test (original)<br>
+++ compiler-rt/trunk/test/fuzzer/merge-sigusr.test Tue May 21 14:15:51 <br>
+++ 2019<br>
@@ -1,8 +1,7 @@<br>
 # Check that libFuzzer honors SIGUSR1/SIGUSR2  # FIXME: Disabled on Windows for now because of reliance on posix only features  # (eg: export, "&", pkill).<br>
-# TODO(morehouse): rethink test and reenable for linux.<br>
-UNSUPPORTED: darwin, windows, linux<br>
+UNSUPPORTED: darwin, windows<br>
 RUN: rm -rf %t<br>
 RUN: mkdir -p %t<br>
 RUN: %cpp_compiler %S/SleepOneSecondTest.cpp -o %t/LFSIGUSR @@ -15,12 +14,20 @@ RUN: echo d > %t/C2/d<br>
 RUN: echo e > %t/C2/e<br>
 RUN: echo f > %t/C2/f<br>
 RUN: echo g > %t/C2/g<br>
+RUN: echo h > %t/C2/g<br>
+RUN: echo i > %t/C2/g<br>
+RUN: echo j > %t/C2/g<br>
+RUN: echo k > %t/C2/g<br>
+RUN: echo l > %t/C2/g<br>
+RUN: echo m > %t/C2/g<br>
+RUN: echo n > %t/C2/g<br>
+RUN: echo o > %t/C2/g<br>
<br>
 RUN: %run %t/LFSIGUSR -merge=1 -merge_control_file=%t/MCF %t/C1 %t/C2 2>%t/log & export PID=$!<br>
 RUN: sleep 3<br>
 RUN: pkill -SIGUSR2 -P $PID<br>
 RUN: kill -SIGUSR2 $PID<br>
-RUN: sleep 6<br>
+RUN: wait $PID<br>
 RUN: cat %t/log | FileCheck %s --dump-input=fail<br>
 RUN: grep C2/g %t/MCF<br>
 RUN: grep STARTED %t/MCF<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>