diff options
| author | Tejun Heo <tj@kernel.org> | 2013-02-27 20:04:10 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-27 22:10:16 -0500 |
| commit | c8c470afe316002d5d5259a134984ed03f40e249 (patch) | |
| tree | d2ed9627ac2bac97e9658211a35292a601204886 /drivers/gpu/drm | |
| parent | 8550cb2e3a3f4a5f2f6aa40b6c99691056f85b9d (diff) | |
drm/i915: convert to idr_alloc()
Convert to the much saner new idr interface.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: David Airlie <airlied@linux.ie>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/gpu/drm')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_gem_context.c | 21 |
1 files changed, 5 insertions, 16 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c index 21177d9df423..94d873a6cffb 100644 --- a/drivers/gpu/drm/i915/i915_gem_context.c +++ b/drivers/gpu/drm/i915/i915_gem_context.c | |||
| @@ -139,7 +139,7 @@ create_hw_context(struct drm_device *dev, | |||
| 139 | { | 139 | { |
| 140 | struct drm_i915_private *dev_priv = dev->dev_private; | 140 | struct drm_i915_private *dev_priv = dev->dev_private; |
| 141 | struct i915_hw_context *ctx; | 141 | struct i915_hw_context *ctx; |
| 142 | int ret, id; | 142 | int ret; |
| 143 | 143 | ||
| 144 | ctx = kzalloc(sizeof(*ctx), GFP_KERNEL); | 144 | ctx = kzalloc(sizeof(*ctx), GFP_KERNEL); |
| 145 | if (ctx == NULL) | 145 | if (ctx == NULL) |
| @@ -164,22 +164,11 @@ create_hw_context(struct drm_device *dev, | |||
| 164 | 164 | ||
| 165 | ctx->file_priv = file_priv; | 165 | ctx->file_priv = file_priv; |
| 166 | 166 | ||
| 167 | again: | 167 | ret = idr_alloc(&file_priv->context_idr, ctx, DEFAULT_CONTEXT_ID + 1, 0, |
| 168 | if (idr_pre_get(&file_priv->context_idr, GFP_KERNEL) == 0) { | 168 | GFP_KERNEL); |
| 169 | ret = -ENOMEM; | 169 | if (ret < 0) |
| 170 | DRM_DEBUG_DRIVER("idr allocation failed\n"); | ||
| 171 | goto err_out; | ||
| 172 | } | ||
| 173 | |||
| 174 | ret = idr_get_new_above(&file_priv->context_idr, ctx, | ||
| 175 | DEFAULT_CONTEXT_ID + 1, &id); | ||
| 176 | if (ret == 0) | ||
| 177 | ctx->id = id; | ||
| 178 | |||
| 179 | if (ret == -EAGAIN) | ||
| 180 | goto again; | ||
| 181 | else if (ret) | ||
| 182 | goto err_out; | 170 | goto err_out; |
| 171 | ctx->id = ret; | ||
| 183 | 172 | ||
| 184 | return ctx; | 173 | return ctx; |
| 185 | 174 | ||
