[PATCH v2 1/6] default-providers: Add VIRTUAL-RUNTIME_dbus variable

[PATCH v2 1/6] default-providers: Add VIRTUAL-RUNTIME_dbus variable

[Niko Mauno]

The purpose of the new variable is to facilitate oe-core users who wish
to use an alternative runtime D-Bus implementation instead of the
default Freedesktop.org's dbus, such as dbus-broker, a recipe for which
is currently available under
meta-openembedded/meta-oe/recipes-core/dbus/

While introducing this facilitation the intent is to preserve the
existing functionality, while allowing the user to optionally select an
alternative runtime D-Bus implementation by adding the following line
e.g. to local.conf file:

  VIRTUAL-RUNTIME_dbus = "dbus-broker"

As a background, for example the Fedora distribution uses dbus-broker
instead of Freedesktop.org's D-Bus implementation. The following
excerpts from
https://fedoraproject.org/wiki/Changes/DbusBrokerAsTheDefaultDbusImplementation
provide background for their technological decision

  This change provides a more scalable and more reliable implementation in place of the reference implementation.
  The reference implementation suffers from long-standing issues including potential dead-locks and susceptibility to denial of service attacks, which the replacement implementation does not.

and

  No visible changes in behavior are expected, except for different log messages.
  Any visibly different behavior to `dbus-daemon` should be reported as a regression.
  Ideally, this change should only improve the performance and security of the message bus.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/conf/distro/include/default-providers.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
index d3eefb754b..cebd0400f4 100644
--- a/meta/conf/distro/include/default-providers.inc
+++ b/meta/conf/distro/include/default-providers.inc
@@ -31,6 +31,7 @@ VIRTUAL-RUNTIME_base-utils ?= "busybox"
 VIRTUAL-RUNTIME_base-utils-hwclock ?= "busybox-hwclock"
 VIRTUAL-RUNTIME_base-utils-syslog ?= "busybox-syslog"
 VIRTUAL-RUNTIME_keymaps ?= "keymaps"
+VIRTUAL-RUNTIME_dbus ?= "dbus"
 
 #
 # Default recipe providers
-- 
2.39.5

[PATCH v2 2/6] dbus: Allow using dbus-lib with alternative D-Bus

[Niko Mauno]

In order to facilitate oe-core users who wish to use e.g. dbus-broker
as their runtime D-Bus implementation, declare the target specific
dbus-lib runtime recommendation so that it will honor the new
VIRTUAL-RUNTIME_dbus variable.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/recipes-core/dbus/dbus_1.16.2.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/dbus/dbus_1.16.2.bb b/meta/recipes-core/dbus/dbus_1.16.2.bb
index 9475e55b57..eeeecde946 100644
--- a/meta/recipes-core/dbus/dbus_1.16.2.bb
+++ b/meta/recipes-core/dbus/dbus_1.16.2.bb
@@ -105,6 +105,7 @@ FILES:${PN}-tools = "${bindir}/dbus-uuidgen \
 "
 FILES:${PN}-lib = "${libdir}/lib*.so.*"
 RRECOMMENDS:${PN}-lib = "${PN}"
+RRECOMMENDS:${PN}-lib:class-target = "${VIRTUAL-RUNTIME_dbus}"
 FILES:${PN}-dev += "${libdir}/dbus-1.0/include ${bindir}/dbus-test-tool ${datadir}/xml/dbus-1"
 
 RDEPENDS:${PN}-ptest += "bash make dbus"
-- 
2.39.5

[PATCH v2 3/6] dbus-glib: Support using alternative runtime D-Bus

[Niko Mauno]

In order to facilitate oe-core users who wish to use e.g. dbus-broker
as their runtime D-Bus implementation, decouple the target specific
dbus-glib runtime recommendation to honor the new VIRTUAL-RUNTIME_dbus
variable.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/recipes-core/dbus/dbus-glib_0.114.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/dbus/dbus-glib_0.114.bb b/meta/recipes-core/dbus/dbus-glib_0.114.bb
index ff7c2d6613..7908a02747 100644
--- a/meta/recipes-core/dbus/dbus-glib_0.114.bb
+++ b/meta/recipes-core/dbus/dbus-glib_0.114.bb
@@ -38,7 +38,9 @@ LICENSE:${PN}-bash-completion = "GPL-2.0-or-later"
 FILES:${PN}-dev += "${libdir}/dbus-1.0/include ${bindir}/dbus-glib-tool"
 FILES:${PN}-dev += "${bindir}/dbus-binding-tool"
 
