[PATCH] drm/omap: avoid aquiring mutex in atomic context (v2)

2012-01-01 Thread Felipe Contreras
On Fri, Dec 16, 2011 at 7:34 PM, Rob Clark wrote: > --- a/drivers/staging/omapdrm/omap_gem.c > +++ b/drivers/staging/omapdrm/omap_gem.c > @@ -538,10 +538,22 @@ int omap_gem_roll(struct drm_gem_object *obj, uint32_t > roll) > ? ? ? ? ? ? ? ?return -EINVAL; > ? ? ? ?} > > - ? ? ? mutex_lock(&obj->d

Re: [PATCH] drm/omap: avoid aquiring mutex in atomic context (v2)

2012-01-01 Thread Felipe Contreras
On Fri, Dec 16, 2011 at 7:34 PM, Rob Clark wrote: > --- a/drivers/staging/omapdrm/omap_gem.c > +++ b/drivers/staging/omapdrm/omap_gem.c > @@ -538,10 +538,22 @@ int omap_gem_roll(struct drm_gem_object *obj, uint32_t > roll) >                return -EINVAL; >        } > > -       mutex_lock(&obj->d

[PATCH] drm/omap: avoid aquiring mutex in atomic context (v2)

2011-12-16 Thread Rob Clark
From: Rob Clark omap_gem_roll() could be called by fbcon in atomic context or when struct_mutext is held. Avoid aquiring mutex (deadlock), or calling tiler_pin() (which itself is not safe for atomic context) in these cases. Signed-off-by: Rob Clark --- drivers/staging/omapdrm/omap_gem.c | 1

[PATCH] drm/omap: avoid aquiring mutex in atomic context (v2)

2011-12-16 Thread Rob Clark
From: Rob Clark omap_gem_roll() could be called by fbcon in atomic context or when struct_mutext is held. Avoid aquiring mutex (deadlock), or calling tiler_pin() (which itself is not safe for atomic context) in these cases. Signed-off-by: Rob Clark --- drivers/staging/omapdrm/omap_gem.c | 1