[Nouveau] [PATCH] drm/nouveau: init some list_heads

[Pekka Paalanen]

On Wed, 26 Aug 2009 23:53:36 +0200
Maarten Maathuis <madman2003 at gmail.com> wrote:

> - In the case of nvbo->head it is really important to avoid an OOPS if ttm_buffer_object_init fails.
> 
> Signed-off-by: Maarten Maathuis <madman2003 at gmail.com>

Aren't you going to push this?
I still haven't really reviewed this, but go ahead.

> ---
>  drivers/gpu/drm/nouveau/nouveau_bo.c      |    2 ++
>  drivers/gpu/drm/nouveau/nouveau_channel.c |    1 +
>  drivers/gpu/drm/nouveau/nouveau_object.c  |    1 +
>  3 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
> index 24d6f8f..6b8ef6b 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
> @@ -63,6 +63,8 @@ nouveau_bo_new(struct drm_device *dev, struct nouveau_channel *chan,
>  	nvbo = kzalloc(sizeof(struct nouveau_bo), GFP_KERNEL);
>  	if (!nvbo)
>  		return -ENOMEM;
> +	INIT_LIST_HEAD(&nvbo->head);
> +	INIT_LIST_HEAD(&nvbo->entry);
>  	nvbo->mappable = mappable;
>  	nvbo->no_vm = no_vm;
>  	nvbo->tile_mode = tile_mode;
> diff --git a/drivers/gpu/drm/nouveau/nouveau_channel.c b/drivers/gpu/drm/nouveau/nouveau_channel.c
> index 382c5b5..6c31d7f 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_channel.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_channel.c
> @@ -138,6 +138,7 @@ nouveau_channel_alloc(struct drm_device *dev, struct nouveau_channel **chan_ret,
>  		return -ENOMEM;
>  	dev_priv->fifo_alloc_count++;
>  	chan = dev_priv->fifos[channel];
> +	INIT_LIST_HEAD(&chan->nvsw.vbl_wait);
>  	chan->dev = dev;
>  	chan->id = channel;
>  	chan->file_priv = file_priv;
> diff --git a/drivers/gpu/drm/nouveau/nouveau_object.c b/drivers/gpu/drm/nouveau/nouveau_object.c
> index 65d8d25..efcce79 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_object.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_object.c
> @@ -468,6 +468,7 @@ nouveau_gpuobj_ref_add(struct drm_device *dev, struct nouveau_channel *chan,
>  	ref = kzalloc(sizeof(*ref), GFP_KERNEL);
>  	if (!ref)
>  		return -ENOMEM;
> +	INIT_LIST_HEAD(&ref->list);
>  	ref->gpuobj   = gpuobj;
>  	ref->channel  = chan;
>  	ref->instance = instance;
> -- 
> 1.6.4
> 
> _______________________________________________
> Nouveau mailing list
> Nouveau at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
> 


-- 
Pekka Paalanen
http://www.iki.fi/pq/