-RDEPENDS:${PN}-tests += "dbus"
+RDEPENDS:${PN}-tests:class-target += "${VIRTUAL-RUNTIME_dbus}"
+RDEPENDS:${PN}-tests:class-native += "dbus"
+RDEPENDS:${PN}-tests:class-nativesdk += "dbus"
 FILES:${PN}-tests = "${datadir}/${BPN}/tests"
 
 BBCLASSEXTEND = "native"
-- 
2.39.5

[PATCH v2 4/6] systemd: Support using alternative runtime D-Bus

[Niko Mauno]

In order to facilitate oe-core users who wish to use e.g. dbus-broker
as their runtime D-Bus implementation, change the Freedesktop.org
specific D-Bus runtime dependency to honor the new VIRTUAL-RUNTIME_dbus
variable.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/recipes-core/systemd/systemd_257.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd_257.4.bb b/meta/recipes-core/systemd/systemd_257.4.bb
index 64fb8fe69a..24aad11c0a 100644
--- a/meta/recipes-core/systemd/systemd_257.4.bb
+++ b/meta/recipes-core/systemd/systemd_257.4.bb
@@ -752,7 +752,7 @@ FILES:${PN} = " ${base_bindir}/* \
 
 FILES:${PN}-dev += "${base_libdir}/security/*.la ${datadir}/dbus-1/interfaces/ ${sysconfdir}/rpm/macros.systemd"
 
-RDEPENDS:${PN} += "kmod dbus util-linux-mount util-linux-umount udev (= ${EXTENDPKGV}) systemd-udev-rules util-linux-agetty util-linux-fsck util-linux-swaponoff util-linux-mkswap"
+RDEPENDS:${PN} += "kmod ${VIRTUAL-RUNTIME_dbus} util-linux-mount util-linux-umount udev (= ${EXTENDPKGV}) systemd-udev-rules util-linux-agetty util-linux-fsck util-linux-swaponoff util-linux-mkswap"
 RDEPENDS:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'serial-getty-generator', '', 'systemd-serialgetty', d)}"
 RDEPENDS:${PN} += "volatile-binds"
 
-- 
2.39.5

[PATCH v2 5/6] glib: Support using alternative runtime D-Bus

[Niko Mauno]

In order to facilitate oe-core users who wish to use e.g. dbus-broker
as their runtime D-Bus implementation, change the Freedesktop.org
specific D-Bus runtime dependency to honor the new VIRTUAL-RUNTIME_dbus
variable.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/recipes-core/glib-2.0/glib.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-2.0/glib.inc
index 61e1a3ef17..2cc48f0a06 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -160,7 +160,7 @@ RDEPENDS:${PN}-codegen += "${CODEGEN_PYTHON_RDEPENDS}"
 RDEPENDS:${PN}-ptest += "\
             coreutils \
             libgcc \
-            dbus \
+            ${VIRTUAL-RUNTIME_dbus} \
             desktop-file-utils \
             gnome-desktop-testing \
             tzdata \
-- 
2.39.5

[PATCH v2 6/6] recipes-connectivity: Support alternative runtime D-Bus

[Niko Mauno]

In order to facilitate oe-core users who wish to use e.g. dbus-broker
as their runtime D-Bus implementation, change the Freedesktop.org
specific D-Bus runtime dependency to honor the new VIRTUAL-RUNTIME_dbus
variable.

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 meta/recipes-connectivity/bluez5/bluez5.inc   | 2 +-
 meta/recipes-connectivity/connman/connman.inc | 2 +-
 meta/recipes-connectivity/neard/neard_0.19.bb | 2 +-
 meta/recipes-connectivity/ofono/ofono_2.15.bb | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc
index d626872103..09afd8f7c9 100644
--- a/meta/recipes-connectivity/bluez5/bluez5.inc
+++ b/meta/recipes-connectivity/bluez5/bluez5.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
                     file://COPYING.LIB;md5=fb504b67c50331fc78734fed90fb0e09 \
                     file://src/main.c;beginline=1;endline=24;md5=0ad83ca0dc37ab08af448777c581e7ac"
 DEPENDS = "dbus glib-2.0"
-RDEPENDS:${PN} += "dbus"
+RDEPENDS:${PN} += "${VIRTUAL-RUNTIME_dbus}"
 PROVIDES += "bluez-hcidump"
 RPROVIDES:${PN} += "bluez-hcidump"
 
diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index 133e8616da..fa546ee872 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -119,7 +119,7 @@ RPROVIDES:${PN} = "\
 	"
 
 RDEPENDS:${PN} = "\
-	dbus \
+	${VIRTUAL-RUNTIME_dbus} \
 	"
 
 PACKAGES_DYNAMIC += "^${PN}-plugin-.*"
diff --git a/meta/recipes-connectivity/neard/neard_0.19.bb b/meta/recipes-connectivity/neard/neard_0.19.bb
index b04187bcc5..6a1314a957 100644
--- a/meta/recipes-connectivity/neard/neard_0.19.bb
+++ b/meta/recipes-connectivity/neard/neard_0.19.bb
@@ -37,7 +37,7 @@ do_install:append() {
 	fi
 }
 
-RDEPENDS:${PN} = "dbus"
+RDEPENDS:${PN} = "${VIRTUAL-RUNTIME_dbus}"
 
 # Bluez & Wifi are not mandatory except for handover
 WIRELESS_DAEMON ??= "wpa-supplicant"
diff --git a/meta/recipes-connectivity/ofono/ofono_2.15.bb b/meta/recipes-connectivity/ofono/ofono_2.15.bb
index 40eeb3a086..fddd9b8738 100644
--- a/meta/recipes-connectivity/ofono/ofono_2.15.bb
+++ b/meta/recipes-connectivity/ofono/ofono_2.15.bb
@@ -37,7 +37,7 @@ PACKAGES =+ "${PN}-tests"
 FILES:${PN} += "${systemd_unitdir}"
 FILES:${PN}-tests = "${libdir}/${BPN}/test"
 
-RDEPENDS:${PN} += "dbus"
+RDEPENDS:${PN} += "${VIRTUAL-RUNTIME_dbus}"
 RDEPENDS:${PN}-tests = "\
     python3-core \
     python3-dbus \
-- 
2.39.5

Re: [OE-core] [PATCH v2 1/6] default-providers: Add VIRTUAL-RUNTIME_dbus variable

[Adrian Freihofer]

Hi Niko

Thank you for the patch. It would be nice to have an easy way to switch
to dbus-broker.

On Fri, 2025-03-28 at 15:01 +0000, Niko Mauno via
lists.openembedded.org wrote:
> The purpose of the new variable is to facilitate oe-core users who
> wish
> to use an alternative runtime D-Bus implementation instead of the
> default Freedesktop.org's dbus, such as dbus-broker, a recipe for
> which
> is currently available under
> meta-openembedded/meta-oe/recipes-core/dbus/
> 
> While introducing this facilitation the intent is to preserve the
> existing functionality, while allowing the user to optionally select
> an
> alternative runtime D-Bus implementation by adding the following line
> e.g. to local.conf file:
> 
>   VIRTUAL-RUNTIME_dbus = "dbus-broker"

Introducing a VIRTUAL-RUNTIME_dbus variable would work. But it requires
to change all dbus related recipes which are not only in oe-core.

I'm wondering if using a PREFERRED_RPROVIDER would simplify this.
https://docs.yoctoproject.org/dev/ref-manual/variables.html#term-PREFERRED_RPROVIDER

This would probably allow a back and forward compatible implementation
with changing only the dbus-broker and maybe the dbus recipes plus
adding something like
  PREFERRED_RPROVIDER_dbus ?= "dbus"
or
  PREFERRED_RPROVIDER_virtual-dbus ?= "dbus"
to OE-core.

Regards,
Adrian

