On Tue, Aug 21, 2012 at 6:57 PM, Matt Turner <[email protected]> wrote: > On Tue, Aug 21, 2012 at 4:32 PM, Anuj Phogat <[email protected]> wrote: >> Draw and probe a QUAD (in place of a LINE) in case of GL_TEXTURE_1D. This >> fixes the test failures on NVIDIA's proprietary linux drivers. In case of >> LINE, glReadPixels() misses the relevant pixels by one pixel on NVIDIA. >> >> Signed-off-by: Anuj Phogat <[email protected]> >> --- >> Brian, both these patches (including the one I sent earlier today) resolve >> separate issues. So, I'm keeping this fix in a separate patch. Let me know >> if you have any comments on the patches. >> >> tests/texturing/copyteximage.c | 16 +++++----------- >> 1 files changed, 5 insertions(+), 11 deletions(-) > > I've only been following this casually, but does this cover up a test > failure on NVIDIA? They've got a bug -- right? I don't understand why > we'd want to modify the test so that it misses the driver bug.
Hitting specific pixels when drawing lines is a bit tricky. The vertex transformation has to be set up just right (one of the appendices of the OpenGL programming guide covers this) and I don't think this piglit test is doing that. I doubt it's a driver bug. -Brian _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
