Bug 44247 - piglit destroys Windows that are currently bound to a GLX context
Summary: piglit destroys Windows that are currently bound to a GLX context
Status: RESOLVED FIXED
Alias: None
Product: piglit
Classification: Unclassified
Component: infrastructure (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Eric Anholt
QA Contact: Eric Anholt
URL:
Whiteboard:
Keywords:
: 43317 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-12-28 15:52 UTC by Nicholas Miell
Modified: 2012-01-02 14:53 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Check if the window being destroyed is currently bound to a GLX context, and release the GLX context if necessary (1.49 KB, patch)
2011-12-28 15:52 UTC, Nicholas Miell
Details | Splinter Review

Description Nicholas Miell 2011-12-28 15:52:09 UTC
Created attachment 54922 [details] [review]
Check if the window being destroyed is currently bound to a GLX context, and release the GLX context if necessary

GLEAN::Window::~Window calls XDestroyWindow on the currently bound GLX drawable, which causes the subsequent test to explode when it calls glXMakeCurrent and Mesa attempts to flush the outstanding state to the now destroyed window.

This manifests as a BadDrawable error from a XPutImage call when using software rendering and causes the basic sanity test to fail.
Comment 1 Brian Paul 2012-01-02 14:06:01 UTC
The patch looks good to me.  I'll push it soon.
Comment 2 Brian Paul 2012-01-02 14:53:17 UTC
*** Bug 43317 has been marked as a duplicate of this bug. ***


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.