[media] v4l: vsp1: Don't sleep in atomic context
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Mon, 13 Apr 2015 14:43:40 +0000 (11:43 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 17 Jul 2015 13:03:43 +0000 (10:03 -0300)
The vsp1_entity_is_streaming() function is called in atomic context when
queuing buffers, and sleeps due to a mutex. As the mutex just protects
access to one structure field, fix this by replace the mutex with a
spinlock.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/vsp1/vsp1_entity.c
drivers/media/platform/vsp1/vsp1_entity.h

index a453bb4ddd37df65562bf98f571c332e239ca628..fd95a75b04f4c04ae63218f1f774e8f3fe7bf760 100644 (file)
 
 bool vsp1_entity_is_streaming(struct vsp1_entity *entity)
 {
+       unsigned long flags;
        bool streaming;
 
-       mutex_lock(&entity->lock);
+       spin_lock_irqsave(&entity->lock, flags);
        streaming = entity->streaming;
-       mutex_unlock(&entity->lock);
+       spin_unlock_irqrestore(&entity->lock, flags);
 
        return streaming;
 }
 
 int vsp1_entity_set_streaming(struct vsp1_entity *entity, bool streaming)
 {
+       unsigned long flags;
        int ret;
 
-       mutex_lock(&entity->lock);
+       spin_lock_irqsave(&entity->lock, flags);
        entity->streaming = streaming;
-       mutex_unlock(&entity->lock);
+       spin_unlock_irqrestore(&entity->lock, flags);
 
        if (!streaming)
                return 0;
@@ -49,9 +51,9 @@ int vsp1_entity_set_streaming(struct vsp1_entity *entity, bool streaming)
 
        ret = v4l2_ctrl_handler_setup(entity->subdev.ctrl_handler);
        if (ret < 0) {
-               mutex_lock(&entity->lock);
+               spin_lock_irqsave(&entity->lock, flags);
                entity->streaming = false;
-               mutex_unlock(&entity->lock);
+               spin_unlock_irqrestore(&entity->lock, flags);
        }
 
        return ret;
@@ -193,7 +195,7 @@ int vsp1_entity_init(struct vsp1_device *vsp1, struct vsp1_entity *entity,
        if (i == ARRAY_SIZE(vsp1_routes))
                return -EINVAL;
 
-       mutex_init(&entity->lock);
+       spin_lock_init(&entity->lock);
 
        entity->vsp1 = vsp1;
        entity->source_pad = num_pads - 1;
@@ -228,6 +230,4 @@ void vsp1_entity_destroy(struct vsp1_entity *entity)
        if (entity->subdev.ctrl_handler)
                v4l2_ctrl_handler_free(entity->subdev.ctrl_handler);
        media_entity_cleanup(&entity->subdev.entity);
-
-       mutex_destroy(&entity->lock);
 }
index 62c768d1c6aa985a92f343514704fa20a054ab94..8867a5787c285975c554c710531d2d12ab64b892 100644 (file)
@@ -14,7 +14,7 @@
 #define __VSP1_ENTITY_H__
 
 #include <linux/list.h>
-#include <linux/mutex.h>
+#include <linux/spinlock.h>
 
 #include <media/v4l2-subdev.h>
 
@@ -73,7 +73,7 @@ struct vsp1_entity {
 
        struct vsp1_video *video;
 
-       struct mutex lock;              /* Protects the streaming field */
+       spinlock_t lock;                /* Protects the streaming field */
        bool streaming;
 };