[OE-core][PATCH] systemd: add group render to udev package

[OE-core][PATCH] systemd: add group render to udev package

[Peter Marko]

From NEWS for v236:
* The "uaccess" udev tag has been dropped from /dev/kvm and
  /dev/dri/renderD*.  These devices now have the 0666 permissions by
  default (but this may be changed at build-time). /dev/dri/renderD*
  will now be owned by the "render" group along with /dev/kfd.

Without the group systemd-udevd startup logs:
  /lib/udev/rules.d/50-udev-default.rules:39 Unknown group 'render', ignoring
  /lib/udev/rules.d/50-udev-default.rules:40 Unknown group 'render', ignoring

Signed-off-by: Peter Marko <peter.marko@siemens.com>
---
 meta/recipes-core/systemd/systemd_251.4.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd_251.4.bb b/meta/recipes-core/systemd/systemd_251.4.bb
index 1b771a7b86..5fb0e86982 100644
--- a/meta/recipes-core/systemd/systemd_251.4.bb
+++ b/meta/recipes-core/systemd/systemd_251.4.bb
@@ -395,11 +395,13 @@ SYSTEMD_PACKAGES = "${@bb.utils.contains('PACKAGECONFIG', 'binfmt', '${PN}-binfm
 SYSTEMD_SERVICE:${PN}-binfmt = "systemd-binfmt.service"
 
 USERADD_PACKAGES = "${PN} ${PN}-extra-utils \
+                    udev \
                     ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-gatewayd', '', d)} \
                     ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-remote', '', d)} \
                     ${@bb.utils.contains('PACKAGECONFIG', 'journal-upload', '${PN}-journal-upload', '', d)} \
 "
 GROUPADD_PARAM:${PN} = "-r systemd-journal;"
+GROUPADD_PARAM:udev = "-r render"
 GROUPADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', '-r systemd-hostname;', '', d)}"
 USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'coredump', '--system -d / -M --shell /sbin/nologin systemd-coredump;', '', d)}"
 USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'networkd', '--system -d / -M --shell /sbin/nologin systemd-network;', '', d)}"
-- 
2.30.2

Re: [OE-core][PATCH] systemd: add group render to udev package

[Alexandre Belloni]

Hello,

This causes the following failure:

https://autobuilder.yoctoproject.org/typhoon/#/builders/117/builds/1839/steps/12/logs/stdio

ERROR: Nothing PROVIDES 'udev' (but /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-bsp/usbutils/usbutils_014.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.20.4.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/wayland/libinput_1.21.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-extended/mdadm/mdadm_4.2.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-support/libusb/libusb1_1.0.26.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-gnome/libgudev/libgudev_237.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/ofono/ofono_2.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/alsa/alsa-utils_1.2.8.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/pulseaudio/pulseaudio_16.1.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/waffle/waffle_1.7.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-bsp/pciutils/pciutils_3.8.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/wayland/weston_11.0.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/dhcpcd/dhcpcd_9.4.1.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/bluez5/bluez5_5.65.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/xorg-xserver/xserver-xorg_21.1.4.bb DEPENDS on or otherwise requires it)
systemd PROVIDES udev but was skipped: systemd - udev: system groupname render does not have a static ID defined. Add render to one of these files: /home/pokybuild/yocto-worker/reproducible/build/build-st/meta-selftest/files/static-group
eudev PROVIDES udev but was skipped: conflicting distro feature 'systemd' (in DISTRO_FEATURES)
ERROR: Required build target 'meta-world-pkgdata' has no buildable providers.
Missing or unbuildable dependency chain was: ['meta-world-pkgdata', 'usbutils', 'udev']



On 08/11/2022 18:51:54+0100, Peter Marko wrote:
> From NEWS for v236:
> * The "uaccess" udev tag has been dropped from /dev/kvm and
>   /dev/dri/renderD*.  These devices now have the 0666 permissions by
>   default (but this may be changed at build-time). /dev/dri/renderD*
>   will now be owned by the "render" group along with /dev/kfd.
> 
> Without the group systemd-udevd startup logs:
>   /lib/udev/rules.d/50-udev-default.rules:39 Unknown group 'render', ignoring
>   /lib/udev/rules.d/50-udev-default.rules:40 Unknown group 'render', ignoring
> 
> Signed-off-by: Peter Marko <peter.marko@siemens.com>
> ---
>  meta/recipes-core/systemd/systemd_251.4.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta/recipes-core/systemd/systemd_251.4.bb b/meta/recipes-core/systemd/systemd_251.4.bb
> index 1b771a7b86..5fb0e86982 100644
> --- a/meta/recipes-core/systemd/systemd_251.4.bb
> +++ b/meta/recipes-core/systemd/systemd_251.4.bb
> @@ -395,11 +395,13 @@ SYSTEMD_PACKAGES = "${@bb.utils.contains('PACKAGECONFIG', 'binfmt', '${PN}-binfm
>  SYSTEMD_SERVICE:${PN}-binfmt = "systemd-binfmt.service"
>  
>  USERADD_PACKAGES = "${PN} ${PN}-extra-utils \
> +                    udev \
>                      ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-gatewayd', '', d)} \
>                      ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-remote', '', d)} \
>                      ${@bb.utils.contains('PACKAGECONFIG', 'journal-upload', '${PN}-journal-upload', '', d)} \
>  "
>  GROUPADD_PARAM:${PN} = "-r systemd-journal;"
> +GROUPADD_PARAM:udev = "-r render"
>  GROUPADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', '-r systemd-hostname;', '', d)}"
>  USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'coredump', '--system -d / -M --shell /sbin/nologin systemd-coredump;', '', d)}"
>  USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'networkd', '--system -d / -M --shell /sbin/nologin systemd-network;', '', d)}"
> -- 
> 2.30.2
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#172987): https://lists.openembedded.org/g/openembedded-core/message/172987
> Mute This Topic: https://lists.openembedded.org/mt/94895791/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

RE: [OE-core][PATCH] systemd: add group render to udev package

[Marko, Peter]

Hello,

How do I define static group ids for this particular build and other autobuilder builds?
Is that just meta-selftest/files/static-group or also some other files?

Is there also something I need to write additionally as this seems to be a breaking change..

Thanks for support,
  Peter

-----Original Message-----

Hello,

This causes the following failure:

https://autobuilder.yoctoproject.org/typhoon/#/builders/117/builds/1839/steps/12/logs/stdio

ERROR: Nothing PROVIDES 'udev' (but /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-bsp/usbutils/usbutils_014.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.20.4.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/wayland/libinput_1.21.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-extended/mdadm/mdadm_4.2.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-support/libusb/libusb1_1.0.26.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-gnome/libgudev/libgudev_237.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/ofono/ofono_2.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/igt-gpu-tools/igt-gpu-tools_git.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/alsa/alsa-utils_1.2.8.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-multimedia/pulseaudio/pulseaudio_16.1.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/waffle/waffle_1.7.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-bsp/pciutils/pciutils_3.8.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/wayland/weston_11.0.0.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/dhcpcd/dhcpcd_9.4.1.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-connectivity/bluez5/bluez5_5.65.bb, /home/pokybuild/yocto-worker/reproducible/build/meta/recipes-graphics/xorg-xserver/xserver-xorg_21.1.4.bb DEPENDS on or otherwise requires it) systemd PROVIDES udev but was skipped: systemd - udev: system groupname render does not have a static ID defined. Add render to one of these files: /home/pokybuild/yocto-worker/reproducible/build/build-st/meta-selftest/files/static-group
eudev PROVIDES udev but was skipped: conflicting distro feature 'systemd' (in DISTRO_FEATURES)
ERROR: Required build target 'meta-world-pkgdata' has no buildable providers.
Missing or unbuildable dependency chain was: ['meta-world-pkgdata', 'usbutils', 'udev']

[OE-core][PATCH v2 0/2] systemd: add group render to udev package

[Peter Marko]

v2: update meta-selftest static ids

Peter Marko (2):
  systemd: add group render to udev package
  meta-selftest/staticids: add render group for systemd

 meta-selftest/files/static-group           | 1 +
 meta/recipes-core/systemd/systemd_251.8.bb | 2 ++
 2 files changed, 3 insertions(+)

-- 
2.30.2

[OE-core][PATCH v2 1/2] systemd: add group render to udev package

[Peter Marko]

From NEWS for v236:
* The "uaccess" udev tag has been dropped from /dev/kvm and
  /dev/dri/renderD*.  These devices now have the 0666 permissions by
  default (but this may be changed at build-time). /dev/dri/renderD*
  will now be owned by the "render" group along with /dev/kfd.

Without the group systemd-udevd startup logs:
  /lib/udev/rules.d/50-udev-default.rules:39 Unknown group 'render', ignoring
  /lib/udev/rules.d/50-udev-default.rules:40 Unknown group 'render', ignoring

Signed-off-by: Peter Marko <peter.marko@siemens.com>
---
 meta/recipes-core/systemd/systemd_251.8.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/systemd/systemd_251.8.bb b/meta/recipes-core/systemd/systemd_251.8.bb
index 1b771a7b86..5fb0e86982 100644
--- a/meta/recipes-core/systemd/systemd_251.8.bb
+++ b/meta/recipes-core/systemd/systemd_251.8.bb
@@ -395,11 +395,13 @@ SYSTEMD_PACKAGES = "${@bb.utils.contains('PACKAGECONFIG', 'binfmt', '${PN}-binfm
 SYSTEMD_SERVICE:${PN}-binfmt = "systemd-binfmt.service"
 
 USERADD_PACKAGES = "${PN} ${PN}-extra-utils \
+                    udev \
                     ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-gatewayd', '', d)} \
                     ${@bb.utils.contains('PACKAGECONFIG', 'microhttpd', '${PN}-journal-remote', '', d)} \
                     ${@bb.utils.contains('PACKAGECONFIG', 'journal-upload', '${PN}-journal-upload', '', d)} \
 "
 GROUPADD_PARAM:${PN} = "-r systemd-journal;"
+GROUPADD_PARAM:udev = "-r render"
 GROUPADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'polkit_hostnamed_fallback', '-r systemd-hostname;', '', d)}"
 USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'coredump', '--system -d / -M --shell /sbin/nologin systemd-coredump;', '', d)}"
 USERADD_PARAM:${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'networkd', '--system -d / -M --shell /sbin/nologin systemd-network;', '', d)}"
-- 
2.30.2

[OE-core][PATCH v2 2/2] meta-selftest/staticids: add render group for systemd

[Peter Marko]

---
 meta-selftest/files/static-group | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-selftest/files/static-group b/meta-selftest/files/static-group
index b2e0e2f870..b13dde3218 100644
--- a/meta-selftest/files/static-group
+++ b/meta-selftest/files/static-group
@@ -23,3 +23,4 @@ _apt:x:523:
 weston-launch:x:524:
 weston:x:525:
 wayland:x:526:
+render:x:527:
-- 
2.30.2