Patch Detail
Show a patch.
GET /api/1.1/patches/19339/?format=api
{ "id": 19339, "url": "https://patchwork.libcamera.org/api/1.1/patches/19339/?format=api", "web_url": "https://patchwork.libcamera.org/patch/19339/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20231222164036.649245-3-jacopo.mondi@ideasonboard.com>", "date": "2023-12-22T16:40:33", "name": "[libcamera-devel,2/5] apps: lc-compliance: Check that requests complete successfully", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "cadcafb1751b2e25190232d7f70bff1ac61e6afb", "submitter": { "id": 143, "url": "https://patchwork.libcamera.org/api/1.1/people/143/?format=api", "name": "Jacopo Mondi", "email": "jacopo.mondi@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/19339/mbox/", "series": [ { "id": 4125, "url": "https://patchwork.libcamera.org/api/1.1/series/4125/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4125", "date": "2023-12-22T16:40:31", "name": "apps: lc-compliance: Add controls and properties test", "version": 1, "mbox": "https://patchwork.libcamera.org/series/4125/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/19339/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/19339/checks/", "tags": {}, "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "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 BD50DC31E9\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 22 Dec 2023 16:40:54 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D3C0462B31;\n\tFri, 22 Dec 2023 17:40:53 +0100 (CET)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C117361D8F\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 22 Dec 2023 17:40:49 +0100 (CET)", "from localhost.localdomain (93-61-96-190.ip145.fastwebnet.it\n\t[93.61.96.190])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id DB645BB2;\n\tFri, 22 Dec 2023 17:39:57 +0100 (CET)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1703263253;\n\tbh=qneqtV9quvOjU67kpSxwhaJzfrt4bucT1eOPM/MgTvQ=;\n\th=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:\n\tFrom;\n\tb=4LMUBt6QkyVVl6RyDyNt6wB698ueMKXZPP/SBzd0MJXbhMoWy1s1hNtidaGu2qBM/\n\tBSBTQgA1wQv8HeXS+U81vf/5W1XNafJ+ZEI4+lP4ORfbwgueTfi5cV3vXsrlP/E+qz\n\ti8L/orxu2IELszwnFSEGmxTVzDbla2YItCtmhK3JljX3sY85xbuJF8AgWX0QFz866O\n\t+I3MKRe8Bt5CYSRpkK2EtIg9cic80FNwtEhfwrM/F4TWhZhzOxCN5qSm8zVEVIBJnZ\n\tcKMhu0IAJXg04bTYMCPLFVGzaI3hC3Iyh7xk0qPDLPn6sitlKh6iO1FY14jHM+KvPN\n\tXmi7O6DELqjFg==", "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1703263198;\n\tbh=qneqtV9quvOjU67kpSxwhaJzfrt4bucT1eOPM/MgTvQ=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=wErTc24o8k/zP/1AsOUaAEPpNa5rjqA+LsA+vx6P/PDrWN6Cr7PZlFClO8iVYxahH\n\tMcUchmxiSP8hYbcpYanf60F9zFDkh6dUs+z4Bq7UkBiZfKIWkogH2MepIRbldCRkov\n\th7a6zvGn44FpWgxI0zmhX8cs7VYLD9gFBJUyKhUE=" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"wErTc24o\"; dkim-atps=neutral", "To": "libcamera-devel@lists.libcamera.org", "Date": "Fri, 22 Dec 2023 17:40:33 +0100", "Message-ID": "<20231222164036.649245-3-jacopo.mondi@ideasonboard.com>", "X-Mailer": "git-send-email 2.41.0", "In-Reply-To": "<20231222164036.649245-1-jacopo.mondi@ideasonboard.com>", "References": "<20231222164036.649245-1-jacopo.mondi@ideasonboard.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH 2/5] apps: lc-compliance: Check that\n\trequests complete successfully", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "From": "Jacopo Mondi via libcamera-devel <libcamera-devel@lists.libcamera.org>", "Reply-To": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "From: Nícolas F. R. A. Prado <nfraprado@collabora.com>\n\nWhen a request fails to queue it is completed but with its status set to\nRequestCancelled. Add a check in the requestComplete callback to make\nsure that the request was completed successfully.\n\nFor the SimpleCaptureUnbalanced test we need to do this check only if\nthe capture isn't over yet, otherwise the few extra requests that get\ncancelled at the end, which is the normal behavior, will make the test\nfail.\n\nSigned-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>\nReviewed-by: Paul Elder <paul.elder@ideasonboard.com>\nSigned-off-by: Paul Elder <paul.elder@ideasonboard.com>\n---\n src/apps/lc-compliance/simple_capture.cpp | 6 ++++++\n 1 file changed, 6 insertions(+)", "diff": "diff --git a/src/apps/lc-compliance/simple_capture.cpp b/src/apps/lc-compliance/simple_capture.cpp\nindex cf4d7cf38bc3..e22a339c4e1c 100644\n--- a/src/apps/lc-compliance/simple_capture.cpp\n+++ b/src/apps/lc-compliance/simple_capture.cpp\n@@ -127,6 +127,9 @@ int SimpleCaptureBalanced::queueRequest(Request *request)\n \n void SimpleCaptureBalanced::requestComplete(Request *request)\n {\n+\tEXPECT_EQ(request->status(), Request::Status::RequestComplete)\n+\t\t<< \"Request didn't complete successfully\";\n+\n \tcaptureCount_++;\n \tif (captureCount_ >= captureLimit_) {\n \t\tloop_->exit(0);\n@@ -184,6 +187,9 @@ void SimpleCaptureUnbalanced::requestComplete(Request *request)\n \t\treturn;\n \t}\n \n+\tEXPECT_EQ(request->status(), Request::Status::RequestComplete)\n+\t\t<< \"Request didn't complete successfully\";\n+\n \trequest->reuse(Request::ReuseBuffers);\n \tif (camera_->queueRequest(request))\n \t\tloop_->exit(-EINVAL);\n", "prefixes": [ "libcamera-devel", "2/5" ] }