Bug 93308 - Primitive restart tests enforce pre-October 2014 spec behavior
Summary: Primitive restart tests enforce pre-October 2014 spec behavior
Alias: None
Product: piglit
Classification: Unclassified
Component: tests (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Piglit Mailing List
QA Contact: Piglit Mailing List
Depends on:
Reported: 2015-12-09 20:14 UTC by Ian Romanick
Modified: 2018-01-16 19:30 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Description Ian Romanick 2015-12-09 20:14:13 UTC
In versions of the desktop OpenGL spec prior to glspec45.core.20141030.pdf, section 10.3.6 (Primitive Restart) said:

    "When one of the Draw* commands transfers..."

Starting with the 20141030 version, it says:

    "When one of the DrawElements* commands transfers...."

This was a fix for Khronos bug #12893, and it was intended to be a retroactive change.

tests/general/primitive-restart.c checks that glDrawArrays respects primitive restart.  This should be changed to check that glDrawArrays igores primitive restart.

tests/spec/gl-3.1/primitive-restart-xfb.c also needs some sort of updates.  At the very least the test should document which behavior it expects.  The only comment in the whole test is, "Test if primitive restart works with glDrawArraysIndirect."

tests/spec/arb_es3_compatibility/es3-drawarrays-primrestart-fixedindex.c needs the same verification.
Comment 1 Fabian Bieler 2018-01-16 19:30:00 UTC
As of 70e7af15b75eb911d2c03c133418a39de0fad0d0 all Piglit primitive restart tests for non indexed draw calls should expect the GL to ignore the primitive restart index.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.