diff mbox series

package/php: select SYSTEMD_TMPFILES for PHP_SAPI_FPM

Message ID 20191122093930.7933-1-p.debruijn@unilogic.nl
State New
Headers show
Series package/php: select SYSTEMD_TMPFILES for PHP_SAPI_FPM | expand

Commit Message

Pascal de Bruijn Nov. 22, 2019, 9:39 a.m. UTC
PrivateTmp=true requires SYSTEMD_TMPFILES otherwise the service will
fail to start on boot.

Signed-off-by: Pascal de Bruijn <p.debruijn@unilogic.nl>
---
 package/php/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Petazzoni Nov. 25, 2019, 9:10 p.m. UTC | #1
Hello Jérémy,

Do you have some feedback about this patch ?

Thanks,

Thomas

On Fri, 22 Nov 2019 10:39:30 +0100
Pascal de Bruijn <p.debruijn@unilogic.nl> wrote:

> PrivateTmp=true requires SYSTEMD_TMPFILES otherwise the service will
> fail to start on boot.
> 
> Signed-off-by: Pascal de Bruijn <p.debruijn@unilogic.nl>
> ---
>  package/php/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/php/Config.in b/package/php/Config.in
> index 2469573..b65438a 100644
> --- a/package/php/Config.in
> +++ b/package/php/Config.in
> @@ -39,6 +39,7 @@ config BR2_PACKAGE_PHP_SAPI_CLI
>  
>  config BR2_PACKAGE_PHP_SAPI_FPM
>  	bool "FPM interface"
> +	select BR2_PACKAGE_SYSTEMD_TMPFILES if BR2_PACKAGE_SYSTEMD
>  	depends on BR2_USE_MMU
>  	# "Sparc v8 and predecessors are not and will not be supported"
>  	depends on !BR2_sparc
Jérémy Rosen Nov. 26, 2019, 5:45 p.m. UTC | #2
systemd-tmpfiles is a small utility provided by systemd to do "file system
tweaks" early during the boot

The primary use-case is to populate /run early so everything is ready when
the "real stuff" starts but it can do all sort of things
* create files
* remove files
* write to files
* adjust file permissions/ACL..
* create nodes in /dev

systemd-tmpfiles avoids launching dozen of small processes/subshells for
small adjustment as it is don intraditionnal sysV startup files

systemd-tmpfiles configuration is drop-in based so any package can add file
in the proper directory to be sure things are created/adjusted at boot time.

moreover systemd-tmpfiles is needed to be able to dynamically load modules :
* at boot time, systemd will scan modules in /lib/modules (using modinfo)
to know all /dev nodes that unloaded modules can handle
* systemd-tmpfiles will create the corresponding nodes in /dev
* when a process opens the node, the kernel will block the process and call
udev
* udev will find the module to load (using modules.alias)
* once the module is loaded, the kernel will honor the open request and
unblock the process.

systemd-tmpfiles is not an optional part of systemd. Buildroot configures
it that way, but packages are supposed to safely assume it's there on any
systemd-based system

So I would consider it a bug to have systemd-tmpfiles be optional. I'd
rather see a patch removing that aspect entirely.

