"The value of j is still equal to q for mip level alignment and QPitch calculation. For cube surfaces, an additional two rows of padding are required at the bottom of the surface... This is due to the potential rotation of cache line orientation from memory to cache." "
Cube map array applies as a cube surface. It's not well understood why this is needed, but it seems like it is. No changes in piglit (the g45 results are invalid). http://otc-gfxtest-01.jf.intel.com/view/dev/job/bwidawsk/118/ Signed-off-by: Ben Widawsky <b...@bwidawsk.net> --- src/mesa/drivers/dri/i965/brw_tex_layout.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_tex_layout.c b/src/mesa/drivers/dri/i965/brw_tex_layout.c index 72b02a2..ea7740a 100644 --- a/src/mesa/drivers/dri/i965/brw_tex_layout.c +++ b/src/mesa/drivers/dri/i965/brw_tex_layout.c @@ -344,7 +344,8 @@ align_cube(struct intel_mipmap_tree *mt) * memory. As a result, the docs say in Surface Padding Requirements: * Sampling Engine Surfaces that two extra rows of padding are required. */ - if (mt->target == GL_TEXTURE_CUBE_MAP) + if (mt->target == GL_TEXTURE_CUBE_MAP || + mt->target == GL_TEXTURE_CUBE_MAP_ARRAY) mt->total_height += 2; } -- 2.4.1 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev