On Tuesday 16 November 2010, Hans Verkuil wrote: > I consider class 3 unacceptable for commonly seen devices. I did a quick scan > of the v4l drivers and the only common driver that falls in that class is uvc.
If uvc is the only important one, that should be easy enough to fix by adding a per-device mutex around uvc_v4l2_do_ioctl() or uvc_v4l2_ioctl(). > There is one other option, although it is very dirty: don't take the lock if > the ioctl command is VIDIOC_DQBUF. It works and reliably as well for uvc and > videobuf (I did a quick code analysis). But I don't know if it works > everywhere. > > I would like to get the opinion of others before I implement such a check. But > frankly, I think this may be our best bet. > > So the patch below would look like this if I add the check: > > - mutex_lock(&v4l2_ioctl_mutex); > + if (cmd != VIDIOC_DQBUF) > + mutex_lock(m); > if (video_is_registered(vdev)) > ret = vdev->fops->ioctl(filp, cmd, arg); > - mutex_unlock(&v4l2_ioctl_mutex); > + if (cmd != VIDIOC_DQBUF) > + mutex_unlock(m); > I was thinking of this as well, but didn't bring it up because I considered it too hacky. The patch you posted looks good, thanks for bringing up the problem with my patch and the solution! Acked-by: Arnd Bergmann <a...@arndb.de> -- 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
