Bug 3972 - [r200 / radeon / r300] segfault in r200DestroyContext when R200_NO_VTXFMT set
Summary: [r200 / radeon / r300] segfault in r200DestroyContext when R200_NO_VTXFMT set
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/r200 (show other bugs)
Version: git
Hardware: All All
: high normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
: 4872 6029 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-08-04 04:33 UTC by Ian Romanick
Modified: 2009-08-24 12:23 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Ian Romanick 2005-08-04 04:33:37 UTC
If the R200_NO_VTXFMT env var is set, the r200 driver will segfault during
context destruction.  The problem is that the env var isn't tested to determine
if r200VtxfmtDestroy should be called.  I suspect that the radeon and r300
drivers will have a similar problem, but I have not verified that yet.

I believe we should do the following two things:

1. Add a flag that is set when r200VtxfmtInit is called.  Use this flag, and
*only* this flag to determine if r200VtxfmtDestroy should be called.

2. Mark all the env vars that are replaced by dri conf as deprecated.  Remove
them in the next release (i.e., X.org 7.1).
Comment 1 Ian Romanick 2005-08-04 04:34:53 UTC
Created attachment 3242 [details] [review]
mesa-6.3.2-freebsd.patch

This implements item #1 in the original bug description.
Comment 2 Eric Anholt 2005-08-05 14:52:24 UTC
as far as this env var, I'd say skip directly to the end of #2.  It's not
something that normal users are using.
Comment 3 Ian Romanick 2005-08-12 05:48:03 UTC
I removed support for R200_NO_VTXFMT.  Closing as fixed.  We can go through and
gut all the env variables from all the drivers later.
Comment 4 Roland Scheidegger 2006-02-17 04:38:21 UTC
*** Bug 4872 has been marked as a duplicate of this bug. ***
Comment 5 Roland Scheidegger 2006-02-25 11:43:47 UTC
*** Bug 6029 has been marked as a duplicate of this bug. ***
Comment 6 Adam Jackson 2009-08-24 12:23:19 UTC
Mass version move, cvs -> git


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.