Re: [GIT PATCHES FOR 2.6.37] BKL removal patch series

2010-11-27 Thread Hans Verkuil
On Friday, November 26, 2010 18:41:32 Laurent Pinchart wrote:
> Hi Hans,
> 
> On Friday 26 November 2010 11:09:38 Hans Verkuil wrote:
> > Changes since the last patch series:
> > 
> > - Fix a race condition between video_unregister_device and v4l2_open.
> > - Return -ERESTART if the mutex_lock_interruptible returns non-zero.
> 
> What's the difference between ERESTART and ERESTARTSYS ?

Grrr, I picked the wrong one. I respun my tree, now using ERESTARTSYS.

Thanks for catching this!

Regards,

Hans

> > - Return POLLERR|POLLHUP when poll is used on an unregistered device.
> 
> 

-- 
Hans Verkuil - video4linux developer - sponsored by Cisco
--
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


Re: [GIT PATCHES FOR 2.6.37] BKL removal patch series

2010-11-26 Thread Laurent Pinchart
Hi Hans,

On Friday 26 November 2010 11:09:38 Hans Verkuil wrote:
> Changes since the last patch series:
> 
> - Fix a race condition between video_unregister_device and v4l2_open.
> - Return -ERESTART if the mutex_lock_interruptible returns non-zero.

What's the difference between ERESTART and ERESTARTSYS ?

> - Return POLLERR|POLLHUP when poll is used on an unregistered device.

-- 
Regards,

Laurent Pinchart
--
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


[GIT PATCHES FOR 2.6.37] BKL removal patch series

2010-11-26 Thread Hans Verkuil
Changes since the last patch series:

- Fix a race condition between video_unregister_device and v4l2_open.
- Return -ERESTART if the mutex_lock_interruptible returns non-zero.
- Return POLLERR|POLLHUP when poll is used on an unregistered device.

Regards,

Hans

The following changes since commit e53beacd23d9cb47590da6a7a7f6d417b941a994:
  Linus Torvalds (1):
Linux 2.6.37-rc2

are available in the git repository at:

  ssh://linuxtv.org/git/hverkuil/media_tree.git bkl-lock-int

Hans Verkuil (15):
  BKL: trivial BKL removal from V4L2 radio drivers
  cadet: use unlocked_ioctl
  tea5764: convert to unlocked_ioctl
  si4713: convert to unlocked_ioctl
  typhoon: convert to unlocked_ioctl.
  BKL: trivial ioctl -> unlocked_ioctl video driver conversions
  sn9c102: convert to unlocked_ioctl.
  et61x251_core: trivial conversion to unlocked_ioctl.
  cafe_ccic: replace ioctl by unlocked_ioctl.
  sh_vou: convert to unlocked_ioctl.
  radio-timb: convert to unlocked_ioctl.
  cx18: convert to unlocked_ioctl.
  v4l2-dev: use mutex_lock_interruptible instead of plain mutex_lock
  V4L: improve the BKL replacement heuristic
  v4l2-dev: fix race condition.

 drivers/media/radio/radio-aimslab.c  |   16 +++---
 drivers/media/radio/radio-aztech.c   |6 +-
 drivers/media/radio/radio-cadet.c|   12 +++-
 drivers/media/radio/radio-gemtek-pci.c   |6 +-
 drivers/media/radio/radio-gemtek.c   |   14 +++---
 drivers/media/radio/radio-maestro.c  |   14 ++---
 drivers/media/radio/radio-maxiradio.c|2 +-
 drivers/media/radio/radio-miropcm20.c|6 ++-
 drivers/media/radio/radio-rtrack2.c  |   10 ++--
 drivers/media/radio/radio-sf16fmi.c  |7 ++-
 drivers/media/radio/radio-sf16fmr2.c |   11 ++--
 drivers/media/radio/radio-si4713.c   |3 +-
 drivers/media/radio/radio-tea5764.c  |   49 +++---
 drivers/media/radio/radio-terratec.c |8 ++--
 drivers/media/radio/radio-timb.c |5 ++-
 drivers/media/radio/radio-trust.c|   18 +++---
 drivers/media/radio/radio-typhoon.c  |   16 +++---
 drivers/media/radio/radio-zoltrix.c  |   30 ++--
 drivers/media/video/arv.c|2 +-
 drivers/media/video/bw-qcam.c|2 +-
 drivers/media/video/c-qcam.c |2 +-
 drivers/media/video/cafe_ccic.c  |2 +-
 drivers/media/video/cx18/cx18-alsa-pcm.c |8 +++-
 drivers/media/video/cx18/cx18-streams.c  |2 +-
 drivers/media/video/et61x251/et61x251_core.c |2 +-
 drivers/media/video/meye.c   |   14 +++---
 drivers/media/video/pms.c|2 +-
 drivers/media/video/sh_vou.c |   13 +++--
 drivers/media/video/sn9c102/sn9c102_core.c   |2 +-
 drivers/media/video/v4l2-dev.c   |   69 +++---
 drivers/media/video/v4l2-device.c|1 +
 drivers/media/video/w9966.c  |2 +-
 include/media/v4l2-device.h  |2 +
 33 files changed, 191 insertions(+), 167 deletions(-)

-- 
Hans Verkuil - video4linux developer - sponsored by Cisco
--
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