This patch series contains an alternative proposal to the patch "Split accuracy test to allow new multisample tests utilize this code" from 5/4, along the lines Anuj and I discussed over email yesterday. Anuj, let me know if you think this alternative is reasonable. I went ahead and left you as the author of the patch, but I would be equally happy taking over authorship if you'd prefer.
Patch 2/2 contains the turn-on-off test, with the necessary modifications to work with the new refactor. I have the impression that this test is still a work in progress--is that right, Anuj? I got that impression because (a) the test ignores its <test_type> command line argument (it always tests in "color" mode), (b) the "small" variant of the test fails on my nVidia reference platform, and (c) the patch doesn't add the test to all.tests. For what it's worth, I think it would be ok to remove the command-line options for the turn-on-off test (hardcoding to num_samples=4, test_type=color, small=false, and depthstencil=true) since varying those options is unlikely to expose any bugs that wouldn't already be caught by the accuracy test. [PATCH v2 1/2] Split accuracy test to allow new multisample tests utilize this code [PATCH v2 2/2] Add test to turn on/off MSAA in a FBO _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
