Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

[Arago Project git]

Module: meta-ti
Branch: fido
Commit: 4f542c3fe47a8c6be195b3c97fd83efbf930bb70
URL:    http://arago-project.org/git/meta-ti.git?a=commit;h=4f542c3fe47a8c6be195b3c97fd83efbf930bb70

Author: Sam Nelson <sam.nelson@ti.com>
Date:   Mon Dec 14 17:47:48 2015 +0000

linux-ti-staging: Work around to avoid race condition with external module

- Currently there is an issue with compiling external kernel module
which depends on symbols from another kernel module which is part of the
list of modules configured to be built as modules in the default config.
- This work around allows to avoid the issue by copying the module
symbols to the kernel directory after making the kernel modules build.
- This issue is already fixed in oe-core: master branch as of date
- This is just work around before updating to latest oe-core

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
Signed-off-by: Denys Dmytriyenko <denys@ti.com>

---

 recipes-kernel/linux/linux-ti-staging_3.14.bb |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/recipes-kernel/linux/linux-ti-staging_3.14.bb b/recipes-kernel/linux/linux-ti-staging_3.14.bb
index 54530e4..88950ae 100644
--- a/recipes-kernel/linux/linux-ti-staging_3.14.bb
+++ b/recipes-kernel/linux/linux-ti-staging_3.14.bb
@@ -65,3 +65,7 @@ KERNEL_GIT_PROTOCOL = "git"
 SRC_URI += "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH} \
             file://defconfig \
            "
+
+do_compile_kernelmodules_append() {
+	cp Module.symvers ${STAGING_KERNEL_BUILDDIR}/
+}

Re: Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

[Denys Dmytriyenko]

Sam,

Is this still a problem on 4.1 kernel? Do we need to copy this work around to 
4.1 recipe in Fido branch?


On Fri, Dec 18, 2015 at 01:07:23AM +0000, Arago Project git wrote:
> Module: meta-ti
> Branch: fido
> Commit: 4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> URL:    http://arago-project.org/git/meta-ti.git?a=commit;h=4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> 
> Author: Sam Nelson <sam.nelson@ti.com>
> Date:   Mon Dec 14 17:47:48 2015 +0000
> 
> linux-ti-staging: Work around to avoid race condition with external module
> 
> - Currently there is an issue with compiling external kernel module
> which depends on symbols from another kernel module which is part of the
> list of modules configured to be built as modules in the default config.
> - This work around allows to avoid the issue by copying the module
> symbols to the kernel directory after making the kernel modules build.
> - This issue is already fixed in oe-core: master branch as of date
> - This is just work around before updating to latest oe-core
> 
> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> 
> ---
> 
>  recipes-kernel/linux/linux-ti-staging_3.14.bb |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/recipes-kernel/linux/linux-ti-staging_3.14.bb b/recipes-kernel/linux/linux-ti-staging_3.14.bb
> index 54530e4..88950ae 100644
> --- a/recipes-kernel/linux/linux-ti-staging_3.14.bb
> +++ b/recipes-kernel/linux/linux-ti-staging_3.14.bb
> @@ -65,3 +65,7 @@ KERNEL_GIT_PROTOCOL = "git"
>  SRC_URI += "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH} \
>              file://defconfig \
>             "
> +
> +do_compile_kernelmodules_append() {
> +	cp Module.symvers ${STAGING_KERNEL_BUILDDIR}/
> +}
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti

Re: Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

[Nelson, Sam]

Yes, this workaround needs to be copied to 4.1 recipe as well.  (As the problem itself will still exist in fido and the issue is not kernel specific)
With regards,
Sam

> -----Original Message-----
> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> Sent: Monday, January 25, 2016 8:00 PM
> To: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] Sam Nelson : linux-ti-staging: Work around to avoid race
> condition with external module
> 
> Sam,
> 
> Is this still a problem on 4.1 kernel? Do we need to copy this work around to
> 4.1 recipe in Fido branch?
> 
> 
> On Fri, Dec 18, 2015 at 01:07:23AM +0000, Arago Project git wrote:
> > Module: meta-ti
> > Branch: fido
> > Commit: 4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> > URL:    http://arago-project.org/git/meta-
> ti.git?a=commit;h=4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> >
> > Author: Sam Nelson <sam.nelson@ti.com>
> > Date:   Mon Dec 14 17:47:48 2015 +0000
> >
> > linux-ti-staging: Work around to avoid race condition with external module
> >
> > - Currently there is an issue with compiling external kernel module
> > which depends on symbols from another kernel module which is part of the
> > list of modules configured to be built as modules in the default config.
> > - This work around allows to avoid the issue by copying the module
> > symbols to the kernel directory after making the kernel modules build.
> > - This issue is already fixed in oe-core: master branch as of date
> > - This is just work around before updating to latest oe-core
> >
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >
> > ---
> >
> >  recipes-kernel/linux/linux-ti-staging_3.14.bb |    4 ++++
> >  1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/recipes-kernel/linux/linux-ti-staging_3.14.bb b/recipes-
> kernel/linux/linux-ti-staging_3.14.bb
> > index 54530e4..88950ae 100644
> > --- a/recipes-kernel/linux/linux-ti-staging_3.14.bb
> > +++ b/recipes-kernel/linux/linux-ti-staging_3.14.bb
> > @@ -65,3 +65,7 @@ KERNEL_GIT_PROTOCOL = "git"
> >  SRC_URI +=
> "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH}
> \
> >              file://defconfig \
> >             "
> > +
> > +do_compile_kernelmodules_append() {
> > +	cp Module.symvers ${STAGING_KERNEL_BUILDDIR}/
> > +}
> >
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
> --
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti

Re: Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

[Denys Dmytriyenko]

Thanks! Would you be submitting the corresponding patch? :)


On Wed, Jan 27, 2016 at 01:54:43PM -0500, Nelson, Sam wrote:
> Yes, this workaround needs to be copied to 4.1 recipe as well.  (As the 
> problem itself will still exist in fido and the issue is not kernel 
> specific)
> With regards,
> Sam
> 
> > -----Original Message-----
> > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > Sent: Monday, January 25, 2016 8:00 PM
> > To: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] Sam Nelson : linux-ti-staging: Work around to avoid race
> > condition with external module
> > 
> > Sam,
> > 
> > Is this still a problem on 4.1 kernel? Do we need to copy this work around to
> > 4.1 recipe in Fido branch?
> > 
> > 
> > On Fri, Dec 18, 2015 at 01:07:23AM +0000, Arago Project git wrote:
> > > Module: meta-ti
> > > Branch: fido
> > > Commit: 4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> > > URL:    http://arago-project.org/git/meta-
> > ti.git?a=commit;h=4f542c3fe47a8c6be195b3c97fd83efbf930bb70
> > >
> > > Author: Sam Nelson <sam.nelson@ti.com>
> > > Date:   Mon Dec 14 17:47:48 2015 +0000
> > >
> > > linux-ti-staging: Work around to avoid race condition with external module
> > >
> > > - Currently there is an issue with compiling external kernel module
> > > which depends on symbols from another kernel module which is part of the
> > > list of modules configured to be built as modules in the default config.
> > > - This work around allows to avoid the issue by copying the module
> > > symbols to the kernel directory after making the kernel modules build.
> > > - This issue is already fixed in oe-core: master branch as of date
> > > - This is just work around before updating to latest oe-core
> > >
> > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > >
> > > ---
> > >
> > >  recipes-kernel/linux/linux-ti-staging_3.14.bb |    4 ++++
> > >  1 files changed, 4 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/recipes-kernel/linux/linux-ti-staging_3.14.bb b/recipes-
> > kernel/linux/linux-ti-staging_3.14.bb
> > > index 54530e4..88950ae 100644
> > > --- a/recipes-kernel/linux/linux-ti-staging_3.14.bb
> > > +++ b/recipes-kernel/linux/linux-ti-staging_3.14.bb
> > > @@ -65,3 +65,7 @@ KERNEL_GIT_PROTOCOL = "git"
> > >  SRC_URI +=
> > "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH}
> > \
> > >              file://defconfig \
> > >             "
> > > +
> > > +do_compile_kernelmodules_append() {
> > > +	cp Module.symvers ${STAGING_KERNEL_BUILDDIR}/
> > > +}
> > >
> > > --
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti

Sam Nelson : linux-ti-staging: Work around to avoid race condition with external module

[Arago Project git]

Module: meta-ti
Branch: fido
Commit: a34131462270ed70116d264495e8d13586956b6e
URL:    http://arago-project.org/git/meta-ti.git?a=commit;h=a34131462270ed70116d264495e8d13586956b6e

Author: Sam Nelson <sam.nelson@ti.com>
Date:   Fri Jan 29 02:29:41 2016 +0000

linux-ti-staging: Work around to avoid race condition with external module

Currently there is an issue with compiling external kernel module
which depends on symbols from another kernel module which is part of the
list of modules configured to be built as modules in the default config.

This work around allows to avoid the issue by copying the module
symbols to the kernel directory after making the kernel modules build.

This issue is already fixed in oe-core: master branch as of date
 - this is just work around before updating to latest oe-core

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
Signed-off-by: Denys Dmytriyenko <denys@ti.com>

---

 recipes-kernel/linux/linux-ti-staging_4.1.bb |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/recipes-kernel/linux/linux-ti-staging_4.1.bb b/recipes-kernel/linux/linux-ti-staging_4.1.bb
index 59805c6..dfe9a44 100644
--- a/recipes-kernel/linux/linux-ti-staging_4.1.bb
+++ b/recipes-kernel/linux/linux-ti-staging_4.1.bb
@@ -84,3 +84,7 @@ KERNEL_GIT_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git"
 KERNEL_GIT_PROTOCOL = "git"
 SRC_URI += "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH} \
             file://defconfig"
+
+do_compile_kernelmodules_append() {
+	cp Module.symvers ${STAGING_KERNEL_BUILDDIR}/
+}