<div dir="ltr"><div dir="ltr"><div>Hi all,</div><div><br></div><div>Could somebody help me with a push if it is acceptable for everyone?</div><div><br></div><div>Thanks,</div><div>Andrii.<br></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 1:11 PM <<a href="mailto:asimiklit.work@gmail.com">asimiklit.work@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
<br>
Test that usage of upside down miptree doesn't cause assertion<br>
<br>
The miptree:<br>
<br>
level 0 = 1x1<br>
level 1 = 2x2<br>
level 2 = 4x4<br>
...<br>
level n = NxN<br>
<br>
should be acceptable for case when we don't use a min filter.<br>
<br>
v2: - Unnecessary function calls were removed<br>
    - The 'glClearColor' call was moved to 'piglit_display' function<br>
    - The program creation was moved to 'piglit_init' function<br>
    - The requirements check was moved to 'piglit_init' function<br>
               ( Erik Faye-Lund <<a href="mailto:erik.faye-lund@collabora.com" target="_blank">erik.faye-lund@collabora.com</a>> )<br>
<br>
    - Fixed a leak of texture which is created in 'setup_texture'<br>
<br>
v3: - Defined a constant for the test color instead of a hardcoded variant<br>
    - Removed unused variable 'pos' which is always was 0<br>
    - Implemented more effective number of levels calculator<br>
    - Removed a stop when one of tests fails<br>
    - Fixed two style issues<br>
               ( Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>> )<br>
<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=107987" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=107987</a><br>
Signed-off-by: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
Reviewed-by: Erik Faye-Lund <<a href="mailto:erik.faye-lund@collabora.com" target="_blank">erik.faye-lund@collabora.com</a>><br>
---<br>
 tests/opengl.py                           |   1 +<br>
 tests/texturing/CMakeLists.gl.txt         |   1 +<br>
 tests/texturing/tex-upside-down-miptree.c | 162 ++++++++++++++++++++++<br>
 3 files changed, 164 insertions(+)<br>
 create mode 100644 tests/texturing/tex-upside-down-miptree.c<br>
<br>
diff --git a/tests/opengl.py b/tests/opengl.py<br>
index f7e408cd5..f6a38e40e 100644<br>
--- a/tests/opengl.py<br>
+++ b/tests/opengl.py<br>
@@ -704,6 +704,7 @@ with profile.test_list.group_manager(<br>
     g(['getteximage-targets', '1D'])<br>
     g(['getteximage-targets', '2D'])<br>
     g(['teximage-scale-bias'])<br>
+    g(['tex-upside-down-miptree'])<br>
     add_msaa_visual_plain_tests(g, ['draw-pixels'])<br>
     add_msaa_visual_plain_tests(g, ['read-front'], run_concurrent=False)<br>
     add_msaa_visual_plain_tests(g, ['read-front', 'clear-front-first'],<br>
diff --git a/tests/texturing/CMakeLists.gl.txt b/tests/texturing/CMakeLists.gl.txt<br>
index e5d41e432..02b572c79 100644<br>
--- a/tests/texturing/CMakeLists.gl.txt<br>
+++ b/tests/texturing/CMakeLists.gl.txt<br>
@@ -98,5 +98,6 @@ piglit_add_executable (texture-al texture-al.c)<br>
 piglit_add_executable (texture-rg texture-rg.c)<br>
 piglit_add_executable (teximage-colors teximage-colors.c)<br>
 piglit_add_executable (zero-tex-coord zero-tex-coord.c)<br>
+piglit_add_executable (tex-upside-down-miptree tex-upside-down-miptree.c)<br>
<br>
 # vim: ft=cmake:<br>
diff --git a/tests/texturing/tex-upside-down-miptree.c b/tests/texturing/tex-upside-down-miptree.c<br>
new file mode 100644<br>
index 000000000..c70fc8058<br>
--- /dev/null<br>
+++ b/tests/texturing/tex-upside-down-miptree.c<br>
@@ -0,0 +1,162 @@<br>
+/*<br>
+ * Copyright (c) 2018 Andrii Simiklit<br>
+ *<br>
+ * Permission is hereby granted, free of charge, to any person obtaining a<br>
+ * copy of this software and associated documentation files (the "Software"),<br>
+ * to deal in the Software without restriction, including without limitation<br>
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,<br>
+ * and/or sell copies of the Software, and to permit persons to whom the<br>
+ * Software is furnished to do so, subject to the following conditions:<br>
+ *<br>
+ * The above copyright notice and this permission notice (including the next<br>
+ * paragraph) shall be included in all copies or substantial portions of the<br>
+ * Software.<br>
+ *<br>
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL<br>
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING<br>
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS<br>
+ * IN THE SOFTWARE.<br>
+ *<br>
+ * Authors:<br>
+ *    Andrii Simiklit <<a href="mailto:asimiklit.work@gmail.com" target="_blank">asimiklit.work@gmail.com</a>><br>
+ *<br>
+ */<br>
+<br>
+/**<br>
+ * Test what there no an assertion when we use upside down miptree and<br>
+ * GL_TEXTURE_MIN_FILTER is GL_LINEAR, base level is not 0<br>
+ * Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=107987" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=107987</a><br>
+ */<br>
+<br>
+#include "piglit-util-gl.h"<br>
+#include <math.h><br>
+<br>
+#define TW 64<br>
+#define TH 64<br>
+#define MINIMUM(X, Y)  ((X) < (Y) ? (X) : (Y))<br>
+#define LEVELS (log2(MINIMUM(TW,TH)) + 1)<br>
+<br>
+PIGLIT_GL_TEST_CONFIG_BEGIN<br>
+<br>
+       config.supports_gl_compat_version = 10;<br>
+<br>
+       config.window_visual = PIGLIT_GL_VISUAL_RGBA | PIGLIT_GL_VISUAL_DOUBLE;<br>
+       config.khr_no_error_support = PIGLIT_NO_ERRORS;<br>
+<br>
+PIGLIT_GL_TEST_CONFIG_END<br>
+<br>
+GLuint prog = 0;<br>
+GLuint texture = 0;<br>
+static const GLubyte fancy_pixel[4] = { 255, 128, 64, 32 };<br>
+static const char *fragShaderText =<br>
+       "uniform sampler2D tex;\n"<br>
+       "void main()\n"<br>
+       "{\n"<br>
+       "   gl_FragColor = texture2D(tex, gl_TexCoord[0].xy).rgba;\n"<br>
+       "}\n";<br>
+<br>
+static void<br>
+get_rect_bounds(int *x, int *y, int *w, int *h)<br>
+{<br>
+       *x = 5;<br>
+       *y = 5;<br>
+       *w = piglit_width / 3 - 10;<br>
+       *h = piglit_height - 10;<br>
+}<br>
+<br>
+<br>
+static void<br>
+draw_rect()<br>
+{<br>
+       int x, y, w, h;<br>
+       get_rect_bounds(&x, &y, &w, &h);<br>
+       piglit_draw_rect_tex(x, y, w, h, 0, 0, 1, 1);<br>
+}<br>
+<br>
+<br>
+static GLboolean<br>
+probe_pos(const GLfloat expected[4])<br>
+{<br>
+       int x, y, w, h;<br>
+       get_rect_bounds(&x, &y, &w, &h);<br>
+       return piglit_probe_rect_rgba(x, y, w, h, expected);<br>
+}<br>
+<br>
+<br>
+enum piglit_result<br>
+piglit_display(void)<br>
+{<br>
+       GLint tex;<br>
+       GLboolean pass = GL_TRUE;<br>
+       unsigned level;<br>
+       const GLfloat oneby255 = (1.0 / 255.0);<br>
+       const GLfloat expected[4] = { oneby255 * fancy_pixel[0],<br>
+                                                                                       oneby255 * fancy_pixel[1],<br>
+                                                                                       oneby255 * fancy_pixel[2],<br>
+                                                                                       oneby255 * fancy_pixel[3] };<br>
+<br>
+       glClearColor(0.5f, 0.5f, 0.5f, 0.5f);<br>
+       piglit_ortho_projection(piglit_width, piglit_height, GL_FALSE);<br>
+       glUseProgram(prog);<br>
+       tex = glGetUniformLocation(prog, "tex");<br>
+       glUniform1i(tex, 0);<br>
+<br>
+       for(level = 1; level < LEVELS; level++) {<br>
+               glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_BASE_LEVEL, level);<br>
+               glClear(GL_COLOR_BUFFER_BIT);<br>
+               /* If we the draw_rect function doesn't cause crash/assertion<br>
+               * it means everything is okay and test will be marked<br>
+               * as pass<br>
+               */<br>
+               draw_rect();<br>
+               /** Just in case we check it<br>
+                */<br>
+               pass = probe_pos(expected) && pass;<br>
+       }<br>
+<br>
+       glUseProgram(0);<br>
+       glDeleteProgram(prog);<br>
+       glDeleteTextures(1, &texture);<br>
+       piglit_present_results();<br>
+<br>
+       return pass ? PIGLIT_PASS : PIGLIT_FAIL;<br>
+}<br>
+<br>
+<br>
+static void<br>
+setup_texture(void)<br>
+{<br>
+       GLubyte img[TH][TW][4];<br>
+       int i, j, w, h;<br>
+<br>
+       for (i = 0; i < TH; i++)<br>
+               for (j = 0; j < TW; j++)<br>
+                       memcpy(&img[i][j], fancy_pixel, sizeof(fancy_pixel));<br>
+<br>
+       /* make an abnormal upside down miptree<br>
+        */<br>
+       glGenTextures(1, &texture);<br>
+       glBindTexture(GL_TEXTURE_2D, texture);<br>
+       w = TW;<br>
+       h = TH;<br>
+       for (i = 0; w > 0 && h > 0; i++) {<br>
+               glTexImage2D(GL_TEXTURE_2D, LEVELS - 1 - i, GL_RGBA, w, h, 0,<br>
+                            GL_RGBA, GL_UNSIGNED_BYTE, img);<br>
+               w /= 2;<br>
+               h /= 2;<br>
+       }<br>
+<br>
+       glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);<br>
+}<br>
+<br>
+<br>
+void<br>
+piglit_init(int argc, char **argv)<br>
+{<br>
+       piglit_require_GLSL_version(110);<br>
+       prog = piglit_build_simple_program(NULL, fragShaderText);<br>
+       setup_texture();<br>
+}<br>
-- <br>
2.17.1<br>
<br>
</blockquote></div></div>