Oops there's a bug in there, I'll send out a new version (we only pass a
one pixel sized array).
FWIW the msaa case also has its issues (generates a warning in
store_texsubimage, I guess that's pretty harmless though at the very
least the default value is going not to be what it should but who cares
about that anyway). Don't know though how that could be fixed.
Roland
Am 28.08.2014 04:28, schrieb Ian Romanick:
> Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
>
> On 08/27/2014 06:00 PM, srol...@vmware.com wrote:
>> From: Roland Scheidegger <srol...@vmware.com>
>>
>> mesa was creating a cube map array texture with just one layer, which is
>> not legal. This caused an assertion failure when using that texture later
>> in llvmpipe (when enabling cube map arrays) since it verifies the number
>> of layers in the view is divisible by 6 (the sampling code might well crash
>> randomly otherwise) with piglit glsl-resource-not-bound CubeArray -fbo -auto.
>> ---
>> src/mesa/main/texobj.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/mesa/main/texobj.c b/src/mesa/main/texobj.c
>> index 2a82c2d..feac771 100644
>> --- a/src/mesa/main/texobj.c
>> +++ b/src/mesa/main/texobj.c
>> @@ -772,7 +772,8 @@ _mesa_get_fallback_texture(struct gl_context *ctx,
>> gl_texture_index tex)
>> {
>> if (!ctx->Shared->FallbackTex[tex]) {
>> /* create fallback texture now */
>> - const GLsizei width = 1, height = 1, depth = 1;
>> + const GLsizei width = 1, height = 1;
>> + GLsizei depth = 1;
>> GLubyte texel[4];
>> struct gl_texture_object *texObj;
>> struct gl_texture_image *texImage;
>> @@ -822,6 +823,7 @@ _mesa_get_fallback_texture(struct gl_context *ctx,
>> gl_texture_index tex)
>> case TEXTURE_CUBE_ARRAY_INDEX:
>> dims = 3;
>> target = GL_TEXTURE_CUBE_MAP_ARRAY;
>> + depth = 6;
>> break;
>> case TEXTURE_EXTERNAL_INDEX:
>> dims = 2;
>>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
