On Fri, 2009-03-06 at 23:27 +0000, Chris Wilson wrote:
> I've hit the occasional oops inside i915_wait_ring() with an indication of
> a NULL derefence of dev->primary->master.  Adding a NULL check is
> consistent with the other potential users of dev->primary->master.
> 
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>

Applied.

Given how many bugs we've had with these perf_boxes settings, we may
want to just nuke them at some point.  It looks like the userland debug
code that used the flags went away in 2007.

Does anyone care for them continuing to exist?  Do we have any
obligation to keep them?  I think for any future performance debug like
these did, we'd just expose values in debugfs (it keeps the debug code
from interfering with app behavior, anyway).

-- 
Eric Anholt
e...@anholt.net                         eric.anh...@intel.com


Attachment: signature.asc
Description: This is a digitally signed message part

------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to