PawletOS/external_libcamera
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

libcamera: pipeline: simple: converter: Decouple input and output completion

Browse Source 
The SimpleConverter API signals completion of input and output buffer
pairs. This unnecessarily delays requeueing the input buffer to the
video capture queue until the output buffer completes, and also delays
signalling request completion until the input buffer completes. While
this shouldn't cause large delays in practice, it will also not scale
when multi-stream support will be added to the converter class.

To address the current issue and prepare for the future, decouple
signalling of input and output buffers completion.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Tested-by: Phi-Bang Nguyen <pnguyen@baylibre.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
This commit is contained in:
Laurent Pinchart
2020-12-26 23:45:04 +02:00
parent fb8c63d69c
commit be50270b7d
3 changed files with 26 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/libcamera/pipeline/simple/simple.cpp
View File

@@ -144,7 +144,8 @@ private:
}
void bufferReady(FrameBuffer *buffer);
void converterDone(FrameBuffer *input, FrameBuffer *output);
void converterInputDone(FrameBuffer *buffer);
void converterOutputDone(FrameBuffer *buffer);
MediaDevice *media_;
std::map<const MediaEntity *, std::unique_ptr<V4L2VideoDevice>> videos_;
@@ -768,7 +769,8 @@ bool SimplePipelineHandler::match(DeviceEnumerator *enumerator)
<< "Failed to create converter, disabling format conversion";
converter_.reset();
} else {
converter_->bufferReady.connect(this, &SimplePipelineHandler::converterDone);
converter_->inputBufferReady.connect(this, &SimplePipelineHandler::converterInputDone);
converter_->outputBufferReady.connect(this, &SimplePipelineHandler::converterOutputDone);
}
}
@@ -925,19 +927,23 @@ void SimplePipelineHandler::bufferReady(FrameBuffer *buffer)
completeRequest(request);
}
void SimplePipelineHandler::converterDone(FrameBuffer *input,
FrameBuffer *output)
void SimplePipelineHandler::converterInputDone(FrameBuffer *buffer)
{
ASSERT(activeCamera_);
SimpleCameraData *data = cameraData(activeCamera_);
/* Complete the request. */
Request *request = output->request();
completeBuffer(request, output);
completeRequest(request);
/* Queue the input buffer back for capture. */
data->video_->queueBuffer(input);
data->video_->queueBuffer(buffer);
}
void SimplePipelineHandler::converterOutputDone(FrameBuffer *buffer)
{
ASSERT(activeCamera_);
/* Complete the request. */
Request *request = buffer->request();
completeBuffer(request, buffer);
completeRequest(request);
}
REGISTER_PIPELINE_HANDLER(SimplePipelineHandler)