> 
> As a background, for example the Fedora distribution uses dbus-broker
> instead of Freedesktop.org's D-Bus implementation. The following
> excerpts from
> https://fedoraproject.org/wiki/Changes/DbusBrokerAsTheDefaultDbusImplementation
> provide background for their technological decision
> 
>   This change provides a more scalable and more reliable
> implementation in place of the reference implementation.
>   The reference implementation suffers from long-standing issues
> including potential dead-locks and susceptibility to denial of
> service attacks, which the replacement implementation does not.
> 
> and
> 
>   No visible changes in behavior are expected, except for different
> log messages.
>   Any visibly different behavior to `dbus-daemon` should be reported
> as a regression.
>   Ideally, this change should only improve the performance and
> security of the message bus.
> 
> Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
> ---
>  meta/conf/distro/include/default-providers.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/conf/distro/include/default-providers.inc
> b/meta/conf/distro/include/default-providers.inc
> index d3eefb754b..cebd0400f4 100644
> --- a/meta/conf/distro/include/default-providers.inc
> +++ b/meta/conf/distro/include/default-providers.inc
> @@ -31,6 +31,7 @@ VIRTUAL-RUNTIME_base-utils ?= "busybox"
>  VIRTUAL-RUNTIME_base-utils-hwclock ?= "busybox-hwclock"
>  VIRTUAL-RUNTIME_base-utils-syslog ?= "busybox-syslog"
>  VIRTUAL-RUNTIME_keymaps ?= "keymaps"
> +VIRTUAL-RUNTIME_dbus ?= "dbus"
>  
>  #
>  # Default recipe providers
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#213847):
> https://lists.openembedded.org/g/openembedded-core/message/213847
> Mute This Topic: https://lists.openembedded.org/mt/111955697/4454582
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe:
> https://lists.openembedded.org/g/openembedded-core/unsub [
> adrian.freihofer@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 

Re: [OE-core] [PATCH v2 1/6] default-providers: Add VIRTUAL-RUNTIME_dbus variable

[Richard Purdie]

On Sat, 2025-03-29 at 15:02 +0100, Adrian Freihofer via lists.openembedded.org wrote:
> Hi Niko
> 
> Thank you for the patch. It would be nice to have an easy way to switch
> to dbus-broker.
> 
> On Fri, 2025-03-28 at 15:01 +0000, Niko Mauno via
> lists.openembedded.org wrote:
> > The purpose of the new variable is to facilitate oe-core users who
> > wish
> > to use an alternative runtime D-Bus implementation instead of the
> > default Freedesktop.org's dbus, such as dbus-broker, a recipe for
> > which
> > is currently available under
> > meta-openembedded/meta-oe/recipes-core/dbus/
> > 
> > While introducing this facilitation the intent is to preserve the
> > existing functionality, while allowing the user to optionally select
> > an
> > alternative runtime D-Bus implementation by adding the following line
> > e.g. to local.conf file:
> > 
> >   VIRTUAL-RUNTIME_dbus = "dbus-broker"
> 
> Introducing a VIRTUAL-RUNTIME_dbus variable would work. But it requires
> to change all dbus related recipes which are not only in oe-core.
> 
> I'm wondering if using a PREFERRED_RPROVIDER would simplify this.
> https://docs.yoctoproject.org/dev/ref-manual/variables.html#term-PREFERRED_RPROVIDER
> 
> This would probably allow a back and forward compatible implementation
> with changing only the dbus-broker and maybe the dbus recipes plus
> adding something like
>   PREFERRED_RPROVIDER_dbus ?= "dbus"
> or
>   PREFERRED_RPROVIDER_virtual-dbus ?= "dbus"
> to OE-core.

Sadly it doesn't work like that. I regret adding PREFERRED_RPROVIDER as
it doesn't work as people think and only helps under very very rare
circumstances.

Runtime issues are a package manager level problem, not a bitbake one,
unless bitbake is doing variable substitution beforehand, which it only
does with the normal rules.

Cheers,

Richard

Re: [PATCH v2 6/6] recipes-connectivity: Support alternative runtime D-Bus

[Ross Burton]

On 28 Mar 2025, at 15:01, Niko Mauno <niko.mauno@vaisala.com> wrote:
> -RDEPENDS:${PN} += "dbus"
> +RDEPENDS:${PN} += "${VIRTUAL-RUNTIME_dbus}"

I suspect you can actually drop these lines in this patch instead of changing it to use the variable, as the binaries use libdbus then they will pull in the bus via that dependency anyway.

Ross

Re: [PATCH v2 6/6] recipes-connectivity: Support alternative runtime D-Bus

[Niko Mauno]

On 31.3.2025 19.21, Ross Burton wrote:
> On 28 Mar 2025, at 15:01, Niko Mauno <niko.mauno@vaisala.com> wrote:
>> -RDEPENDS:${PN} += "dbus"
>> +RDEPENDS:${PN} += "${VIRTUAL-RUNTIME_dbus}"
> 
> I suspect you can actually drop these lines in this patch instead of changing it to use the variable, as the binaries use libdbus then they will pull in the bus via that dependency anyway.
> 
> Ross

Thanks Ross, submitted v3 in which the RDEPENDS:${PN} lines are dropped altogether in the topmost commit.
-Niko