[Intel-gfx] [PATCH] drm/i915: Check for dev->primary->master before dereference.

[Eric Anholt]

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 <chris at 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
eric at anholt.net                         eric.anholt at intel.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20090320/4428f1ce/attachment.sig>