[PATCH 08/18] drm: Rip out totally bogus vga_switcheroo->can_switch locking

2014-04-17 Thread Thierry Reding
On Fri, Apr 11, 2014 at 11:36:05PM +0200, Daniel Vetter wrote: [...] > diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c > index 4dff829c2d89..dcb67c6564dd 100644 > --- a/drivers/gpu/drm/i915/i915_dma.c > +++ b/drivers/gpu/drm/i915/i915_dma.c > @@ -1286,9 +1286,12 @@ st

[PATCH 08/18] drm: Rip out totally bogus vga_switcheroo->can_switch locking

2014-04-11 Thread Daniel Vetter
So I just wanted to add a new field to struct drm_device and accidentally stumbled over something. According to comments dev->open_count is protected by dev->count_lock, but that's totally not the case. It's protected by drm_global_mutex. Unfortunately the vga switcheroo callbacks took this commen