Now... if we want to keep that option (which, again, I don't recommend)
yes. This patch looks good to me.



Le lun. 25 nov. 2019 à 22:10, Thomas Petazzoni <thomas.petazzoni@bootlin.com>
a écrit :

> Hello Jérémy,
>
> Do you have some feedback about this patch ?
>
> Thanks,
>
> Thomas
>
> On Fri, 22 Nov 2019 10:39:30 +0100
> Pascal de Bruijn <p.debruijn@unilogic.nl> wrote:
>
> > PrivateTmp=true requires SYSTEMD_TMPFILES otherwise the service will
> > fail to start on boot.
> >
> > Signed-off-by: Pascal de Bruijn <p.debruijn@unilogic.nl>
> > ---
> >  package/php/Config.in | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/package/php/Config.in b/package/php/Config.in
> > index 2469573..b65438a 100644
> > --- a/package/php/Config.in
> > +++ b/package/php/Config.in
> > @@ -39,6 +39,7 @@ config BR2_PACKAGE_PHP_SAPI_CLI
> >
> >  config BR2_PACKAGE_PHP_SAPI_FPM
> >       bool "FPM interface"
> > +     select BR2_PACKAGE_SYSTEMD_TMPFILES if BR2_PACKAGE_SYSTEMD
> >       depends on BR2_USE_MMU
> >       # "Sparc v8 and predecessors are not and will not be supported"
> >       depends on !BR2_sparc
>
>
>
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
Jérémy Rosen Dec. 4, 2019, 8:53 a.m. UTC | #3
Ok, for the special case of PHP, this has been fixed upstream with

https://github.com/systemd/systemd/pull/14207

The official word from upstream is "yes tmpfiles is optional but it should
always be here except for really tiny embedded systems"

So we are good (tmpfile's default is activated)

Ideally, we should backport that PR until upstream adds it to
systemd-stable.

BR
Jérémy


Le mar. 26 nov. 2019 à 18:45, Jérémy ROSEN <jeremy.rosen@smile.fr> a écrit :

> systemd-tmpfiles is a small utility provided by systemd to do "file system
> tweaks" early during the boot
>
> The primary use-case is to populate /run early so everything is ready when
> the "real stuff" starts but it can do all sort of things
> * create files
> * remove files
> * write to files
> * adjust file permissions/ACL..
> * create nodes in /dev
>
> systemd-tmpfiles avoids launching dozen of small processes/subshells for
> small adjustment as it is don intraditionnal sysV startup files
>
> systemd-tmpfiles configuration is drop-in based so any package can add
> file in the proper directory to be sure things are created/adjusted at boot
> time.
>
> moreover systemd-tmpfiles is needed to be able to dynamically load modules
> :
> * at boot time, systemd will scan modules in /lib/modules (using modinfo)
> to know all /dev nodes that unloaded modules can handle
> * systemd-tmpfiles will create the corresponding nodes in /dev
> * when a process opens the node, the kernel will block the process and
> call udev
> * udev will find the module to load (using modules.alias)
> * once the module is loaded, the kernel will honor the open request and
> unblock the process.
>
> systemd-tmpfiles is not an optional part of systemd. Buildroot configures
> it that way, but packages are supposed to safely assume it's there on any
> systemd-based system
>
> So I would consider it a bug to have systemd-tmpfiles be optional. I'd
> rather see a patch removing that aspect entirely.
>
> Now... if we want to keep that option (which, again, I don't recommend)
> yes. This patch looks good to me.
>
>
>
> Le lun. 25 nov. 2019 à 22:10, Thomas Petazzoni <
> thomas.petazzoni@bootlin.com> a écrit :
>
>> Hello Jérémy,
>>
>> Do you have some feedback about this patch ?
>>
>> Thanks,
>>
>> Thomas
>>
>> On Fri, 22 Nov 2019 10:39:30 +0100
>> Pascal de Bruijn <p.debruijn@unilogic.nl> wrote:
>>
>> > PrivateTmp=true requires SYSTEMD_TMPFILES otherwise the service will
>> > fail to start on boot.
>> >
>> > Signed-off-by: Pascal de Bruijn <p.debruijn@unilogic.nl>
>> > ---
>> >  package/php/Config.in | 1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/package/php/Config.in b/package/php/Config.in
>> > index 2469573..b65438a 100644
>> > --- a/package/php/Config.in
>> > +++ b/package/php/Config.in
>> > @@ -39,6 +39,7 @@ config BR2_PACKAGE_PHP_SAPI_CLI
>> >
>> >  config BR2_PACKAGE_PHP_SAPI_FPM
>> >       bool "FPM interface"
>> > +     select BR2_PACKAGE_SYSTEMD_TMPFILES if BR2_PACKAGE_SYSTEMD
>> >       depends on BR2_USE_MMU
>> >       # "Sparc v8 and predecessors are not and will not be supported"
>> >       depends on !BR2_sparc
>>
>>
>>
>> --
>> Thomas Petazzoni, CTO, Bootlin
>> Embedded Linux and Kernel engineering
>> https://bootlin.com
>>
>
>
> --
> [image: SMILE]  <http://www.smile.eu/>
>
> 20 rue des Jardins
> 92600 Asnières-sur-Seine
> *Jérémy ROSEN*
> Architecte technique
>
> [image: email] jeremy.rosen@smile.fr
> [image: phone]  +33 6 88 25 87 42
> [image: url] http://www.smile.eu
>
> [image: Twitter] <https://twitter.com/GroupeSmile> [image: Facebook]
> <https://www.facebook.com/smileopensource> [image: LinkedIn]
> <https://www.linkedin.com/company/smile> [image: Github]
> <https://github.com/Smile-SA>
>
> [image: Découvrez l’univers Smile, rendez-vous sur smile.eu]
> <https://www.smile.eu/fr/publications/livres-blancs/yocto?utm_source=signature&utm_medium=email&utm_campaign=signature>
>
diff mbox series

Patch

diff --git a/package/php/Config.in b/package/php/Config.in
index 2469573..b65438a 100644
--- a/package/php/Config.in
+++ b/package/php/Config.in
@@ -39,6 +39,7 @@  config BR2_PACKAGE_PHP_SAPI_CLI
 
 config BR2_PACKAGE_PHP_SAPI_FPM
 	bool "FPM interface"
+	select BR2_PACKAGE_SYSTEMD_TMPFILES if BR2_PACKAGE_SYSTEMD
 	depends on BR2_USE_MMU
 	# "Sparc v8 and predecessors are not and will not be supported"
 	depends on !BR2_sparc