[Buildroot] [PATCH 0/3] system: extend the list of init systems (branch yem/init-choice)

[Buildroot] [PATCH 0/3] system: extend the list of init systems (branch yem/init-choice)

[Yann E. MORIN]

Hello All!

This little series is a proposal to follow on the discussion with Dario
and Thomas, about Dario's series adding tiny-init:
    https://lore.kernel.org/buildroot/ZtF1pHPZqUkUit8N@landeda/T/#mc1ef6288de44b9d35d382187ae14edf3185aa0c5

Note that, contrary to what was said in that discussion, there is no
need to add a build dependency from Busybox to tini, as tini does not
install itself as /init or /sbin/init, so will not be conflicting with
Busybox.

Regards,
Yann E. MORIN.


----------------------------------------------------------------
Yann E. MORIN (3):
      system: change label for BR2_INIT_NONE
      system: classify init systems
      system: add tini as init system

 package/tini/Config.in |  1 +
 system/Config.in       | 22 +++++++++++++++++++++-
 2 files changed, 22 insertions(+), 1 deletion(-)

--
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 1/3] system: change label for BR2_INIT_NONE

[Yann E. MORIN]

Currently, the label reads "None", but it can be a bit misleading.
Indeed, since its inception in 0c00636e8bbc (system/init: add option for
no init system), what it really meant was "none of the above", as
briefly laid out in the commit log for 0c00636e8bbc (typo included):

    Can be usefull if a local package provides an init procedure.

This was later refined in 4d185e5ec51b (system: add help entry to "none"
init system), which explained:

    Add a comment that states the user has to provide his own init
    system, either via a package or a rootfs overlay.

So, change the label to better reflect that.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 system/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/system/Config.in b/system/Config.in
index 318c538ef1..c710b38e74 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -146,7 +146,7 @@ comment "systemd highly recommends Linux >= 4.15"
 	depends on !BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_15
 
 config BR2_INIT_NONE
-	bool "None"
+	bool "Custom (none)"
 	select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
 	help
 	  Buildroot will not install any init system. You will
-- 
2.46.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 2/3] system: classify init systems

[Yann E. MORIN]

We're going to add more init systems, and some have a very special
purpose, so we do not want to mix them with the mundane platitude of
existing init systems.

Add comments to separate the two.

Note that the list would already contain comments, when a br2-external
tree provides an init system, so those new comments would not be totally
weird-looking.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 system/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/system/Config.in b/system/Config.in
index c710b38e74..18cc39ad94 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -87,6 +87,8 @@ choice
 	prompt "Init system"
 	default BR2_INIT_BUSYBOX
 
+comment "General purpose"
+
 config BR2_INIT_BUSYBOX
 	bool "BusyBox"
 	select BR2_PACKAGE_BUSYBOX
@@ -145,6 +147,8 @@ comment "systemd highly recommends Linux >= 4.15"
 	depends on BR2_INIT_SYSTEMD
 	depends on !BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_15
 
+comment "Special purpose (read help)"
+
 config BR2_INIT_NONE
 	bool "Custom (none)"
 	select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
-- 
2.46.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 2/3] system: classify init systems

[Dario Binacchi]

Hi Yann,

On Fri, Aug 30, 2024 at 10:26 PM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
>
> We're going to add more init systems, and some have a very special
> purpose, so we do not want to mix them with the mundane platitude of
> existing init systems.
>
> Add comments to separate the two.
>
> Note that the list would already contain comments, when a br2-external
> tree provides an init system, so those new comments would not be totally
> weird-looking.
>
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> ---
>  system/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/system/Config.in b/system/Config.in
> index c710b38e74..18cc39ad94 100644
> --- a/system/Config.in
> +++ b/system/Config.in
> @@ -87,6 +87,8 @@ choice
>         prompt "Init system"
>         default BR2_INIT_BUSYBOX
>
> +comment "General purpose"
> +
>  config BR2_INIT_BUSYBOX
>         bool "BusyBox"
>         select BR2_PACKAGE_BUSYBOX
> @@ -145,6 +147,8 @@ comment "systemd highly recommends Linux >= 4.15"
>         depends on BR2_INIT_SYSTEMD
>         depends on !BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_15
>
> +comment "Special purpose (read help)"
> +
>  config BR2_INIT_NONE
>         bool "Custom (none)"
>         select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
> --
> 2.46.0
>

Reviewed-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

Regards,
Dario
-- 

Dario Binacchi

Senior Embedded Linux Developer

dario.binacchi@amarulasolutions.com

__________________________________


Amarula Solutions SRL

Via Le Canevare 30, 31100 Treviso, Veneto, IT

T. +39 042 243 5310
info@amarulasolutions.com

www.amarulasolutions.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 3/3] system: add tini as init system

[Yann E. MORIN]

Tini is a special-purpose init system, aimed at containers.

Add it to the choice of init systems.

tini does not require much, so we re-use the default "none" skeleton.

Note that tini is no longer selectable on its own, which it used to be.
This change will require that people using tini be careful when
upgrading, though, but that will be very easily caught, as the
containers would no longer start at all.

We could ad a comment stating why, but no other init system has such a
comment, so adding one for tini alone would be weird, so we don't add
one.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 package/tini/Config.in |  1 +
 system/Config.in       | 16 ++++++++++++++++
 2 files changed, 17 insertions(+)

diff --git a/package/tini/Config.in b/package/tini/Config.in
index 719e1e1f22..bca8ad8e96 100644
--- a/package/tini/Config.in
+++ b/package/tini/Config.in
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_TINI
 	bool "tini"
 	depends on BR2_USE_MMU # fork()
+	depends on BR2_INIT_TINI
 	help
 	  tini is a simple but valid init binary to
 	  act as PID 1 for containers.
diff --git a/system/Config.in b/system/Config.in
index 18cc39ad94..d7d59db47d 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -149,6 +149,22 @@ comment "systemd highly recommends Linux >= 4.15"
 
 comment "Special purpose (read help)"
 
+config BR2_INIT_TINI
+	bool "tini"
+	depends on BR2_USE_MMU  # tini
+	select BR2_PACKAGE_TINI
+	select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
+	help
+	  Tini - A tiny but valid init for containers
+
+	  Tini is the simplest init you could think of.
+
+	  All Tini does is spawn a single child (Tini is meant to be run
+	  in a container), and wait for it to exit all the while reaping
+	  zombies and performing signal forwarding.
+
+	  https://github.com/krallin/tini
+
 config BR2_INIT_NONE
 	bool "Custom (none)"
 	select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
-- 
2.46.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 3/3] system: add tini as init system

[Dario Binacchi]

Hi Yann,

On Fri, Aug 30, 2024 at 10:26 PM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
>
> Tini is a special-purpose init system, aimed at containers.
>
> Add it to the choice of init systems.
>
> tini does not require much, so we re-use the default "none" skeleton.
>
> Note that tini is no longer selectable on its own, which it used to be.
> This change will require that people using tini be careful when
> upgrading, though, but that will be very easily caught, as the
> containers would no longer start at all.
>
> We could ad a comment stating why, but no other init system has such a
> comment, so adding one for tini alone would be weird, so we don't add
> one.
>
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> ---
>  package/tini/Config.in |  1 +
>  system/Config.in       | 16 ++++++++++++++++
>  2 files changed, 17 insertions(+)
>
> diff --git a/package/tini/Config.in b/package/tini/Config.in
> index 719e1e1f22..bca8ad8e96 100644
> --- a/package/tini/Config.in
> +++ b/package/tini/Config.in
> @@ -1,6 +1,7 @@
>  config BR2_PACKAGE_TINI
>         bool "tini"
>         depends on BR2_USE_MMU # fork()
> +       depends on BR2_INIT_TINI
>         help
>           tini is a simple but valid init binary to
>           act as PID 1 for containers.
> diff --git a/system/Config.in b/system/Config.in
> index 18cc39ad94..d7d59db47d 100644
> --- a/system/Config.in
> +++ b/system/Config.in
> @@ -149,6 +149,22 @@ comment "systemd highly recommends Linux >= 4.15"
>
>  comment "Special purpose (read help)"
>
> +config BR2_INIT_TINI
> +       bool "tini"
> +       depends on BR2_USE_MMU  # tini
> +       select BR2_PACKAGE_TINI
> +       select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
> +       help
> +         Tini - A tiny but valid init for containers
> +
> +         Tini is the simplest init you could think of.
> +
> +         All Tini does is spawn a single child (Tini is meant to be run
> +         in a container), and wait for it to exit all the while reaping
> +         zombies and performing signal forwarding.
> +
> +         https://github.com/krallin/tini
> +
>  config BR2_INIT_NONE
>         bool "Custom (none)"
>         select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
> --
> 2.46.0
>

Reviewed-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

Regards,
Dario

-- 

Dario Binacchi

Senior Embedded Linux Developer

dario.binacchi@amarulasolutions.com

__________________________________


Amarula Solutions SRL

Via Le Canevare 30, 31100 Treviso, Veneto, IT

T. +39 042 243 5310
info@amarulasolutions.com

www.amarulasolutions.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 3/3] system: add tini as init system

[TIAN Yuanhao]

Hi Yann,

