mbox series

[0/3] Update Busybox to 1.31 and use mdev daemon mode

Message ID 20190619164257.183607-1-titouan.christophe@railnova.eu
Headers show
Series Update Busybox to 1.31 and use mdev daemon mode | expand

Message

Titouan Christophe June 19, 2019, 4:42 p.m. UTC 
The latest Busybox release introduces the daemon mode
for mdev, which can be used as a replacement for the
uevent+mdev setup, or the old kernel hotplug mechanism.

This new mode prevent race conditions in mdev invocations,
but on the other hand it requires CONFIG_NET to be enabled
in the kernel.

In this patchset, Busybox is bumped to the latest version,
the default configuration is updated to support the mdev
daemon mode, and the mdev init script is updated accordingly

Titouan Christophe (3):
  package/busybox: bump version to 1.31.0
  package/busybox: convert S10mdev to the canonical init script format
  package/busybox: run mdev in daemon mode

 ...-scripted_main-as-externally-visible.patch | 53 -------------------
 package/busybox/S10mdev                       | 42 ++++++++++-----
 package/busybox/busybox.config                |  1 +
 package/busybox/busybox.hash                  |  4 +-
 package/busybox/busybox.mk                    |  2 +-
 5 files changed, 34 insertions(+), 68 deletions(-)
 delete mode 100644 package/busybox/0003-libbb-mark-scripted_main-as-externally-visible.patch

Comments

Thomas Petazzoni June 20, 2019, 2:31 p.m. UTC | #1 
On Wed, 19 Jun 2019 18:42:54 +0200
Titouan Christophe <titouan.christophe@railnova.eu> wrote:

> Titouan Christophe (3):
>   package/busybox: bump version to 1.31.0
>   package/busybox: convert S10mdev to the canonical init script format
>   package/busybox: run mdev in daemon mode

Series applied. Thanks!

Thomas
Peter Korsgaard June 20, 2019, 3:16 p.m. UTC | #2 
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

 > On Wed, 19 Jun 2019 18:42:54 +0200
 > Titouan Christophe <titouan.christophe@railnova.eu> wrote:

 >> Titouan Christophe (3):
 >> package/busybox: bump version to 1.31.0
 >> package/busybox: convert S10mdev to the canonical init script format
 >> package/busybox: run mdev in daemon mode

 > Series applied. Thanks!

Yes, thanks. We still need to force CONFIG_NET=y in the Linux kernel if
mdev is enabled, right?
Titouan Christophe June 20, 2019, 4:02 p.m. UTC | #3 
Hello all,

On 6/20/19 5:16 PM, Peter Korsgaard wrote:
>>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:
> 
>   > On Wed, 19 Jun 2019 18:42:54 +0200
>   > Titouan Christophe <titouan.christophe@railnova.eu> wrote:
> 
>   >> Titouan Christophe (3):
>   >> package/busybox: bump version to 1.31.0
>   >> package/busybox: convert S10mdev to the canonical init script format
>   >> package/busybox: run mdev in daemon mode
> 
>   > Series applied. Thanks!
> 
> Yes, thanks. We still need to force CONFIG_NET=y in the Linux kernel if
> mdev is enabled, right?
> 

Yes, since mdev is now launched in daemon mode CONFIG_NET is now needed. 
Didn't think of enforcing that in the kernel config, I'm preparing a 
patch right now.

Regards,

Titouan
Titouan Christophe June 23, 2019, 1:20 p.m. UTC | #4 
Hi all,

On 6/20/19 6:02 PM, Titouan Christophe wrote:
> Hello all,
> 
> On 6/20/19 5:16 PM, Peter Korsgaard wrote:
>> Yes, thanks. We still need to force CONFIG_NET=y in the Linux kernel if
>> mdev is enabled, right?
>>
> 
> Yes, since mdev is now launched in daemon mode CONFIG_NET is now needed. 
> Didn't think of enforcing that in the kernel config, I'm preparing a 
> patch right now.
> 
> Regards,
> 
> Titouan

The patch has been submitted here 
http://patchwork.ozlabs.org/patch/1119590/.

Titouan