diff mbox series

[V2,2/3] base-passwd: add clock group

Message ID 20251016011104.145401-2-Qi.Chen@windriver.com
State New
Headers show
Series [V2,1/3] musl: avoid redefinition error on ethhdr | expand

Commit Message

Chen, Qi Oct. 16, 2025, 1:11 a.m. UTC
From: Chen Qi <Qi.Chen@windriver.com>

New systemd version (v258) introduces a new group, clock, to
"enable applications like linuxptp to open clocks without root
privileges".[1]

This results in warning at do_rootfs time:
WARNING: Group clock has never been defined

Add group clock with gid 81 to fix this issue. The wheel group's
gid is 80, so 81 is chosen. And as the patch to add wheel group
was not accepted by upstream, directly mark this one's upstream
status to be 'Inappropriate'.

[1] https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 .../base-passwd/0001-Add-clock-group.patch    | 34 +++++++++++++++++++
 .../base-passwd/base-passwd_3.6.7.bb          |  1 +
 2 files changed, 35 insertions(+)
 create mode 100644 meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch

Comments

Yoann Congal Oct. 16, 2025, 6:45 a.m. UTC | #1
Le jeu. 16 oct. 2025 à 03:11, Chen Qi via lists.openembedded.org <Qi.Chen=
windriver.com@lists.openembedded.org> a écrit :

> From: Chen Qi <Qi.Chen@windriver.com>
>
> New systemd version (v258) introduces a new group, clock, to
> "enable applications like linuxptp to open clocks without root
> privileges".[1]
>
> This results in warning at do_rootfs time:
> WARNING: Group clock has never been defined
>
> Add group clock with gid 81 to fix this issue. The wheel group's
> gid is 80, so 81 is chosen. And as the patch to add wheel group
> was not accepted by upstream, directly mark this one's upstream
> status to be 'Inappropriate'.
>
> [1]
> https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>  .../base-passwd/0001-Add-clock-group.patch    | 34 +++++++++++++++++++
>  .../base-passwd/base-passwd_3.6.7.bb          |  1 +
>  2 files changed, 35 insertions(+)
>  create mode 100644
> meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
>
> diff --git
> a/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
> b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
> new file mode 100644
> index 0000000000..fe5a4bb210
> --- /dev/null
> +++ b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
> @@ -0,0 +1,34 @@
> +From 4b3a8f347179d7e2a1062eeed13a6474deb28728 Mon Sep 17 00:00:00 2001
> +From: Chen Qi <Qi.Chen@windriver.com>
> +Date: Thu, 16 Oct 2025 08:48:49 +0800
> +Subject: [PATCH] Add clock group
> +
> +The clock group is needed to "to enable applications like linuxptp
> +to open clocks without root privileges". See the link below:
> +
> https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd
> +
> +Note that this group is needed as long as people use udev from
> +systemd because 50-udev-default.rules needs it. See the above
> +commit.
> +
> +Upstream-Status: Inappropriate [enable feature]
> +
> +Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> +---
> + group.master | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/group.master b/group.master
> +index e54fd1d..6f84539 100644
> +--- a/group.master
> ++++ b/group.master
> +@@ -39,5 +39,6 @@ staff:*:50:
> + games:*:60:
> + shutdown:*:70:
> + wheel:*:80:
> ++clock:*:80:
>
There is an error in the GID here, from the commit message, it should say
81.

Cheers,


> + users:*:100:
> + nogroup:*:65534:
> +--
> +2.34.1
> +
> diff --git a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
> b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
> index 177927d674..e4e0890293 100644
> --- a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
> +++ b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
> @@ -13,6 +13,7 @@ SRC_URI = "
> https://launchpad.net/debian/+archive/primary/+files/${BPN}_${PV}.tar
> <https://launchpad.net/debian/+archive/primary/+files/$%7BBPN%7D_$%7BPV%7D.tar>
>             file://0005-Add-kvm-group.patch \
>             file://0007-Add-wheel-group.patch \
>             file://0001-base-passwd-Add-the-sgx-group.patch \
> +           file://0001-Add-clock-group.patch \
>             "
>
>  SRC_URI[sha256sum] =
> "cf869870fed7862b57bfa9e99cd5cd6f365e2349705a1b65af7fc182629bd1ab"
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#224910):
> https://lists.openembedded.org/g/openembedded-core/message/224910
> Mute This Topic: https://lists.openembedded.org/mt/115782555/4316185
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> yoann.congal@smile.fr]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Chen, Qi Oct. 16, 2025, 6:49 a.m. UTC | #2
Got it. Thanks for spotting the error.

Regards,
Qi

From: Yoann Congal <yoann.congal@smile.fr>
Sent: Thursday, October 16, 2025 2:46 PM
To: Chen, Qi <Qi.Chen@windriver.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core][PATCH V2 2/3] base-passwd: add clock group



Le jeu. 16 oct. 2025 à 03:11, Chen Qi via lists.openembedded.org<https://urldefense.com/v3/__http:/lists.openembedded.org__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBuyaduFkL$> <Qi.Chen=windriver.com@lists.openembedded.org<mailto:windriver.com@lists.openembedded.org>> a écrit :
From: Chen Qi <Qi.Chen@windriver.com<mailto:Qi.Chen@windriver.com>>

New systemd version (v258) introduces a new group, clock, to
"enable applications like linuxptp to open clocks without root
privileges".[1]

This results in warning at do_rootfs time:
WARNING: Group clock has never been defined

Add group clock with gid 81 to fix this issue. The wheel group's
gid is 80, so 81 is chosen. And as the patch to add wheel group
was not accepted by upstream, directly mark this one's upstream
status to be 'Inappropriate'.

[1] https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd<https://urldefense.com/v3/__https:/github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu0K89e4V$>

Signed-off-by: Chen Qi <Qi.Chen@windriver.com<mailto:Qi.Chen@windriver.com>>
---
 .../base-passwd/0001-Add-clock-group.patch    | 34 +++++++++++++++++++
 .../base-passwd/base-passwd_3.6.7.bb<https://urldefense.com/v3/__http:/base-passwd_3.6.7.bb__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu3nzN2Wh$>          |  1 +
 2 files changed, 35 insertions(+)
 create mode 100644 meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch

diff --git a/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
new file mode 100644
index 0000000000..fe5a4bb210
--- /dev/null
+++ b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
@@ -0,0 +1,34 @@
+From 4b3a8f347179d7e2a1062eeed13a6474deb28728 Mon Sep 17 00:00:00 2001
+From: Chen Qi <Qi.Chen@windriver.com<mailto:Qi.Chen@windriver.com>>
+Date: Thu, 16 Oct 2025 08:48:49 +0800
+Subject: [PATCH] Add clock group
+
+The clock group is needed to "to enable applications like linuxptp
+to open clocks without root privileges". See the link below:
+https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd<https://urldefense.com/v3/__https:/github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu0K89e4V$>
+
+Note that this group is needed as long as people use udev from
+systemd because 50-udev-default.rules needs it. See the above
+commit.
+
+Upstream-Status: Inappropriate [enable feature]
+
+Signed-off-by: Chen Qi <Qi.Chen@windriver.com<mailto:Qi.Chen@windriver.com>>
+---
+ group.master | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/group.master b/group.master
+index e54fd1d..6f84539 100644
+--- a/group.master
++++ b/group.master
+@@ -39,5 +39,6 @@ staff:*:50:
+ games:*:60:
+ shutdown:*:70:
+ wheel:*:80:
++clock:*:80:
There is an error in the GID here, from the commit message, it should say 81.

Cheers,

+ users:*:100:
+ nogroup:*:65534:
+--
+2.34.1
+
diff --git a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb<https://urldefense.com/v3/__http:/base-passwd_3.6.7.bb__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu3nzN2Wh$> b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb<https://urldefense.com/v3/__http:/base-passwd_3.6.7.bb__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu3nzN2Wh$>
index 177927d674..e4e0890293 100644
--- a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb<https://urldefense.com/v3/__http:/base-passwd_3.6.7.bb__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu3nzN2Wh$>
+++ b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb<https://urldefense.com/v3/__http:/base-passwd_3.6.7.bb__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu3nzN2Wh$>
@@ -13,6 +13,7 @@ SRC_URI = "https://launchpad.net/debian/+archive/primary/+files/${BPN}_${PV}.tar<https://urldefense.com/v3/__https:/launchpad.net/debian/*archive/primary/*files/$*7BBPN*7D_$*7BPV*7D.tar__;KyslJSUl!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBuz665G4N$>
            file://0005-Add-kvm-group.patch \
            file://0007-Add-wheel-group.patch \
            file://0001-base-passwd-Add-the-sgx-group.patch \
+           file://0001-Add-clock-group.patch \
            "

 SRC_URI[sha256sum] = "cf869870fed7862b57bfa9e99cd5cd6f365e2349705a1b65af7fc182629bd1ab"
--
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#224910): https://lists.openembedded.org/g/openembedded-core/message/224910<https://urldefense.com/v3/__https:/lists.openembedded.org/g/openembedded-core/message/224910__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBuwXo5eeI$>
Mute This Topic: https://lists.openembedded.org/mt/115782555/4316185<https://urldefense.com/v3/__https:/lists.openembedded.org/mt/115782555/4316185__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu9PrSkL-$>
Group Owner: openembedded-core+owner@lists.openembedded.org<mailto:openembedded-core%2Bowner@lists.openembedded.org>
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub<https://urldefense.com/v3/__https:/lists.openembedded.org/g/openembedded-core/unsub__;!!AjveYdw8EvQ!cfbH5WCDEAZZ2LX2GX4j--mYpN-Wrr8b2rh1j3jIH8o6NviPIRMZVa5woryIYu73h7gzwMh-evTIEaqBu41k-6he$> [yoann.congal@smile.fr<mailto:yoann.congal@smile.fr>]
-=-=-=-=-=-=-=-=-=-=-=-


--
Yoann Congal
Smile ECS
diff mbox series

Patch

diff --git a/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
new file mode 100644
index 0000000000..fe5a4bb210
--- /dev/null
+++ b/meta/recipes-core/base-passwd/base-passwd/0001-Add-clock-group.patch
@@ -0,0 +1,34 @@ 
+From 4b3a8f347179d7e2a1062eeed13a6474deb28728 Mon Sep 17 00:00:00 2001
+From: Chen Qi <Qi.Chen@windriver.com>
+Date: Thu, 16 Oct 2025 08:48:49 +0800
+Subject: [PATCH] Add clock group
+
+The clock group is needed to "to enable applications like linuxptp
+to open clocks without root privileges". See the link below:
+https://github.com/systemd/systemd/commit/af96ccfc24bc4803078a46b4ef2cdeb5decdfbcd
+
+Note that this group is needed as long as people use udev from
+systemd because 50-udev-default.rules needs it. See the above
+commit.
+
+Upstream-Status: Inappropriate [enable feature]
+
+Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
+---
+ group.master | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/group.master b/group.master
+index e54fd1d..6f84539 100644
+--- a/group.master
++++ b/group.master
+@@ -39,5 +39,6 @@ staff:*:50:
+ games:*:60:
+ shutdown:*:70:
+ wheel:*:80:
++clock:*:80:
+ users:*:100:
+ nogroup:*:65534:
+-- 
+2.34.1
+
diff --git a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
index 177927d674..e4e0890293 100644
--- a/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
+++ b/meta/recipes-core/base-passwd/base-passwd_3.6.7.bb
@@ -13,6 +13,7 @@  SRC_URI = "https://launchpad.net/debian/+archive/primary/+files/${BPN}_${PV}.tar
            file://0005-Add-kvm-group.patch \
            file://0007-Add-wheel-group.patch \
            file://0001-base-passwd-Add-the-sgx-group.patch \
+           file://0001-Add-clock-group.patch \
            "
 
 SRC_URI[sha256sum] = "cf869870fed7862b57bfa9e99cd5cd6f365e2349705a1b65af7fc182629bd1ab"