diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2010-08-20 18:25:16 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-08 05:13:28 -0400 |
commit | 75ef9da2cdb64e7926404dd2b755bbbfe98eaeaf (patch) | |
tree | ef200ecba7bb6c9a8cb2ee3d830f30c5d5377207 /drivers/gpu/drm/i915/i915_gem.c | |
parent | c911fc1c6ad61b56869ee521f1a477c741b039da (diff) |
drm/i915: unload: fix retire_work races
ums-gem code correctly cancels the retire work (at lastclose time),
kms does not do so. Fix this by canceling the work right after ideling
the gpu.
While staring at the code I noticed that the work function is not
static. Fix this, too.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem.c')
-rw-r--r-- | drivers/gpu/drm/i915/i915_gem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 4cccdce5f80f..26eb6e31c743 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c | |||
@@ -1797,7 +1797,7 @@ i915_gem_retire_requests(struct drm_device *dev) | |||
1797 | i915_gem_retire_requests_ring(dev, &dev_priv->bsd_ring); | 1797 | i915_gem_retire_requests_ring(dev, &dev_priv->bsd_ring); |
1798 | } | 1798 | } |
1799 | 1799 | ||
1800 | void | 1800 | static void |
1801 | i915_gem_retire_work_handler(struct work_struct *work) | 1801 | i915_gem_retire_work_handler(struct work_struct *work) |
1802 | { | 1802 | { |
1803 | drm_i915_private_t *dev_priv; | 1803 | drm_i915_private_t *dev_priv; |