v4l2: v4l2_compat: Support multiple open

Previously, since we acquired the libcamera camera upon open(), it was
impossible to support multiple open, as any subsequent opens would
return error because the camera would already be acquired.

To fix this, we first initialize the camera in the first call to
V4L2CameraProxy::open(), just to heat up the stream format cache. We
then add ownership by a V4L2CameraFile of a V4L2Camera via the
V4L2CameraProxy. All vidioc ioctls prior to reqbufs > 0 (except for
s_fmt) are able to access the camera without ownership. A call to
reqbufs > 0 (and s_fmt) will take ownership, and the ownership will be
released at reqbufs = 0. While ownership is assigned, the eventfd that
should be signaled (and cleared) by V4L2Camera and V4L2CameraProxy is
set to the V4L2CameraFile that has ownership, and is cleared when the
ownership is released. In case close() is called without a
reqbufs = 0 first, the ownership is also released on close().

We also use the V4L2CameraFile to contain all the information specific
to an open instance of the file. This removes the need to keep track of
such information within V4L2CameraProxy via multiple maps from int fd to
info.

Since V4L2 does not expect reqbufs 0 to ever return error, make
V4L2CameraProxy::freeBuffers() return void.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

1 files changed, 15 insertions, 3 deletions

diff --git a/src/v4l2/v4l2_camera_proxy.h b/src/v4l2/v4l2_camera_proxy.h
index b2197ef8..36d1dbc8 100644
--- a/src/v4l2/v4l2_camera_proxy.h
+++ b/src/v4l2/v4l2_camera_proxy.h
@@ -33,7 +33,6 @@ public:
 	void *mmap(void *addr, size_t length, int prot, int flags, off64_t offset);
 	int munmap(void *addr, size_t length);
 
-	void bind(int fd);
 	int ioctl(V4L2CameraFile *file, unsigned long request, void *arg);
 
 private:
@@ -44,7 +43,7 @@ private:
 	void querycap(std::shared_ptr<Camera> camera);
 	void tryFormat(struct v4l2_format *arg);
 	void updateBuffers();
-	int freeBuffers();
+	void freeBuffers();
 
 	int vidioc_querycap(struct v4l2_capability *arg);
 	int vidioc_enum_fmt(V4L2CameraFile *file, struct v4l2_fmtdesc *arg);
@@ -58,6 +57,10 @@ private:
 	int vidioc_streamon(V4L2CameraFile *file, int *arg);
 	int vidioc_streamoff(V4L2CameraFile *file, int *arg);
 
+	bool hasOwnership(V4L2CameraFile *file);
+	int acquire(V4L2CameraFile *file);
+	void release(V4L2CameraFile *file);
+
 	static unsigned int bplMultiplier(uint32_t format);
 	static unsigned int imageSize(uint32_t format, unsigned int width,
 				      unsigned int height);
@@ -80,7 +83,16 @@ private:
 
 	std::unique_ptr<V4L2Camera> vcam_;
 
-	int efd_;
+	/*
+	 * This is the exclusive owner of this V4L2CameraProxy instance.
+	 * When there is no owner, anybody can call any ioctl before reqbufs.
+	 * The first file to call reqbufs with count > 0 or s_fmt will become
+	 * the owner, and when the owner calls reqbufs with count = 0 it will
+	 * release ownership. Any buffer-related ioctl (except querybuf) or
+	 * s_fmt that is called by a non-owner while there exists an owner
+	 * will return -EBUSY.
+	 */
+	V4L2CameraFile *owner_;
 };
 
 #endif /* __V4L2_CAMERA_PROXY_H__ */