On Tue, 19 Jun 2012 17:16:01 +0200
[email protected] wrote:

> From: Daniel Vetter <[email protected]>
> 
> Otherwise userspace has no way to figure this out.
> 
> Signed-Off-by: Daniel Vetter <[email protected]>

It would probably be ideal to change the if DRIVER_GEM stuff to return
something else, just to make things a bit clearer to people.

Either way it's
Reviewed-by: Ben Widawsky <[email protected]>
> ---
>  drivers/gpu/drm/i915/i915_gem_context.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gem_context.c 
> b/drivers/gpu/drm/i915/i915_gem_context.c
> index fe3f21f..8f533cf 100644
> --- a/drivers/gpu/drm/i915/i915_gem_context.c
> +++ b/drivers/gpu/drm/i915/i915_gem_context.c
> @@ -480,6 +480,7 @@ int i915_switch_context(struct intel_ring_buffer *ring,
>  int i915_gem_context_create_ioctl(struct drm_device *dev, void *data,
>                                 struct drm_file *file)
>  {
> +     struct drm_i915_private *dev_priv = dev->dev_private;
>       struct drm_i915_gem_context_create *args = data;
>       struct drm_i915_file_private *file_priv = file->driver_priv;
>       struct i915_hw_context *ctx;
> @@ -488,6 +489,9 @@ int i915_gem_context_create_ioctl(struct drm_device *dev, 
> void *data,
>       if (!(dev->driver->driver_features & DRIVER_GEM))
>               return -ENODEV;
>  
> +     if (dev_priv->hw_contexts_disabled)
> +             return -ENODEV;
> +
>       ret = i915_mutex_lock_interruptible(dev);
>       if (ret)
>               return ret;



-- 
Ben Widawsky, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to