CHROMIUM: uvcvideo: Handle status_start while status_stop is in progress.
authorShawn Nematbakhsh <shawnn@google.com>
Tue, 23 Apr 2013 22:50:23 +0000 (15:50 -0700)
committerChromeBot <chrome-bot@google.com>
Wed, 24 Apr 2013 16:58:26 +0000 (09:58 -0700)
commit70fec1cb10165bfd6b2b45edd40dd146fef850e6
tree05bc869f981b547214cdb0d6fec96cbc5f41e680
parentffda9965b095730d206b931ec2088e924474781f
CHROMIUM: uvcvideo: Handle status_start while status_stop is in progress.

While usb_kill_urb is in progress, calls to usb_submit_urb will fail
with -EPERM (documented in Documentation/usb/URB.txt). The UVC driver
does not correctly handle this case -- there is no synchronization
between uvc_v4l2_open / uvc_status_start and uvc_v4l2_release /
uvc_status_stop.

This patch adds a retry / timeout when uvc_status_open / usb_submit_urb
returns -EPERM. This usually means that usb_kill_urb is in progress, and
we just need to wait a while.

BUG=chromium:226216
TEST=Manual. Repeat suspend / resume on Stout many times, verify that
open() on /dev/video0 never fails.

Change-Id: I8a69dab345a18c89e175bd916aa943edd080b6cc
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/48982
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
drivers/media/video/uvc/uvc_v4l2.c
drivers/media/video/uvc/uvcvideo.h