On Thu, Apr 11, 2019 at 3:29 AM Beniamin Sandu <[email protected]> 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@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@qemux86:~# insmod hello.ko > [ 39.094288] hello: loading out-of-tree module taints kernel. > [ 39.108044] Hello World! > > Signed-off-by: Beniamin Sandu <[email protected]>
Looks good to me. Acked-by: Bruce Ashfield <[email protected]> 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 > [email protected] > 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 -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
