Re: [Mesa-dev] [PATCH] getteximage: avoid to lookup textures with id 0

2017-03-06 Thread Samuel Pitoiset
Fixed with https://cgit.freedesktop.org/mesa/mesa/commit/?id=4317cd96d3022ad0ef7e1d5ffa82f5c2693bb44e. Sorry. On 03/06/2017 09:26 PM, Mark Janes wrote: Does this work on AMD hardware? It regresses every Intel GPU. https://bugs.freedesktop.org/show_bug.cgi?id=100088 Samuel Pitoiset writes:

Re: [Mesa-dev] [PATCH] getteximage: avoid to lookup textures with id 0

2017-03-06 Thread Mark Janes
Does this work on AMD hardware? It regresses every Intel GPU. https://bugs.freedesktop.org/show_bug.cgi?id=100088 Samuel Pitoiset writes: > This fixes the following assertion when the key is 0. > > main/hash.c:181: _mesa_HashLookup_unlocked: Assertion `key' failed. > > Fixes: 633c959fae ("gett

Re: [Mesa-dev] [PATCH] getteximage: avoid to lookup textures with id 0

2017-03-06 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Wed, Mar 1, 2017 at 10:54 PM, Samuel Pitoiset wrote: > This fixes the following assertion when the key is 0. > > main/hash.c:181: _mesa_HashLookup_unlocked: Assertion `key' failed. > > Fixes: 633c959fae ("getteximage: Return correct error value when texure > o

[Mesa-dev] [PATCH] getteximage: avoid to lookup textures with id 0

2017-03-01 Thread Samuel Pitoiset
This fixes the following assertion when the key is 0. main/hash.c:181: _mesa_HashLookup_unlocked: Assertion `key' failed. Fixes: 633c959fae ("getteximage: Return correct error value when texure object is not found") Signed-off-by: Samuel Pitoiset --- src/mesa/main/texgetimage.c | 10 +++---