i965: Allow mapped VBOs during drawing in non-debug contexts.
authorKenneth Graunke <kenneth@whitecape.org>
Sun, 30 Jul 2017 23:15:56 +0000 (16:15 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 11 Oct 2017 19:22:25 +0000 (12:22 -0700)
commite401cfa28a6e32d7081f9de998e731965ce966fb
tree65ff65ec43115016ced60526288004816abb773a
parent3c66a461f3180d0fc7270a7c919df16bb0a8d6e0
i965: Allow mapped VBOs during drawing in non-debug contexts.

Section 6.3.2 of the GL 4.5 spec says:

   "Any GL command which attempts to read from, write to, or change
    the state of a buffer object may generate an INVALID_OPERATION error
    if all or part of the buffer object is mapped ... However, only
    commands which explicitly describe this error are required to do so.
    If an error is not generated, such commands will have undefined
    results and may result in GL interruption or termination."

Setting this flag allows us to skip walking over the buffer bindings
for every enabled vertex attribute (_mesa_all_buffers_are_unmapped).

Improves performance in GFXBench4's gl_driver2_off microbenchmark by
3.05797% +/- 0.709031% (n=33) on Apollolake.

This breaks KHR-*.draw_elements_base_vertex_tests.invalid_mapped_bos,
but that test is invalid and has been removed from the upstream CTS.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/brw_context.c