USB: usb-skeleton.c: add retry for nonblocking read
authorChen Wang <unicornxx.wang@gmail.com>
Fri, 19 Jul 2013 02:15:18 +0000 (10:15 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 25 Jul 2013 19:01:13 +0000 (12:01 -0700)
Updated skel_read() in usb-skeleton.c. When there is no data in the
buffer, we would allow retry for both blocking and nonblocking cases.
Original logic give retry only for blocking case. Actually we can also
allow retry for nonblocking case. This will reuse the existing retry
logic and handle the return of -EAGAIN in one place. Also if the data to
be read is short and can be retrieved in quick time, we can also give a
chance for nonblocking case and may catch the data and copy it back to
userspace in one read() call too.

Signed-off-by: Chen Wang <unicornxx.wang@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/usb-skeleton.c

index 7ed3b039dbe878a8a7c7db038e6150c3395e5bec..ff97652343a34fdde446f981f14b01ec1c65280c 100644 (file)
@@ -325,9 +325,8 @@ retry:
                rv = skel_do_read_io(dev, count);
                if (rv < 0)
                        goto exit;
-               else if (!(file->f_flags & O_NONBLOCK))
+               else
                        goto retry;
-               rv = -EAGAIN;
        }
 exit:
        mutex_unlock(&dev->io_mutex);