[Buildroot] [PATCH v2] busybox: S01mdev: fix module autoloading
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Tue May 8 13:04:10 UTC 2018
Hello,
On Mon, 7 May 2018 20:35:01 +0200, Peter Korsgaard wrote:
> Commit b4fc5a180c (package/busybox: support spaces in module aliases in
> mdev) changed the mdev coldplugging to handle sysfs path elements and
> modalias values containing spaces. This unfortunately doesn't work as was
> recently reported:
>
> http://lists.busybox.net/pipermail/buildroot/2018-May/220903.html
>
> The problem is that sort -z also expects the fields of the input files to be
> zero terminated, which is not the case for modalias sysfs entries.
>
> So drop the -z option to sort. Spaces in modalias entries could be handled
> with the xargs -d '\n' option, but that is unfortunately not supported by
> the busybox applet. Instead, use tr to convert newlines to zeros so we can
> use xargs -0.
>
> Reported-by: Daniel Palmer <daniel at 0x0f.com>
> Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
> ---
> package/busybox/S10mdev | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list