[Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-16 Thread Chris Wilson
We have two once very similar functions, i915_gpu_idle() and i915_gem_idle(). The former is used as the lower level operation to flush work on the GPU, whereas the latter is the high level interface to flush the GEM bookkeeping in addition to flushing the GPU. As such i915_gem_idle() also clears

Re: [Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-16 Thread Daniel Vetter
On Wed, Oct 16, 2013 at 11:50:01AM +0100, Chris Wilson wrote: We have two once very similar functions, i915_gpu_idle() and i915_gem_idle(). The former is used as the lower level operation to flush work on the GPU, whereas the latter is the high level interface to flush the GEM bookkeeping in

Re: [Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-16 Thread Chris Wilson
On Wed, Oct 16, 2013 at 01:50:29PM +0200, Daniel Vetter wrote: On Wed, Oct 16, 2013 at 11:50:01AM +0100, Chris Wilson wrote: We have two once very similar functions, i915_gpu_idle() and i915_gem_idle(). The former is used as the lower level operation to flush work on the GPU, whereas the

Re: [Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-16 Thread Daniel Vetter
On Wed, Oct 16, 2013 at 05:47:26PM +0100, Chris Wilson wrote: On Wed, Oct 16, 2013 at 01:50:29PM +0200, Daniel Vetter wrote: On Wed, Oct 16, 2013 at 11:50:01AM +0100, Chris Wilson wrote: We have two once very similar functions, i915_gpu_idle() and i915_gem_idle(). The former is used as

[Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-11 Thread Chris Wilson
We have two once very similar functions, i915_gpu_idle() and i915_gem_idle(). The former is used as the lower level operation to flush work on the GPU, whereas the latter is the high level interface to flush the GEM bookkeeping in addition to flushing the GPU. As such i915_gem_idle() also clears

Re: [Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-11 Thread Daniel Vetter
On Fri, Oct 11, 2013 at 11:14 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: void @@ -4711,11 +4707,9 @@ i915_gem_lastclose(struct drm_device *dev) if (drm_core_check_feature(dev, DRIVER_MODESET)) return; - mutex_lock(dev-struct_mutex); - ret =

Re: [Intel-gfx] [PATCH] drm/i915: Disable all GEM timers and work on unload

2013-10-11 Thread Chris Wilson
On Fri, Oct 11, 2013 at 06:22:55PM +0200, Daniel Vetter wrote: On Fri, Oct 11, 2013 at 11:14 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: void @@ -4711,11 +4707,9 @@ i915_gem_lastclose(struct drm_device *dev) if (drm_core_check_feature(dev, DRIVER_MODESET))