At 2024-09-14 17:04:26, "Dario Binacchi" <dario.binacchi@amarulasolutions.com> wrote:
>Hi Yann,
>
>On Fri, Aug 30, 2024 at 10:26 PM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
>>
>> Tini is a special-purpose init system, aimed at containers.
>>
>> Add it to the choice of init systems.
>>
>> tini does not require much, so we re-use the default "none" skeleton.
>>
>> Note that tini is no longer selectable on its own, which it used to be.
>> This change will require that people using tini be careful when
>> upgrading, though, but that will be very easily caught, as the
>> containers would no longer start at all.
>>
>> We could ad a comment stating why, but no other init system has such a
>> comment, so adding one for tini alone would be weird, so we don't add
>> one.
>>
>> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
>> Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
>> ---
>>  package/tini/Config.in |  1 +
>>  system/Config.in       | 16 ++++++++++++++++
>>  2 files changed, 17 insertions(+)
>>
>> diff --git a/package/tini/Config.in b/package/tini/Config.in
>> index 719e1e1f22..bca8ad8e96 100644
>> --- a/package/tini/Config.in
>> +++ b/package/tini/Config.in
>> @@ -1,6 +1,7 @@
>>  config BR2_PACKAGE_TINI
>>         bool "tini"
>>         depends on BR2_USE_MMU # fork()
>> +       depends on BR2_INIT_TINI
>>         help
>>           tini is a simple but valid init binary to
>>           act as PID 1 for containers.
>> diff --git a/system/Config.in b/system/Config.in
>> index 18cc39ad94..d7d59db47d 100644
>> --- a/system/Config.in
>> +++ b/system/Config.in
>> @@ -149,6 +149,22 @@ comment "systemd highly recommends Linux >= 4.15"
>>
>>  comment "Special purpose (read help)"
>>
>> +config BR2_INIT_TINI
>> +       bool "tini"
>> +       depends on BR2_USE_MMU  # tini
>> +       select BR2_PACKAGE_TINI
>> +       select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
>> +       help
>> +         Tini - A tiny but valid init for containers
>> +
>> +         Tini is the simplest init you could think of.
>> +
>> +         All Tini does is spawn a single child (Tini is meant to be run
>> +         in a container), and wait for it to exit all the while reaping
>> +         zombies and performing signal forwarding.
>> +
>> +         https://github.com/krallin/tini
>> +
>>  config BR2_INIT_NONE
>>         bool "Custom (none)"
>>         select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
>> --
>> 2.46.0
>>
>
>Reviewed-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>
>Regards,
>Dario
>

I think this is completely **wrong**.

People usually do not put docker-init (such as tini/catatonit) in container images, but in the host system, and use it by `docker run --init`. Only inits such as supervisor/s6 will be placed in container images.

Regards,
TIAN Yuanhao
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 3/3] system: add tini as init system

[yann.morin]

TIAN Yuanhao, All,

On 2024-09-18 16:02 +0800, TIAN Yuanhao spake thusly:
> At 2024-09-14 17:04:26, "Dario Binacchi" <dario.binacchi@amarulasolutions.com> wrote:
> >On Fri, Aug 30, 2024 at 10:26 PM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> >> Tini is a special-purpose init system, aimed at containers.
> >> Add it to the choice of init systems.
[--SNIP--]
> I think this is completely **wrong**.
> People usually do not put docker-init (such as tini/catatonit) in
> container images, but in the host system, and use it by
> `docker run --init`.

The container world does not revolve only around docker; there are other
solutions.

We do have a situation where the container runtime will expect us to
actually provide our own PID-1 that is a proper reaper. In that
situation, we do need to have tini in the container and running as the
entrypoint.

Note that tini also works if it is not PID-1; in that case, it uses the
subreaper feature (since kernel 3.4, ages ago now!).

You mention catatonit. I was not aware about it, and neither that it
was packaged in Buildroot, but then it might make sense to also add
that to the list, now.

If one still wants to delegate the PID-1 reaper to a docker-provided
tini (or alternative), then it is still possible to select the "Custom
(none)" entry in the choice.

Regards,
Yann E. MORIN.

-- 
                                        ____________
.-----------------.--------------------:       _    :------------------.
|  Yann E. MORIN  | Real-Time Embedded |    __/ )   | /"\ ASCII RIBBON |
|                 | Software  Designer |  _/ - /'   | \ / CAMPAIGN     |
| +33 638.411.245 '--------------------: (_    `--, |  X  AGAINST      |
| yann.morin (at) orange.com           |_="    ,--' | / \ HTML MAIL    |
'--------------------------------------:______/_____:------------------'

____________________________________________________________________________________________________________
Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 0/3] system: extend the list of init systems (branch yem/init-choice)

[Yann E. MORIN]

All,

On 2024-08-30 22:26 +0200, Yann E. MORIN spake thusly:
> Hello All!
> 
> This little series is a proposal to follow on the discussion with Dario
> and Thomas, about Dario's series adding tiny-init:
>     https://lore.kernel.org/buildroot/ZtF1pHPZqUkUit8N@landeda/T/#mc1ef6288de44b9d35d382187ae14edf3185aa0c5
> 
> Note that, contrary to what was said in that discussion, there is no
> need to add a build dependency from Busybox to tini, as tini does not
> install itself as /init or /sbin/init, so will not be conflicting with
> Busybox.
> 
> Regards,
> Yann E. MORIN.
> 
> 
> ----------------------------------------------------------------
> Yann E. MORIN (3):
>       system: change label for BR2_INIT_NONE
>       system: classify init systems
>       system: add tini as init system

As suggested by Thomas, series applied to master with reviews from
Dario, thanks!

Regards,
Yann E. MORIN.

>  package/tini/Config.in |  1 +
>  system/Config.in       | 22 +++++++++++++++++++++-
>  2 files changed, 22 insertions(+), 1 deletion(-)
> 
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot