[Piglit] [PATCH 7/7] copypixels-sync: New test for i965 IRQ handling bug.

[Eric Anholt]

On Fri, 20 May 2011 10:59:10 +0200, Ian Romanick <idr at freedesktop.org> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 05/20/2011 04:09 AM, Eric Anholt wrote:
> > ---
> >  tests/all.tests                 |    1 +
> >  tests/general/CMakeLists.gl.txt |    1 +
> >  tests/general/copypixels-sync.c |   79 +++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 81 insertions(+), 0 deletions(-)
> >  create mode 100644 tests/general/copypixels-sync.c
> > 
> > diff --git a/tests/all.tests b/tests/all.tests
> > index a18cbfe..0c22a0a 100644
> > --- a/tests/all.tests
> > +++ b/tests/all.tests
> > @@ -177,6 +177,7 @@ add_plain_test(general, 'bgra-vert-attrib-pointer')
> >  add_plain_test(general, 'blendminmax')
> >  add_plain_test(general, 'blendsquare')
> >  add_plain_test(general, 'clear-varray-2.0')
> > +add_plain_test(general, 'copypixels-sync')
> >  add_plain_test(general, 'depthrange-clear')
> >  add_plain_test(general, 'depth_clamp')
> >  add_plain_test(general, 'depth-clamp-range')
> > diff --git a/tests/general/CMakeLists.gl.txt b/tests/general/CMakeLists.gl.txt
> > index 66d65a0..3de0d2a 100644
> > --- a/tests/general/CMakeLists.gl.txt
> > +++ b/tests/general/CMakeLists.gl.txt
> > @@ -18,6 +18,7 @@ add_executable (bgra-sec-color-pointer bgra-sec-color-pointer.c)
> >  add_executable (blendminmax blendminmax.c)
> >  add_executable (blendsquare blendsquare.c)
> >  add_executable (clear-varray-2.0 clear-varray-2.0.c)
> > +add_executable (copypixels-sync copypixels-sync.c)
> >  add_executable (depth-clamp-range depth-clamp-range.c)
> >  add_executable (dlist-clear dlist-clear.c)
> >  add_executable (dlist-fdo3129-01 dlist-fdo3129-01.c)
> > diff --git a/tests/general/copypixels-sync.c b/tests/general/copypixels-sync.c
> > new file mode 100644
> > index 0000000..9a13628
> > --- /dev/null
> > +++ b/tests/general/copypixels-sync.c
> > @@ -0,0 +1,79 @@
> > +/*
> > + * Copyright © 2011 Intel Corporation
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining a
> > + * copy of this software and associated documentation files (the "Software"),
> > + * to deal in the Software without restriction, including without limitation
> > + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice (including the next
> > + * paragraph) shall be included in all copies or substantial portions of the
> > + * Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> > + * IN THE SOFTWARE.
> > + */
> > +
> > +/**
> > + * @file copypixels-sync.c
> > + *
> > + * Test repeated readback of small glCopyPixels calls.  This stresses
> > + * blit ring IRQ handling on the i965 driver.
> > + */
> > +
> > +#include "piglit-util.h"
> > +#include "piglit-framework.h"
> > +
> > +int piglit_width = 64;
> > +int piglit_height = 64;
> > +int piglit_window_mode = GLUT_RGB | GLUT_DOUBLE | GLUT_ALPHA;
> > +
> > +enum piglit_result
> > +piglit_display(void)
> > +{
> > +	int x, y;
> > +	float green[4] = {0.0, 1.0, 0.0, 0.0};
> > +	bool pass = true;
> > +
> > +	piglit_ortho_projection(piglit_width, piglit_height, GL_FALSE);
> > +
> > +	glClearColor(1.0, 0.0, 0.0, 0.0);
> > +	glClear(GL_COLOR_BUFFER_BIT);
> > +
> > +	glColor4fv(green);
> > +	piglit_draw_rect(0, 0, 1, 1);
> > +
> > +	for (y = 0; y < piglit_height; y++) {
> > +		for (x = 0; x < piglit_width; x++) {
> > +			if (x == 0 && y == 0)
> > +				continue;
> > +
> > +			glRasterPos2i(x, y);
> > +			if (x != 0) {
> > +				glCopyPixels(x - 1, y, 1, 1, GL_COLOR);
> > +			} else {
> > +				glCopyPixels(piglit_width - 1, y - 1, 1, 1,
> > +					     GL_COLOR);
> > +			}
> > +
> > +			pass = pass && piglit_probe_pixel_rgba(x, y, green);
> 
> Same question as for the previous patch.

Usually I do it this way so that when there's a failure, you don't get
64 * 64 printouts of the same complaint.  It's split about half and half
in the repo.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20110520/ceb324c3/attachment.pgp>