<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 10, 2018 at 9:54 PM, Mark Janes <span dir="ltr"><<a href="mailto:mark.a.janes@intel.com" target="_blank">mark.a.janes@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="m_-1019844029714621975HOEnZb"><div class="m_-1019844029714621975h5">Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>> writes:<br>
<br>
> Quoting Marek Olšák (2018-04-10 14:22:10)<br>
>> On Tue, Apr 10, 2018 at 2:15 PM, Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>> wrote:<br>
>><br>
>>     Quoting Eric Anholt (2018-04-09 17:10:35)<br>
>>     > Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> writes:<br>
>>     ><br>
>>     > > Is this use case affected?<br>
>>     > ><br>
>>     > > piglit run --deqp-mustpass-list --process-isolation 0 -p gbm -c quick<br>
>>     > > cts_gl45 deqp_gles2 deqp_gles3 deqp_gles31<br>
>>     > ><br>
>>     > > Yes, that is just 1 command to run all those test suites at the same<br>
>>     time.<br>
>>     > ><br>
>>     > > I use my personal "deqp" piglit branch that also disables process<br>
>>     isolation<br>
>>     > > for glcts and deqp, and parses the deqp mustpass lists which are in txt<br>
>>     > > files.<br>
>>     ><br>
>>     > Parsing the mustpass lists sounds really useful.  Trying to construct an<br>
>>     > appropriate command line otherwise has been quite a challenge.<br>
>><br>
>>     That option is already in core piglit, you just need to configure your<br>
>>     piglit.conf appropriately. I guess we really should have that in the<br>
>>     piglit.conf.example file...<br>
>><br>
>><br>
>> I doubt it. Why would I have these then:<br>
>><br>
>> <a href="https://cgit.freedesktop.org/~mareko/piglit/commit/?h=deqp&id=" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~<wbr>mareko/piglit/commit/?h=deqp&i<wbr>d=</a><br>
>> 0b11344e0c18b9bf07ad12381b94f3<wbr>08f362eb88<br>
>> <a href="https://cgit.freedesktop.org/~mareko/piglit/commit/?h=deqp&id=" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~<wbr>mareko/piglit/commit/?h=deqp&i<wbr>d=</a><br>
>> b086d8f82d41338055ab48bdda78c4<wbr>a0c1ee02d0<br>
>> <a href="https://cgit.freedesktop.org/~mareko/piglit/commit/?h=deqp&id=" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~<wbr>mareko/piglit/commit/?h=deqp&i<wbr>d=</a><br>
>> 90beefa825cda792eaa72bff2cefac<wbr>463af6d08a<br>
>><br>
>> Marek<br>
><br>
> Because in late 2016 (there is some internal stuff that happened so we stopped<br>
> using piglit to run deqp I'm not entirely happy about) they changed the mustpass<br>
> list from xml to txt and no one ever updated it in master.<br>
<br>
</div></div>I'm not sure what you are referring to here.  We stopped using piglit<br>
for dEQP because the run-time was 10X faster with a custom runner.  We<br>
don't have process isolation anymore, but we recover from<br>
crashes/assertions without paying the penalty of iterating the dEQP test<br>
list for each test run.<br>
<br>
Google dEQP authors recommended this path.<br></blockquote><div><br></div></div>My deqp piglit branch might be just as fast or nearly as fast as your custom runner if process isolation is explicitly disabled.<br><br></div><div class="gmail_extra">Marek<br></div></div>