{"id":12265,"url":"https://patchwork.libcamera.org/api/1.1/patches/12265/?format=json","web_url":"https://patchwork.libcamera.org/patch/12265/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20210513092246.42847-4-jacopo@jmondi.org>","date":"2021-05-13T09:22:41","name":"[libcamera-devel,v2,3/8] libcamera: pipeline_handler: Notify Request queueing failure","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"79c0b914536624ba4c1cef76125c56c2bb09c726","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/1.1/people/3/?format=json","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"delegate":{"id":15,"url":"https://patchwork.libcamera.org/api/1.1/users/15/?format=json","username":"jmondi","first_name":"Jacopo","last_name":"Mondi","email":"jacopo@jmondi.org"},"mbox":"https://patchwork.libcamera.org/patch/12265/mbox/","series":[{"id":2025,"url":"https://patchwork.libcamera.org/api/1.1/series/2025/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=2025","date":"2021-05-13T09:22:38","name":"Implement flush() camera operation","version":2,"mbox":"https://patchwork.libcamera.org/series/2025/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/12265/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/12265/checks/","tags":{},"headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 4F0FAC31EE\n\tfor ;\n\tThu, 13 May 2021 09:22:12 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 1733D6891F;\n\tThu, 13 May 2021 11:22:12 +0200 (CEST)","from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net\n\t[217.70.183.193])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9D69F68919\n\tfor ;\n\tThu, 13 May 2021 11:22:08 +0200 (CEST)","from uno.LocalDomain (93-61-96-190.ip145.fastwebnet.it\n\t[93.61.96.190]) (Authenticated sender: jacopo@jmondi.org)\n\tby relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 29AA1240006;\n\tThu, 13 May 2021 09:22:07 +0000 (UTC)"],"From":"Jacopo Mondi ","To":"libcamera-devel@lists.libcamera.org","Date":"Thu, 13 May 2021 11:22:41 +0200","Message-Id":"<20210513092246.42847-4-jacopo@jmondi.org>","X-Mailer":"git-send-email 2.31.1","In-Reply-To":"<20210513092246.42847-1-jacopo@jmondi.org>","References":"<20210513092246.42847-1-jacopo@jmondi.org>","MIME-Version":"1.0","Subject":"[libcamera-devel] [PATCH v2 3/8] libcamera: pipeline_handler:\n\tNotify Request queueing failure","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"Capture requests are queued by the PipelineHandler base class to each\npipeline handler implementation using the virtual queueRequestDevice()\nfunction.\n\nHowever, if the pipeline handler fails to queue the request to the\nhardware, the request gets silently deleted from the list of queued\nones, without notifying application of the error.\n\nAllowing applications to know if a Request has failed to queue\nby emitting the Camera::bufferCompleted and Camera::requestCompleted\nsignals allows them to maintain a state consistent with the one\ninternal to the library.\n\nTo do so, if queuing a request to the device fails, cancel the request\nand emit the completion signal to report the failure to applications.\n\nSigned-off-by: Jacopo Mondi \n---\n src/libcamera/pipeline_handler.cpp | 15 ++++++++++++++-\n 1 file changed, 14 insertions(+), 1 deletion(-)","diff":"diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp\nindex f41b7a7b3308..260a52018a4d 100644\n--- a/src/libcamera/pipeline_handler.cpp\n+++ b/src/libcamera/pipeline_handler.cpp\n@@ -391,6 +391,8 @@ bool PipelineHandler::hasPendingRequests(const Camera *camera) const\n * This method queues a capture request to the pipeline handler for processing.\n * The request is first added to the internal list of queued requests, and\n * then passed to the pipeline handler with a call to queueRequestDevice().\n+ * If the pipeline handler fails in queuing the request to the hardware the\n+ * request is immediately completed with error.\n *\n * Keeping track of queued requests ensures automatic completion of all requests\n * when the pipeline handler is stopped with stop(). Request completion shall be\n@@ -409,8 +411,19 @@ void PipelineHandler::queueRequest(Request *request)\n \trequest->sequence_ = data->requestSequence_++;\n \n \tint ret = queueRequestDevice(camera, request);\n-\tif (ret)\n+\tif (ret) {\n+\t\t/*\n+\t\t * Cancel the request and notify completion of its buffers in\n+\t\t * error state. Then complete the cancelled request and remove\n+\t\t * it from the queued requests list.\n+\t\t */\n+\t\trequest->cancel();\n+\t\tfor (auto bufferMap : request->buffers())\n+\t\t\tcamera->bufferCompleted.emit(request, bufferMap.second);\n+\n+\t\tcamera->requestComplete(request);\n \t\tdata->queuedRequests_.remove(request);\n+\t}\n }\n \n /**\n","prefixes":["libcamera-devel","v2","3/8"]}