RE: [PATCH 0/6] staging: comedi: fix some minor issues with file poll op

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, October 09, 2015 4:27 AM, Ian Abbott wrote:
> A few changes for the "poll" file operation to avoid poll-waiting on the
> same subdevice for both read and write (patch 1), avoid allocating write
> buffer space unnecessarily and possibly inappropriately (patch 4),
> consider whether any active commands belong to the current file object
> (patch 5), and avoid using the main mutex (for performance reasons)
> (patch 6).
>
> 1) staging: comedi: don't poll_wait on same subdevice twice
> 2) staging: comedi: rename comedi_buf_write_n_available
> 3) staging: comedi: add new comedi_buf_write_n_available()
> 4) staging: comedi: don't allocate buffer space when polling for write
> 5) staging: comedi: check command started by file being polled
> 6) staging: comedi: don't use mutex when polling file
>
>  drivers/staging/comedi/comedi_buf.c      | 19 +++++++++++++------
>  drivers/staging/comedi/comedi_fops.c     | 17 +++++++++--------
>  drivers/staging/comedi/comedi_internal.h |  1 +
>  3 files changed, 23 insertions(+), 14 deletions(-)

Thanks!

Reviewed-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux