Re: [PATCH] nouveau: fix channel closing

[Ben Skeggs <skeggsb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On Tue, 2012-05-01 at 23:24 +0200, Marcin Slusarz wrote:
> Restore code lost in libdrm_nouveau rewrite.
Thanks, pushed.

> ---
>  nouveau/nouveau.c |   17 ++++++++++++-----
>  1 files changed, 12 insertions(+), 5 deletions(-)
> 
> diff --git a/nouveau/nouveau.c b/nouveau/nouveau.c
> index f0bc2c3..5aa4107 100644
> --- a/nouveau/nouveau.c
> +++ b/nouveau/nouveau.c
> @@ -278,15 +278,22 @@ nouveau_object_new(struct nouveau_object *parent, uint64_t handle,
>  void
>  nouveau_object_del(struct nouveau_object **pobj)
>  {
> -	struct drm_nouveau_gpuobj_free req;
>  	struct nouveau_object *obj = *pobj;
>  	struct nouveau_device *dev;
>  	if (obj) {
>  		dev = nouveau_object_find(obj, NOUVEAU_DEVICE_CLASS);
> -		req.channel = obj->parent->handle;
> -		req.handle  = obj->handle;
> -		drmCommandWrite(dev->fd, DRM_NOUVEAU_GPUOBJ_FREE,
> -				&req, sizeof(req));
> +		if (obj->oclass == NOUVEAU_FIFO_CHANNEL_CLASS) {
> +			struct drm_nouveau_channel_free req;
> +			req.channel = obj->handle;
> +			drmCommandWrite(dev->fd, DRM_NOUVEAU_CHANNEL_FREE,
> +					&req, sizeof(req));
> +		} else {
> +			struct drm_nouveau_gpuobj_free req;
> +			req.channel = obj->parent->handle;
> +			req.handle  = obj->handle;
> +			drmCommandWrite(dev->fd, DRM_NOUVEAU_GPUOBJ_FREE,
> +					&req, sizeof(req));
> +		}
>  	}
>  	free(obj);
>  	*pobj = NULL;