Patch Detail
Show a patch.
GET /api/patches/24309/?format=api
{ "id": 24309, "url": "https://patchwork.libcamera.org/api/patches/24309/?format=api", "web_url": "https://patchwork.libcamera.org/patch/24309/", "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": "<20250911092945.16517-3-mzamazal@redhat.com>", "date": "2025-09-11T09:29:32", "name": "[v17,02/12] libcamera: camera_manager: Construct GlobalConfiguration instance", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "971a438b44418b8e80dcff7143dc8ecba8ebec5f", "submitter": { "id": 177, "url": "https://patchwork.libcamera.org/api/people/177/?format=api", "name": "Milan Zamazal", "email": "mzamazal@redhat.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/24309/mbox/", "series": [ { "id": 5431, "url": "https://patchwork.libcamera.org/api/series/5431/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5431", "date": "2025-09-11T09:29:30", "name": "Add global configuration file", "version": 17, "mbox": "https://patchwork.libcamera.org/series/5431/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/24309/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/24309/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 C674BC328C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 11 Sep 2025 09:30:06 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id AD6CB69371;\n\tThu, 11 Sep 2025 11:30:05 +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 57BD769369\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 11 Sep 2025 11:30:03 +0200 (CEST)", "from mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com\n\t(ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97])\n\tby relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3,\n\tcipher=TLS_AES_256_GCM_SHA384) id us-mta-594-DrG9k2cUM8efsj9GLmXNQA-1;\n\tThu, 11 Sep 2025 05:30:00 -0400", "from mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com\n\t(mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com\n\t[10.30.177.12])\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-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTPS id A98DA1800343; Thu, 11 Sep 2025 09:29:59 +0000 (UTC)", "from mzamazal-thinkpadp1gen7.tpbc.com (unknown [10.44.34.98])\n\tby mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTP id 2229119560B1; Thu, 11 Sep 2025 09:29:56 +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=\"OEfu8IBe\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1757583002;\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\tin-reply-to:in-reply-to:references:references;\n\tbh=GBHXZPfFtSB+MH/uAL2AtkFIccHTCqbrnpNQFVW5BM4=;\n\tb=OEfu8IBetjURDv+WWjcUDU6oSJZ2C0s7hZVSFKNEqWKLsKNP+yfX0zoJKQ4XjunDQlzou/\n\t/+QY70p5zFelL1fAhK9uP8NiMw36apw9QfnUsG0sN0+utcgx42rv+kFY8oj44HFyP+JgWW\n\tUsl0l7AsjpUWxHeeEbKmW/H1VwSbjM8=", "X-MC-Unique": "DrG9k2cUM8efsj9GLmXNQA-1", "X-Mimecast-MFC-AGG-ID": "DrG9k2cUM8efsj9GLmXNQA_1757582999", "From": "Milan Zamazal <mzamazal@redhat.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Milan Zamazal <mzamazal@redhat.com>, Kieran Bingham\n\t<kieran.bingham@ideasonboard.com>, =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?=\n\t<barnabas.pocze@ideasonboard.com>, Laurent Pinchart\n\t<laurent.pinchart@ideasonboard.com>, Paul Elder\n\t<paul.elder@ideasonboard.com>", "Subject": "[PATCH v17 02/12] libcamera: camera_manager: Construct\n\tGlobalConfiguration instance", "Date": "Thu, 11 Sep 2025 11:29:32 +0200", "Message-ID": "<20250911092945.16517-3-mzamazal@redhat.com>", "In-Reply-To": "<20250911092945.16517-1-mzamazal@redhat.com>", "References": "<20250911092945.16517-1-mzamazal@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.0 on 10.30.177.12", "X-Mimecast-Spam-Score": "0", "X-Mimecast-MFC-PROC-ID": "JvOwDGARvPLlSKE3VEvChTb9CGGBQMprMh4HbRawkNM_1757582999", "X-Mimecast-Originator": "redhat.com", "Content-Transfer-Encoding": "8bit", "content-type": "text/plain; charset=\"US-ASCII\"; x-default=true", "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": "Global configuration is accessed via a GlobalConfiguration instance.\nThe instance is conceptually a singleton, but singletons are not welcome\nin libcamera so we must store the (preferably single) instance\nsomewhere.\n\nThis patch creates a GlobalConfiguration instance in CameraManager and\ndefines the corresponding access method. CameraManager is typically\ninstantiated only once or a few times, it is accessible in many places\nin libcamera and the configuration can be retrieved from it and passed\nto other places if needed (it's read-only once created). Using\nCameraManager for the purpose is still suboptimal and we use it only due\nto lack of better options. An alternative could be Logger, which is\nstill a singleton and it's accessible from everywhere. But with Logger,\nwe have a chicken and egg problem -- GlobalConfiguration should log\ncontingent problems with the configuration when it's loaded but if it is\ncreated in the logger then there are mutual infinite recursive calls.\nOne possible way to deal with this is to look at the environment\nvariables only during logging initialisation and apply the logging\nconfiguration when a CameraManager is constructed. Considering there\nare intentions to remove the Logger singleton, let's omit logging\nconfiguration for now.\n\nIf there are multiple CameraManager instances, there are also multiple\nGlobalConfiguration instances, each CameraManager instance is meant to\nbe fully independent, including configuration. They may or may not\ncontain the same data, depending on whether the global configuration\nfile in the file system was changed in the meantime.\n\nThe configuration is stored in the private CameraManager. It's\naccessible within libcamera (via CameraManager) but it's not meant to be\naccessed by applications.\n\nSigned-off-by: Milan Zamazal <mzamazal@redhat.com>\nReviewed-by: Paul Elder <paul.elder@ideasonboard.com>\nSigned-off-by: Milan Zamazal <mzamazal@redhat.com>\n---\n include/libcamera/internal/camera_manager.h | 12 ++++++++++--\n src/libcamera/camera_manager.cpp | 8 ++++++++\n 2 files changed, 18 insertions(+), 2 deletions(-)", "diff": "diff --git a/include/libcamera/internal/camera_manager.h b/include/libcamera/internal/camera_manager.h\nindex 5dfbe1f65..b8b2966b5 100644\n--- a/include/libcamera/internal/camera_manager.h\n+++ b/include/libcamera/internal/camera_manager.h\n@@ -7,8 +7,6 @@\n \n #pragma once\n \n-#include <libcamera/camera_manager.h>\n-\n #include <memory>\n #include <sys/types.h>\n #include <vector>\n@@ -18,6 +16,9 @@\n #include <libcamera/base/thread.h>\n #include <libcamera/base/thread_annotations.h>\n \n+#include <libcamera/camera_manager.h>\n+\n+#include \"libcamera/internal/global_configuration.h\"\n #include \"libcamera/internal/process.h\"\n \n namespace libcamera {\n@@ -38,6 +39,11 @@ public:\n \tvoid addCamera(std::shared_ptr<Camera> camera) LIBCAMERA_TSA_EXCLUDES(mutex_);\n \tvoid removeCamera(std::shared_ptr<Camera> camera) LIBCAMERA_TSA_EXCLUDES(mutex_);\n \n+\tconst GlobalConfiguration &configuration() const\n+\t{\n+\t\treturn configuration_;\n+\t}\n+\n \tIPAManager *ipaManager() const { return ipaManager_.get(); }\n \n protected:\n@@ -65,6 +71,8 @@ private:\n \tstd::unique_ptr<DeviceEnumerator> enumerator_;\n \n \tstd::unique_ptr<IPAManager> ipaManager_;\n+\n+\tconst GlobalConfiguration configuration_;\n };\n \n } /* namespace libcamera */\ndiff --git a/src/libcamera/camera_manager.cpp b/src/libcamera/camera_manager.cpp\nindex f81794bfd..dca3d9a83 100644\n--- a/src/libcamera/camera_manager.cpp\n+++ b/src/libcamera/camera_manager.cpp\n@@ -15,6 +15,7 @@\n \n #include \"libcamera/internal/camera.h\"\n #include \"libcamera/internal/device_enumerator.h\"\n+#include \"libcamera/internal/global_configuration.h\"\n #include \"libcamera/internal/ipa_manager.h\"\n #include \"libcamera/internal/pipeline_handler.h\"\n \n@@ -258,6 +259,13 @@ void CameraManager::Private::removeCamera(std::shared_ptr<Camera> camera)\n \to->cameraRemoved.emit(camera);\n }\n \n+/**\n+ * \\fn const GlobalConfiguration &CameraManager::Private::configuration() const\n+ * \\brief Get global configuration bound to the camera manager\n+ *\n+ * \\return Reference to the configuration\n+ */\n+\n /**\n * \\fn CameraManager::Private::ipaManager() const\n * \\brief Retrieve the IPAManager\n", "prefixes": [ "v17", "02/12" ] }