@@ -17,6 +17,7 @@
#include <libcamera/base/signal.h>
#include <libcamera/controls.h>
+#include <libcamera/fence.h>
namespace libcamera {
@@ -51,7 +52,8 @@ public:
ControlList &controls() { return *controls_; }
ControlList &metadata() { return *metadata_; }
const BufferMap &buffers() const { return bufferMap_; }
- int addBuffer(const Stream *stream, FrameBuffer *buffer);
+ int addBuffer(const Stream *stream, FrameBuffer *buffer,
+ std::unique_ptr<Fence> fence = nullptr);
FrameBuffer *findBuffer(const Stream *stream) const;
uint32_t sequence() const;
@@ -14,6 +14,7 @@
#include <libcamera/camera.h>
#include <libcamera/control_ids.h>
+#include <libcamera/fence.h>
#include <libcamera/framebuffer.h>
#include <libcamera/stream.h>
@@ -294,6 +295,7 @@ void Request::reuse(ReuseFlag flags)
* \brief Add a FrameBuffer with its associated Stream to the Request
* \param[in] stream The stream the buffer belongs to
* \param[in] buffer The FrameBuffer to add to the request
+ * \param[in] fence The optional synchronization Fence
*
* A reference to the buffer is stored in the request. The caller is responsible
* for ensuring that the buffer will remain valid until the request complete
@@ -302,11 +304,24 @@ void Request::reuse(ReuseFlag flags)
* A request can only contain one buffer per stream. If a buffer has already
* been added to the request for the same stream, this function returns -EEXIST.
*
+ * A synchronization Fence can be optionally associated with the \a buffer.
+ *
+ * When a valid fence is provided to this function, \a fence is moved to \a
+ * buffer and this Request will only be queued to the device once the
+ * synchronization fences of all its buffers have been correctly signalled.
+ *
+ * If the \a fence associated with \a buffer fails, the application is
+ * responsible for resetting it before associating this buffer with a new
+ * Request by calling this function again.
+ *
+ * \sa FrameBuffer::resetFence()
+ *
* \return 0 on success or a negative error code otherwise
* \retval -EEXIST The request already contains a buffer for the stream
* \retval -EINVAL The buffer does not reference a valid Stream
*/
-int Request::addBuffer(const Stream *stream, FrameBuffer *buffer)
+int Request::addBuffer(const Stream *stream, FrameBuffer *buffer,
+ std::unique_ptr<Fence> fence)
{
if (!stream) {
LOG(Request, Error) << "Invalid stream reference";
@@ -323,6 +338,15 @@ int Request::addBuffer(const Stream *stream, FrameBuffer *buffer)
_d()->pending_.insert(buffer);
bufferMap_[stream] = buffer;
+ /*
+ * Make sure the fence has been extracted from the buffer
+ * to avoid waiting on a stale fence.
+ */
+ ASSERT(!buffer->fence());
+
+ if (fence && fence->isValid())
+ buffer->_d()->setFence(std::move(fence));
+
return 0;
}
Add an optional synchronization fence to Request::addBuffer() to allow associating a Fence with a FrameBuffer part of a Request. Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> --- include/libcamera/request.h | 4 +++- src/libcamera/request.cpp | 26 +++++++++++++++++++++++++- 2 files changed, 28 insertions(+), 2 deletions(-)