[Mesa-dev] [PATCH 2/2] ir_to_mesa: Let check_resources halt compilation
idr at freedesktop.org
Fri Oct 28 09:54:07 PDT 2011
On 10/27/2011 10:32 PM, Kenneth Graunke wrote:
> On 10/27/2011 02:59 PM, Ian Romanick wrote:
>> From: Ian Romanick<ian.d.romanick at intel.com>
>> Previously check_resources could fail, but we'd still try to optimize
>> the shader, do device-specific code generation, etc. In some cases,
>> this could explode (especially in the device-specific code
>> generation). I haven't found that I could trigger this with the
>> current code.
> Really? Try running oglconform...or the attached shader_runner test.
Ah, right. Anything that uses vertex texturing would trigger this.
>> When too many samplers were used with the new uniform
>> handling code, I observed several crashes deep down in the driver.
> Yeah. Basically anything that fails to link crashes in
> brw_fs_precompile with current code (master).
>> Signed-off-by: Ian Romanick<ian.d.romanick at intel.com>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41609
>> Cc: Eric Anholt<eric at anholt.net>
> For patch 1, you get a
> Reviewed-by: Kenneth Graunke<kenneth at whitecape.org>
> For patch 2, you get a whopping
> Reviewed-and-tested-by: Kenneth Graunke<kenneth at whitecape.org>
> (it fixes the crash in the attached test)
> Thanks for fixing this, Ian.
More information about the mesa-dev