Patch Detail
Show a patch.
GET /api/1.1/patches/8444/?format=api
{ "id": 8444, "url": "https://patchwork.libcamera.org/api/1.1/patches/8444/?format=api", "web_url": "https://patchwork.libcamera.org/patch/8444/", "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": "<20200627030043.2088585-3-niklas.soderlund@ragnatech.se>", "date": "2020-06-27T03:00:32", "name": "[libcamera-devel,02/13] libcamera: ipu3: Import instead of allocate statistic buffers", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "285c39efd63e71c32a6c8a2defe63726a061cae5", "submitter": { "id": 5, "url": "https://patchwork.libcamera.org/api/1.1/people/5/?format=api", "name": "Niklas Söderlund", "email": "niklas.soderlund@ragnatech.se" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/8444/mbox/", "series": [ { "id": 1046, "url": "https://patchwork.libcamera.org/api/1.1/series/1046/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=1046", "date": "2020-06-27T03:00:30", "name": "libcamera: ipu3: Refactoring of ImgU", "version": 1, "mbox": "https://patchwork.libcamera.org/series/1046/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/8444/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/8444/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 8D3AFC2E66\n\tfor <parsemail@patchwork.libcamera.org>;\n\tSat, 27 Jun 2020 03:00:53 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6831560AF2;\n\tSat, 27 Jun 2020 05:00:53 +0200 (CEST)", "from bin-mail-out-05.binero.net (bin-mail-out-05.binero.net\n\t[195.74.38.228])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 0ACF1609D6\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSat, 27 Jun 2020 05:00:50 +0200 (CEST)", "from bismarck.berto.se (p4fca2eca.dip0.t-ipconnect.de\n\t[79.202.46.202]) by bin-vsp-out-03.atm.binero.net (Halon) with ESMTPA\n\tid 53f3bd94-b822-11ea-86ee-0050569116f7;\n\tSat, 27 Jun 2020 05:00:16 +0200 (CEST)" ], "X-Halon-ID": "53f3bd94-b822-11ea-86ee-0050569116f7", "Authorized-sender": "niklas@soderlund.pp.se", "From": "=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Sat, 27 Jun 2020 05:00:32 +0200", "Message-Id": "<20200627030043.2088585-3-niklas.soderlund@ragnatech.se>", "X-Mailer": "git-send-email 2.27.0", "In-Reply-To": "<20200627030043.2088585-1-niklas.soderlund@ragnatech.se>", "References": "<20200627030043.2088585-1-niklas.soderlund@ragnatech.se>", "MIME-Version": "1.0", "Subject": "[libcamera-devel] [PATCH 02/13] libcamera: ipu3: Import instead of\n\tallocate statistic buffers", "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>", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "base64", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Statistics buffers are not yet used by the IPU3 pipeline, they are never\nqueued to the video device or in any other way consumed. The kernel\ndriver will however not allow video streaming to start if buffers are\nnot either allocated or imported on the video device. Instead of\nallocating the buffers wasting memory that is never used import buffers.\n\nSigned-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>\n---\n src/libcamera/pipeline/ipu3/ipu3.cpp | 8 +++++---\n 1 file changed, 5 insertions(+), 3 deletions(-)", "diff": "diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp\nindex fba45935741e0e4e..405550b1302fb370 100644\n--- a/src/libcamera/pipeline/ipu3/ipu3.cpp\n+++ b/src/libcamera/pipeline/ipu3/ipu3.cpp\n@@ -1135,11 +1135,13 @@ int ImgUDevice::allocateBuffers(IPU3CameraData *data, unsigned int bufferCount)\n \t}\n \n \t/*\n-\t * Use for the stat's internal pool the same number of buffers as for\n-\t * the input pool.\n+\t * The kernel fails to start if buffers are not either imported or\n+\t * allocated for the statisitcs video device. As statistics buffers are\n+\t * not yet used by the pipeline import buffers to save memory.\n+\t *\n \t * \\todo To be revised when we'll actually use the stat node.\n \t */\n-\tret = stat_.dev->allocateBuffers(bufferCount, &stat_.buffers);\n+\tret = stat_.dev->importBuffers(bufferCount);\n \tif (ret < 0) {\n \t\tLOG(IPU3, Error) << \"Failed to allocate ImgU stat buffers\";\n \t\tgoto error;\n", "prefixes": [ "libcamera-devel", "02/13" ] }