<div dir="ltr">Thanks for looking into the problem.<div><br></div><div style>The new version I sent in produced the same number of passes as Piglit with the revert on Sandy Bridge for all.tests without glean.</div><div style>
<br></div><div style>Stuart</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 6, 2013 at 9:18 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.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="im">On 02/06/2013 06:11 PM, Stuart Abercrombie wrote:<br>
> Sorry about the breakage.  I need to resign myself to a full test run every<br>
> time.<br>
><br>
> Is part of this message missing?<br>
><br>
<br>
</div>Strange, it seems that git-send-email choked after patch 0/2.<br>
<br>
Since you already sent a remake of the reverted patch, my little 2-patch<br>
series is moot. There's no reason now for me to send the missing emails.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> On Wed, Feb 6, 2013 at 5:37 PM, Chad Versace<br>
> <<a href="mailto:chad.versace@linux.intel.com">chad.versace@linux.intel.com</a>>wrote:<br>
><br>
>> This 2-patch series is based on Stuart's patch, titled<br>
>>   shader_runner: Require [require] and GLSL version requirement<br>
>><br>
>> I committed Stuart's patch today, and then Eric rightfully reverted it.<br>
>> This<br>
>> series shouldn't be buggy like the original patch.<br>
>><br>
>> No regressions on a full run on Sandybridge and mesa master.<br>
>><br>
>> Chad Versace (2):<br>
>>   shader_runner: Require [require] section<br>
>>   shader_runner: Require either a GLSL version or GL_ARB_fp extension<br>
>><br>
>>  tests/shaders/shader_runner.c | 24 +++++++++++++++++++++++-<br>
>>  1 file changed, 23 insertions(+), 1 deletion(-)<br>
>><br>
>> --<br>
>> 1.8.1.1<br>
>><br>
>><br>
><br>
<br>
</div></div></blockquote></div><br></div>