While looking at use cases of the wake queues in order to add support
for simple wait queues, I noticed that there was only a single user of
wake_up_interruptible_nr(), and that use was doing a single task wake
up. Have that user use the proper wake_up_interruptible() instead, and
perhaps we can even remove the function wake_up_interruptible_nr().

Signed-off-by: Steven Rostedt <rost...@goodmis.org>
---
diff --git a/drivers/media/usb/cx231xx/cx231xx-video.c 
b/drivers/media/usb/cx231xx/cx231xx-video.c
index c6ff896..9798160 100644
--- a/drivers/media/usb/cx231xx/cx231xx-video.c
+++ b/drivers/media/usb/cx231xx/cx231xx-video.c
@@ -1875,7 +1875,7 @@ static int cx231xx_close(struct file *filp)
                        v4l2_fh_exit(&fh->fh);
                        kfree(fh);
                        dev->users--;
-                       wake_up_interruptible_nr(&dev->open, 1);
+                       wake_up_interruptible(&dev->open);
                        return 0;
                }
 
@@ -1908,7 +1908,7 @@ static int cx231xx_close(struct file *filp)
        }
        v4l2_fh_exit(&fh->fh);
        kfree(fh);
-       wake_up_interruptible_nr(&dev->open, 1);
+       wake_up_interruptible(&dev->open);
        return 0;
 }
 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to