Commit 47bd4bc1 authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

[media] v4l2-dev.h: add comment not to use V4L2_FL_LOCK_ALL_FOPS in new drivers

This flag is for legacy drivers only and will go away in the future.

A note regarding commit 5126f259
(v4l2-dev: add flag to have the core lock all file operations):

That commit message suggests that by not taking the core lock for fops
other than unlocked_ioctl all problems relating to AB-BA locking and
mm->mmap_sem are solved. This is not the case.

More work needs to be done by moving the core lock further down into
video_ioctl2. It should only be taken after the copy_from/to_user calls
are done.
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 74f22c48
...@@ -40,7 +40,7 @@ struct v4l2_ctrl_handler; ...@@ -40,7 +40,7 @@ struct v4l2_ctrl_handler;
/* Use the prio field of v4l2_fh for core priority checking */ /* Use the prio field of v4l2_fh for core priority checking */
#define V4L2_FL_USE_FH_PRIO (2) #define V4L2_FL_USE_FH_PRIO (2)
/* If ioctl core locking is in use, then apply that also to all /* If ioctl core locking is in use, then apply that also to all
file operations. */ file operations. Don't use this flag in new drivers! */
#define V4L2_FL_LOCK_ALL_FOPS (3) #define V4L2_FL_LOCK_ALL_FOPS (3)
/* Priority helper functions */ /* Priority helper functions */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment