From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 16 Mar 2023 15:47:25 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pcot2-002lw4-Pl for lore@lore.pengutronix.de; Thu, 16 Mar 2023 15:47:25 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1pcot2-0007oc-Tp; Thu, 16 Mar 2023 15:47:24 +0100 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pcosq-0007oO-BV; Thu, 16 Mar 2023 15:47:12 +0100 Received: from [2a0a:edc0:0:1101:1d::54] (helo=dude05.red.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.94.2) (envelope-from ) id 1pcosp-004ZUY-Nu; Thu, 16 Mar 2023 15:47:11 +0100 Received: from mol by dude05.red.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1pcosp-000LJj-3f; Thu, 16 Mar 2023 15:47:11 +0100 Date: Thu, 16 Mar 2023 15:47:11 +0100 From: Michael Olbrich To: Roland Hieber , ptxdist@pengutronix.de Message-ID: Mail-Followup-To: Roland Hieber , ptxdist@pengutronix.de References: <20230315200130.2699501-1-rhi@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230315200130.2699501-1-rhi@pengutronix.de> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain Subject: Re: [ptxdist] [PATCH] systemd: add option to install systemd-repart X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false On Wed, Mar 15, 2023 at 09:01:31PM +0100, Roland Hieber wrote: > It runs on first boot via systemd-repart.service, so don't install it by > default. > > Signed-off-by: Roland Hieber > --- > rules/systemd.in | 8 ++++++++ > rules/systemd.make | 9 +++++++-- > 2 files changed, 15 insertions(+), 2 deletions(-) > > diff --git a/rules/systemd.in b/rules/systemd.in > index 00d922a05c0a..3829bb925a96 100644 > --- a/rules/systemd.in > +++ b/rules/systemd.in > @@ -19,6 +19,7 @@ menuconfig SYSTEMD > select LIBKMOD_TOOLS > select UTIL_LINUX > select UTIL_LINUX_LIBMOUNT > + select UTIL_LINUX_LIBFDISK if SYSTEMD_REPART > select MACHINE_ID if RUNTIME > select XZ if SYSTEMD_XZ > select LZ4 if SYSTEMD_LZ4 > @@ -170,6 +171,13 @@ config SYSTEMD_NSPAWN > bool > prompt "install systemd-nspawn" > > +config SYSTEMD_REPART > + bool > + prompt "install systemd-repart" > + help > + systemd-repart repartitions the disk on first start according to > + configuration files in /usr/lib/repart.d > + > config SYSTEMD_DEFAULT_NET_NAMING_SCHEME > string > prompt "default net.naming-scheme= value" > diff --git a/rules/systemd.make b/rules/systemd.make > index d42ab860624e..cd8c0bf9a70d 100644 > --- a/rules/systemd.make > +++ b/rules/systemd.make > @@ -86,7 +86,7 @@ SYSTEMD_CONF_OPT := \ > -Delfutils=$(call ptx/truefalse,PTXCONF_SYSTEMD_COREDUMP) \ > -Denvironment-d=false \ > -Dfallback-hostname=$(call ptx/ifdef,PTXCONF_ROOTFS_ETC_HOSTNAME,$(PTXCONF_ROOTFS_ETC_HOSTNAME),ptxdist) \ > - -Dfdisk=false \ > + -Dfdisk=$(call ptx/truefalse,PTXCONF_SYSTEMD_REPART) \ > -Dfexecve=false \ > -Dfirstboot=false \ > -Dfuzz-tests=false \ > @@ -162,7 +162,7 @@ SYSTEMD_CONF_OPT := \ > -Dquotaon-path=/usr/sbin/quotaon \ > -Drandomseed=$(call ptx/falsetrue,PTXCONF_SYSTEMD_DISABLE_RANDOM_SEED) \ > -Dremote=$(call ptx/ifdef,PTXCONF_SYSTEMD_JOURNAL_REMOTE,auto,false) \ > - -Drepart=false \ > + -Drepart=$(call ptx/truefalse,PTXCONF_SYSTEMD_REPART) \ > -Dresolve=$(call ptx/truefalse,PTXCONF_SYSTEMD_NETWORK) \ > -Drfkill=false \ > -Dseccomp=$(call ptx/truefalse,PTXCONF_SYSTEMD_SECCOMP) \ > @@ -381,6 +381,11 @@ ifdef PTXCONF_SYSTEMD_NSPAWN > @$(call install_copy, systemd, 0, 0, 0755, -, /usr/bin/systemd-nspawn) > endif > > +ifdef PTXCONF_SYSTEMD_REPART > + @$(call install_copy, systemd, 0, 0, 0755, -, /usr/bin/systemd-repart) > + @$(call install_alternative_tree, systemd, 0, 0, /usr/lib/repart.d/) I don't think these configs should be installed here. Upstream systemd does not provide any configs (for good reasons) so targetinstall will fail unless this directory exists in the BSP. That's not acceptable for ptxdist upstream. And if a config is installed here on the target, then systemd-repart.service runs at every boot and I don't think all use-cases that want the systemd-repart tool will also want to run this service. So I think the configuration should be in a separate package in the BSP. Michael > +endif > + > @$(call install_tree, systemd, 0, 0, -, /usr/lib/systemd/system-generators/) > @$(foreach helper, $(SYSTEMD_HELPER), \ > $(call install_copy, systemd, 0, 0, 0755, -, \ > -- > 2.30.2 > > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |