<p dir="ltr">It appears that our Piglit tests that hit glGenerateMipmap always create the cube map texture with Texture Storage, so Num Layers always gets set to 6.  I will make a simple test this afternoon for non texture storage cube maps.</p>
<div class="gmail_quote">On Mar 12, 2015 12:33 PM, "Laura Ekstrand" <<a href="mailto:laura@jlekstrand.net">laura@jlekstrand.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This reverts commit 1ee000a0b6737d6c140d4f07b6044908b8ebfdc7.<br>
Failures with the GLES3 conformance suite and Synmark2 OGLHdrBloom revealed<br>
that this commit might be in error.  A look at the offended test in GLES3<br>
conformance suite, NPOT gen mipmap, suggests that NumLayers may not actually<br>
always be 6 for a cube complete texture with target = GL_TEXTURE_CUBE_MAP.<br>
<br>
Extensive testing with Piglit prior to patch review and upstreaming did not<br>
reveal this problem because, in the few Piglit tests that test for cube<br>
completeness, NumLayers = 6.  So it appears that perhaps existing test<br>
coverage in Piglit is inadequate.<br>
---<br>
 src/mesa/main/texobj.c | 4 ----<br>
 1 file changed, 4 deletions(-)<br>
<br>
diff --git a/src/mesa/main/texobj.c b/src/mesa/main/texobj.c<br>
index a99b108..e018ab9 100644<br>
--- a/src/mesa/main/texobj.c<br>
+++ b/src/mesa/main/texobj.c<br>
@@ -879,10 +879,6 @@ _mesa_cube_level_complete(const struct gl_texture_object *texObj,<br>
    if (texObj->Target != GL_TEXTURE_CUBE_MAP)<br>
       return GL_FALSE;<br>
<br>
-   /* Make sure we have enough image planes for a cube map. */<br>
-   if (texObj->NumLayers < 6)<br>
-      return GL_FALSE;<br>
-<br>
    if ((level < 0) || (level >= MAX_TEXTURE_LEVELS))<br>
       return GL_FALSE;<br>
<br>
--<br>
2.1.0<br>
<br>
</blockquote></div>