Cover Letter Detail
Show a cover letter.
GET /api/covers/20874/?format=api
{ "id": 20874, "url": "https://patchwork.libcamera.org/api/covers/20874/?format=api", "web_url": "https://patchwork.libcamera.org/cover/20874/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/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": "<20240812115009.946036-1-mzamazal@redhat.com>", "date": "2024-08-12T11:49:49", "name": "[00/16] Software ISP: Share params and stats buffers", "submitter": { "id": 177, "url": "https://patchwork.libcamera.org/api/people/177/?format=api", "name": "Milan Zamazal", "email": "mzamazal@redhat.com" }, "mbox": "https://patchwork.libcamera.org/cover/20874/mbox/", "series": [ { "id": 4511, "url": "https://patchwork.libcamera.org/api/series/4511/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4511", "date": "2024-08-12T11:49:49", "name": "Software ISP: Share params and stats buffers", "version": 1, "mbox": "https://patchwork.libcamera.org/series/4511/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/20874/comments/", "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 787FBC323E\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 12 Aug 2024 11:50:21 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 53F8C633B5;\n\tMon, 12 Aug 2024 13:50:20 +0200 (CEST)", "from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.133.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 23CCA63369\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 12 Aug 2024 13:50:19 +0200 (CEST)", "from mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com\n\t(ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63])\n\tby relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3,\n\tcipher=TLS_AES_256_GCM_SHA384) id us-mta-346-uuAUsb_jOd6jjQaqaBzN8Q-1;\n\tMon, 12 Aug 2024 07:50:16 -0400", "from mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com\n\t(mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com\n\t[10.30.177.15])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\tkey-exchange X25519 server-signature RSA-PSS (2048 bits)\n\tserver-digest SHA256) (No client certificate requested)\n\tby mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTPS\n\tid B8C4C195420E for <libcamera-devel@lists.libcamera.org>;\n\tMon, 12 Aug 2024 11:50:15 +0000 (UTC)", "from nuthatch.redhat.com (unknown [10.45.225.57])\n\tby mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTP id 7C1CD19772C4; Mon, 12 Aug 2024 11:50:14 +0000 (UTC)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"eZtvnM+A\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1723463418;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tcontent-transfer-encoding:content-transfer-encoding;\n\tbh=kUcY1hoB0OinqkFGYvthoBa8AFGVPy0zkTDEaj7Vxzo=;\n\tb=eZtvnM+AKfNvlnIeu0lMjv8kFX0UpWuUCanK7TT9Yt/iFIVA93xO+azaw1RoG0zhkVbzLb\n\t8NHDqZ+lsoTcQJ8gb3CMrWJWr1tG3BHmaDGiR4zr3jRSM8Td4jB9m0HOdVBfpq0/rhLzaY\n\tidkEv1jjcXGMQFZphmzn1MGIkf/sUZ4=", "X-MC-Unique": "uuAUsb_jOd6jjQaqaBzN8Q-1", "From": "Milan Zamazal <mzamazal@redhat.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Milan Zamazal <mzamazal@redhat.com>", "Subject": "[PATCH 00/16] Software ISP: Share params and stats buffers", "Date": "Mon, 12 Aug 2024 13:49:49 +0200", "Message-ID": "<20240812115009.946036-1-mzamazal@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.0 on 10.30.177.15", "X-Mimecast-Spam-Score": "0", "X-Mimecast-Originator": "redhat.com", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "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>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "This patch series implements parameters and statistics buffer sharing in\nsoftware ISP. The buffers are currently shared only partially and they\nare copied sometimes. This is unlikely to be a performance problem, at\nleast with the current CPU implementation, as the buffers are not large,\nespecially when compared to the image data being processed. But it\nwould be more elegant and consistent if the buffers were used in a way\nsimilar to the hardware pipelines, i.e. by having a ring of shared\nreusable buffers and passing the buffer ids everywhere.\n\nWe also have the following software ISP TODOs:\n\n- 2. Reconsider stats sharing\n- 5. Store ISP parameters in per-frame buffers\n\nImplementation of this change is perhaps more complicated than initially expected. The first part of the patches deals with parameters buffers, the next part with statistics buffers. The whole patch series is quite long but the patches are split in the way to make them as simple as possible.\n\nThis patch series is a continuation of the “Software ISP refactoring” patch series, which is currently in review.\n\nMilan Zamazal (16):\n libcamera: software_isp: Remove initializer_list include\n libcamera: software_isp: Introduce arguments for parameters buffers\n libcamera: software_isp: Separate allocation of the parameters buffer\n libcamera: software_isp: Track unused parameters buffers\n libcamera: software_isp: Allocation of multiple params buffers\n libcamera: software_isp: Allocate multiple parameters buffers\n libcamera: software_isp: Use multiple parameters buffers in IPA\n libcamera: software_isp: Share parameters buffers with debayering\n libcamera: software_isp: Avoid copying particular color tables\n libcamera: software_isp: Remove per-frame params buffers TODO item\n libcamera: software_isp: Drop a comment about DelayedControls\n libcamera: software_isp: Introduce arguments for statistics buffers\n libcamera: software_isp: Allocate statistics buffers\n libcamera: software_isp: Track statistics buffers\n libcamera: software_isp: Share statistics buffers with IPA\n libcamera: software_isp: Remove stats-sharing TODO item\n\n .../internal/software_isp/software_isp.h | 21 ++-\n include/libcamera/ipa/soft.mojom | 11 +-\n src/ipa/simple/soft_simple.cpp | 80 ++++-----\n src/libcamera/pipeline/simple/simple.cpp | 14 +-\n src/libcamera/software_isp/TODO | 45 -----\n src/libcamera/software_isp/debayer.cpp | 10 +-\n src/libcamera/software_isp/debayer.h | 5 +-\n src/libcamera/software_isp/debayer_cpu.cpp | 97 ++++++-----\n src/libcamera/software_isp/debayer_cpu.h | 23 ++-\n src/libcamera/software_isp/software_isp.cpp | 158 ++++++++++++++----\n src/libcamera/software_isp/swstats_cpu.cpp | 55 +++---\n src/libcamera/software_isp/swstats_cpu.h | 21 ++-\n 12 files changed, 307 insertions(+), 233 deletions(-)" }