[OE-core] [[PATCH v2] kernel-devsrc: check for localversion files in the kernel source tree

[Bruce Ashfield]

On Thu, Apr 11, 2019 at 3:29 AM Beniamin Sandu <beniaminsandu at gmail.com> wrote:
>
> The kernel-devsrc rework doesn't take into account localversion
> files in the kernel source tree. This might result in modules with an
> incomplete version magic when building out-of-tree using the SDK.
>
> Before:
> root at qemux86:~# insmod hello.ko
> [   42.000212] hello: version magic '5.0.3-yocto-standard SMP preempt
> mod_unload PENTIUMM ' should be '5.0.3-my-custom-release-yocto-standard
> SMP preempt mod_unload PENTIUMM '
> [   42.008200] hello: version magic '5.0.3-yocto-standard SMP preempt
> mod_unload PENTIUMM ' should be '5.0.3-my-custom-release-yocto-standard
> SMP preempt mod_unload PENTIUMM '
> insmod: can't insert 'hello.ko': invalid module format
>
> After:
> root at qemux86:~# insmod hello.ko
> [   39.094288] hello: loading out-of-tree module taints kernel.
> [   39.108044] Hello World!
>
> Signed-off-by: Beniamin Sandu <beniamin.sandu at gmail.com>

Looks good to me.

Acked-by: Bruce Ashfield <bruce.ashfield at gmail.com>

Richard, I assume you'll pick this up directly and don't want to wait
for me to queue and send it along with my next pull request.

Bruce

> ---
>  meta/recipes-kernel/linux/kernel-devsrc.bb | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb b/meta/recipes-kernel/linux/kernel-devsrc.bb
> index 69a8eaef81..5ec5929735 100644
> --- a/meta/recipes-kernel/linux/kernel-devsrc.bb
> +++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
> @@ -228,6 +228,9 @@ do_install() {
>
>          # required to build scripts/selinux/genheaders/genheaders
>          cp -a --parents security/selinux/include/* $kerneldir/build/
> +
> +       # copy any localversion files
> +       cp -a localversion* $kerneldir/build/ 2>/dev/null || :
>      )
>
>      # Make sure the Makefile and version.h have a matching timestamp so that
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II