<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 18, 2016 at 11:29 AM, Dan Liew <span dir="ltr"><<a href="mailto:dan@su-root.co.uk" target="_blank">dan@su-root.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="gmail-">>> Perhaps the 1 second time is<br>
>> too short for some buildbots if they are under load? We could extend<br>
>> the time we wait (to something > 1 second) but we'd have to probably<br>
>> increase `-max_total_time=4` to something larger. This wouldn't be a<br>
>> real fix though as the test would still be racey.<br>
><br>
><br>
> I think it can be rewritten to be non-racey.<br>
> Just run it with -max_total_time=2 -jobs=2 and let it and all children exit<br>
> -- then check that the files exist and are good.<br>
<br>
</span>I agree that doing that would be non-racey however that would be<br>
testing a slightly different property (that all jobs eventually run)<br>
rather than what the test is currently trying to check (that LibFuzzer<br>
can spawn multiple copies of itself running in parallel). The change<br>
you proposing to the test would fail to catch the problem that I fixed<br>
on macOS (LibFuzzer would not run multiple jobs in parallel).<br></blockquote><div><br></div><div>Ok... </div><div>You can probably modify your test to do grep "Running 2 workers" instead of sleeping... </div><div>This sucks. :( </div><div><br></div><div><br></div></div></div></div>