st/dri: Track drawable context bindings
authorJakob Bornecrantz <wallbraker@gmail.com>
Wed, 1 Dec 2010 04:04:25 +0000 (05:04 +0100)
committerMarek Olšák <maraeo@gmail.com>
Sun, 20 Feb 2011 15:31:48 +0000 (16:31 +0100)
commit94ccc31ba4f64ac480137fd90f1ded44d2072f6e
tree45bdac7b10a933233353c472f5ab69e35cca85d9
parent9e872a5865c66ed0a518dd1c6c54e72f3afa71f1
st/dri: Track drawable context bindings

Needs to track this ourself since because we get into a race condition with
the dri_util.c code on make current when rendering to the front buffer.

This is what happens:
Old context is rendering to the front buffer.

App calls MakeCurrent with a new context. dri_util.c sets
drawable->driContextPriv to the new context and then calls the driver make
current. st/dri make current flushes the old context, which calls back into
st/dri via the flush frontbuffer hook. st/dri calls dri loader flush
frontbuffer, which calls invalidate buffer on the drawable into st/dri.

This is where things gets wrong. st/dri grabs the context from the dri
drawable (which now points to the new context) and calls invalidate
framebuffer to the new context which has not yet set the new drawable as its
framebuffers since we have not called make current yet, it asserts.
src/gallium/state_trackers/dri/common/dri_context.c
src/gallium/state_trackers/dri/common/dri_drawable.c
src/gallium/state_trackers/dri/common/dri_drawable.h
src/gallium/state_trackers/dri/drm/dri2.c