[v6,02/16] config: Introduce global runtime configuration
diff mbox series

Message ID 20241206171148.1189292-3-mzamazal@redhat.com
State New
Headers show
Series
  • Add global configuration file
Related show

Commit Message

Milan Zamazal Dec. 6, 2024, 5:11 p.m. UTC
Currently, libcamera can be configured in runtime using several
environment variables.  With introducing more and more variables, this
mechanism reaches its limits.  It would be simpler and more flexible if
it was possible to configure libcamera in a single file.

For example, there have been a request for defining pipeline precedence
in runtime.  We want to compile in multiple pipelines, in order to have
them accessible within single packages in distributions.  And then being
able to select among the pipelines manually as needed based on the
particular hardware or operating system environment.  Having the
configuration file then allows easy switching between hardware, GPU or
CPU IPAs.  Another possible use case is tuning image output, especially
with software ISP, to user liking.  For example, some users may prefer
higher contrast without the need to use the corresponding knobs, if
present at all, in every application.  The configuration file can also
be used to enable or disable experimental features and avoid the need to
track local patches changing configuration options hard-wired in the
code when working on new features.

This patch introduces basic support for configuration files.  It reads,
stores and accesses the configuration.  The configuration is meant to
use as a hidden singleton with public access methods.  It is implemented
as GlobalConfiguration namespace exposing the public functions, with
private functions and the configuration data hidden in an anonymous
namespace.  This model suits well the purpose of having a hidden
singleton wrapped by public access functions.

libcamera configuration can be specified using a system-wide
configuration file or a user configuration file.  The user configuration
file takes precedence if present.  There is currently no way to merge
multiple configuration files, the one found is used as the only
configuration file.  If no configuration file is present, nothing
changes to the current libcamera behavior (except for some log
messages related to configuration file lookup).

The configuration file is a YAML file.  We already have a mechanism for
handling YAML configuration files in libcamera and the given
infrastructure can be reused for the purpose.  However, the
configuration type is abstracted to make contingent future change of the
underlying class easier while retaining (most of) the original API.

The configuration is versioned.  This has currently no particular
meaning but is likely to have its purpose in future, especially once
configuration validation is introduced.

The configuration YAML file looks as follows:

  ---
  version: 1
  configuration:
    WHATEVER CONFIGURATION NEEDED

There is no logging about reading the configuration file and contingent
errors.  This is on purpose because logging will query configuration,
which can lead to various problems when done during configuration
initialization.  Reporting the errors will be added later.

A complication arises from the fact that when the configuration is
loaded, logging may be called and logging will ask for the
configuration.  This is error-prone and may lead to subtle problems.
For this reason, the global configuration is instantiated to a pointer,
with an empty configuration initially.  The real configuration will be
created through initialize() method.  It will be clearer how it helps in
the followup patch introducing logging configuration.

Logging is also the most notable component from base that uses global
configuration.  In order to be able to do it, the global configuration
must be put to base.

This patch introduces just the basic idea.  Actually using the
configuration in the corresponding places (everything what is currently
configurable via environment variables should be configurable in the
file configuration) and other enhancements are implemented in the
followup patches.

Signed-off-by: Milan Zamazal <mzamazal@redhat.com>
---
 .../libcamera/internal/global_configuration.h |  25 +++
 include/libcamera/internal/meson.build        |   1 +
 src/libcamera/base/global_configuration.cpp   | 153 ++++++++++++++++++
 src/libcamera/base/meson.build                |   1 +
 4 files changed, 180 insertions(+)
 create mode 100644 include/libcamera/internal/global_configuration.h
 create mode 100644 src/libcamera/base/global_configuration.cpp

Patch
diff mbox series

diff --git a/include/libcamera/internal/global_configuration.h b/include/libcamera/internal/global_configuration.h
new file mode 100644
index 00000000..bed88279
--- /dev/null
+++ b/include/libcamera/internal/global_configuration.h
@@ -0,0 +1,25 @@ 
+/* SPDX-License-Identifier: LGPL-2.1-or-later */
+/*
+ * Copyright (C) 2024 Red Hat, inc.
+ *
+ * Global configuration handling
+ */
+
+#pragma once
+
+#include "libcamera/internal/yaml_parser.h"
+
+namespace libcamera {
+
+namespace GlobalConfiguration {
+
+using Configuration = const YamlObject &;
+
+void initialize();
+
+unsigned int version();
+Configuration configuration();
+
+} /* namespace GlobalConfiguration */
+
+} /* namespace libcamera */
diff --git a/include/libcamera/internal/meson.build b/include/libcamera/internal/meson.build
index 7d6aa8b7..cf1765bf 100644
--- a/include/libcamera/internal/meson.build
+++ b/include/libcamera/internal/meson.build
@@ -22,6 +22,7 @@  libcamera_internal_headers = files([
     'dma_buf_allocator.h',
     'formats.h',
     'framebuffer.h',
+    'global_configuration.h',
     'ipa_data_serializer.h',
     'ipa_manager.h',
     'ipa_module.h',
diff --git a/src/libcamera/base/global_configuration.cpp b/src/libcamera/base/global_configuration.cpp
new file mode 100644
index 00000000..3350a26a
--- /dev/null
+++ b/src/libcamera/base/global_configuration.cpp
@@ -0,0 +1,153 @@ 
+/* SPDX-License-Identifier: LGPL-2.1-or-later */
+/*
+ * Copyright (C) 2024 Red Hat, inc.
+ *
+ * Global configuration handling
+ */
+
+#include "libcamera/internal/global_configuration.h"
+
+#include <filesystem>
+#include <sys/types.h>
+
+#include <libcamera/base/file.h>
+#include <libcamera/base/log.h>
+#include <libcamera/base/utils.h>
+
+#include "libcamera/internal/yaml_parser.h"
+
+namespace libcamera {
+
+namespace GlobalConfiguration {
+
+namespace {
+
+std::unique_ptr<YamlObject> yamlConfiguration = std::make_unique<YamlObject>();
+
+bool loadFile(const std::filesystem::path &fileName)
+{
+	File file(fileName);
+	if (!file.exists()) {
+		return false;
+	}
+
+	if (!file.open(File::OpenModeFlag::ReadOnly))
+		return true;
+
+	auto root = YamlParser::parse(file);
+	if (!root)
+		return true;
+	yamlConfiguration = std::move(root);
+
+	return true;
+}
+
+const std::vector<std::filesystem::path>
+	globalConfigurationFiles = {
+		std::filesystem::path(LIBCAMERA_SYSCONF_DIR) / "configuration.yaml",
+		std::filesystem::path("/etc/libcamera/configuration.yaml"),
+	};
+
+void load()
+{
+	std::filesystem::path userConfigurationDirectory;
+	char *xdgConfigHome = utils::secure_getenv("XDG_CONFIG_HOME");
+	if (xdgConfigHome) {
+		userConfigurationDirectory = xdgConfigHome;
+	} else {
+		const char *home = utils::secure_getenv("HOME");
+		if (home)
+			userConfigurationDirectory =
+				std::filesystem::path(home) / ".config";
+	}
+
+	if (!userConfigurationDirectory.empty()) {
+		std::filesystem::path user_configuration_file =
+			userConfigurationDirectory / "libcamera" / "configuration.yaml";
+		if (loadFile(user_configuration_file))
+			return;
+	}
+
+	for (const auto &path : globalConfigurationFiles)
+		if (loadFile(path))
+			return;
+}
+
+Configuration get()
+{
+	return *yamlConfiguration;
+}
+
+} /* namespace */
+
+/**
+ * \brief Initialize the global configuration
+ *
+ * This must be called before global configuration is accessed.
+ */
+void initialize()
+{
+	load();
+}
+
+/**
+ * \namespace GlobalConfiguration
+ * \brief Support for global libcamera configuration
+ *
+ * The configuration file is a YAML file and the configuration itself is stored
+ * under `configuration' top-level item.
+ *
+ * The configuration file is looked up in user's home directory first and if it
+ * is not found then in system-wide configuration directories. If multiple
+ * configuration files exist then only the first one found is used and no
+ * configuration merging is performed.
+ *
+ * The configuration can be accessed using GlobalConfiguration::configuration().
+ */
+
+/**
+ * \typedef GlobalConfiguration::Configuration
+ * \brief Type representing global libcamera configuration
+ *
+ * All code outside GlobalConfiguration must use this type declaration and not
+ * the underlying type.
+ */
+
+/**
+ * \brief Return configuration version
+ *
+ * The version is (optionally) declared in the configuration file in the
+ * top-level section `version', alongside `configuration'. This has currently no
+ * real use but may be needed in future if configuration incompatibilities
+ * occur.
+ *
+ * \return Configuration version as declared in the configuration file or 0 if
+ * no version is declared there
+ */
+unsigned int version()
+{
+	return get()["version"].get<unsigned int>().value_or(0);
+}
+
+/**
+ * \brief Return libcamera global configuration
+ *
+ * This returns the whole configuration stored in the top-level section
+ * `configuration' of the YAML configuration file.
+ *
+ * The requested part of the configuration can be accessed using \a YamlObject
+ * methods.
+ *
+ * \note \a YamlObject type itself shouldn't be used in type declarations to
+ * avoid trouble if we decide to change the underlying data objects in future.
+ *
+ * \return The whole configuration section
+ */
+Configuration configuration()
+{
+	return get()["configuration"];
+}
+
+} /* namespace GlobalConfiguration */
+
+} /* namespace libcamera */
diff --git a/src/libcamera/base/meson.build b/src/libcamera/base/meson.build
index 94843eb9..4c003284 100644
--- a/src/libcamera/base/meson.build
+++ b/src/libcamera/base/meson.build
@@ -16,6 +16,7 @@  libcamera_base_internal_sources = files([
     'event_dispatcher_poll.cpp',
     'event_notifier.cpp',
     'file.cpp',
+    'global_configuration.cpp',
     'log.cpp',
     'memfd.cpp',
     'message.cpp',