<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - No test for Image Load/Store on texture buffer sized greater than MAX_TEXTURE_BUFFER_SIZE_ARB" href="https://bugs.freedesktop.org/show_bug.cgi?id=106481#c2">Comment # 2</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - No test for Image Load/Store on texture buffer sized greater than MAX_TEXTURE_BUFFER_SIZE_ARB" href="https://bugs.freedesktop.org/show_bug.cgi?id=106481">bug 106481</a> from <a class="email" href="mailto:nanleychery@gmail.com" title="Nanley Chery <nanleychery@gmail.com>"> Nanley Chery</a> <pre>(In reply to Illia from <a href="show_bug.cgi?id=106481#c1">comment #1</a>) > Created <a href="attachment.cgi?id=139707" name="attach_139707" title="piglit test workaround">attachment 139707</a> <a href="attachment.cgi?id=139707&action=edit" title="piglit test workaround">[details]</a> > piglit test workaround > > Hello, i try to edit piglit test to cover that situation, and your mesa > patch doesn't fix the issue. Hi, That's expected. The bug fix clamps the buffer size in the surface state. It doesn't modify glBufferData's behavior. > My output is > > MAX_TEXTURE_BUFFER_SIZE: 134217728 > Error code: 0 > MAX_TEXTURE_BUFFER_SIZE: 134218728 > Probe color at (0,0) > Expected: 0 255 0 0 > Observed: 0 0 0 0 > Probe color at (0,0) > Expected: 0 255 0 0 > Observed: 0 0 0 0 > Probe color at (0,0) > Expected: 0 255 0 0 > Observed: 0 0 0 0 > Probe color at (0,0) > Expected: 0 255 0 0 > Observed: 0 0 0 0 > > Is need to make additional test witch cover that behavior, or add assert to > that test? I'm not sure what you'd assert in the max-size test. A new test can be created or an existing one modified. I'm guessing adding a test for GL_ARB_shader_image_size would be the easiest way to test the issue.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>