* [PATCH] kern-devsrc: Drop auto.conf creation @ 2022-08-09 11:14 Richard Purdie 2022-08-09 13:16 ` Bruce Ashfield 0 siblings, 1 reply; 7+ messages in thread From: Richard Purdie @ 2022-08-09 11:14 UTC (permalink / raw) To: openembedded-core; +Cc: Bruce Ashfield After this commit in the mainline kernel, the quoting is different in auto.conf compared to .config: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b We therefore can't just copy one over the other or it leads to syntax/quoting errors. We end up running make prepare anyway in all our tests so drop this copy for fix 5.19 kernels. Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> --- meta/recipes-kernel/linux/kernel-devsrc.bb | 3 --- 1 file changed, 3 deletions(-) diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb b/meta/recipes-kernel/linux/kernel-devsrc.bb index a38d35e82ae..46d706b955f 100644 --- a/meta/recipes-kernel/linux/kernel-devsrc.bb +++ b/meta/recipes-kernel/linux/kernel-devsrc.bb @@ -308,9 +308,6 @@ do_install() { # external modules can be built touch -r $kerneldir/build/Makefile $kerneldir/build/include/generated/uapi/linux/version.h - # Copy .config to include/config/auto.conf so "make prepare" is unnecessary. - cp $kerneldir/build/.config $kerneldir/build/include/config/auto.conf - # make sure these are at least as old as the .config, or rebuilds will trigger touch -r $kerneldir/build/.config $kerneldir/build/include/generated/autoconf.h 2>/dev/null || : touch -r $kerneldir/build/.config $kerneldir/build/include/config/auto.conf* 2>/dev/null || : -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 11:14 [PATCH] kern-devsrc: Drop auto.conf creation Richard Purdie @ 2022-08-09 13:16 ` Bruce Ashfield 2022-08-09 13:26 ` Richard Purdie 0 siblings, 1 reply; 7+ messages in thread From: Bruce Ashfield @ 2022-08-09 13:16 UTC (permalink / raw) To: Richard Purdie; +Cc: openembedded-core On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > > After this commit in the mainline kernel, the quoting is different in auto.conf > compared to .config: > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > We therefore can't just copy one over the other or it leads to syntax/quoting > errors. We end up running make prepare anyway in all our tests so drop > this copy for fix 5.19 kernels. It is safe to stop doing this manipulation now. But I am curious, I was running on-target tests for 5.19+ and didn't have any issues, what is breaking that I managed to miss ? There is a risk that if the .cmd files start regenerating because of this missing, or the different timestamp, that we'll have issues. But putting it through the AB should show us if that is true. Bruce > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > --- > meta/recipes-kernel/linux/kernel-devsrc.bb | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb b/meta/recipes-kernel/linux/kernel-devsrc.bb > index a38d35e82ae..46d706b955f 100644 > --- a/meta/recipes-kernel/linux/kernel-devsrc.bb > +++ b/meta/recipes-kernel/linux/kernel-devsrc.bb > @@ -308,9 +308,6 @@ do_install() { > # external modules can be built > touch -r $kerneldir/build/Makefile $kerneldir/build/include/generated/uapi/linux/version.h > > - # Copy .config to include/config/auto.conf so "make prepare" is unnecessary. > - cp $kerneldir/build/.config $kerneldir/build/include/config/auto.conf > - > # make sure these are at least as old as the .config, or rebuilds will trigger > touch -r $kerneldir/build/.config $kerneldir/build/include/generated/autoconf.h 2>/dev/null || : > touch -r $kerneldir/build/.config $kerneldir/build/include/config/auto.conf* 2>/dev/null || : > -- > 2.34.1 > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 13:16 ` Bruce Ashfield @ 2022-08-09 13:26 ` Richard Purdie 2022-08-09 13:29 ` Bruce Ashfield 0 siblings, 1 reply; 7+ messages in thread From: Richard Purdie @ 2022-08-09 13:26 UTC (permalink / raw) To: Bruce Ashfield; +Cc: openembedded-core On Tue, 2022-08-09 at 09:16 -0400, Bruce Ashfield wrote: > On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie > <richard.purdie@linuxfoundation.org> wrote: > > > > After this commit in the mainline kernel, the quoting is different in auto.conf > > compared to .config: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > > > We therefore can't just copy one over the other or it leads to syntax/quoting > > errors. We end up running make prepare anyway in all our tests so drop > > this copy for fix 5.19 kernels. > > It is safe to stop doing this manipulation now. > > But I am curious, I was running on-target tests for 5.19+ and didn't > have any issues, what is breaking that I managed to miss ? https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/5629/steps/13/logs/stdio Basically any kernelmodule.KernelModuleTest.test_kernel_module which was run was failing (which would run on any image with kernel-devsrc in and a toolchain). It failed everywhere, all sdk qemu machines equally. > There is a risk that if the .cmd files start regenerating because of > this missing, or the different timestamp, that we'll have issues. But > putting it through the AB should show us if that is true. After this change is applied, the build looks a lot greener and this issue goes away. Cheers, Richard ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 13:26 ` Richard Purdie @ 2022-08-09 13:29 ` Bruce Ashfield 2022-08-09 14:08 ` Richard Purdie 0 siblings, 1 reply; 7+ messages in thread From: Bruce Ashfield @ 2022-08-09 13:29 UTC (permalink / raw) To: Richard Purdie; +Cc: openembedded-core On Tue, Aug 9, 2022 at 9:26 AM Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > > On Tue, 2022-08-09 at 09:16 -0400, Bruce Ashfield wrote: > > On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > After this commit in the mainline kernel, the quoting is different in auto.conf > > > compared to .config: > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > > > > > We therefore can't just copy one over the other or it leads to syntax/quoting > > > errors. We end up running make prepare anyway in all our tests so drop > > > this copy for fix 5.19 kernels. > > > > It is safe to stop doing this manipulation now. > > > > But I am curious, I was running on-target tests for 5.19+ and didn't > > have any issues, what is breaking that I managed to miss ? > > https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/5629/steps/13/logs/stdio > > Basically any kernelmodule.KernelModuleTest.test_kernel_module which > was run was failing (which would run on any image with kernel-devsrc in > and a toolchain). It failed everywhere, all sdk qemu machines equally. odd. I never did see that same error, and I built and installed kernel module for all the architectures. Bruce > > > There is a risk that if the .cmd files start regenerating because of > > this missing, or the different timestamp, that we'll have issues. But > > putting it through the AB should show us if that is true. > > After this change is applied, the build looks a lot greener and this > issue goes away. > > Cheers, > > Richard -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 13:29 ` Bruce Ashfield @ 2022-08-09 14:08 ` Richard Purdie 2022-08-09 14:22 ` Bruce Ashfield 0 siblings, 1 reply; 7+ messages in thread From: Richard Purdie @ 2022-08-09 14:08 UTC (permalink / raw) To: Bruce Ashfield; +Cc: openembedded-core On Tue, 2022-08-09 at 09:29 -0400, Bruce Ashfield wrote: > On Tue, Aug 9, 2022 at 9:26 AM Richard Purdie > <richard.purdie@linuxfoundation.org> wrote: > > > > On Tue, 2022-08-09 at 09:16 -0400, Bruce Ashfield wrote: > > > On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie > > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > > > After this commit in the mainline kernel, the quoting is different in auto.conf > > > > compared to .config: > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > > > > > > > We therefore can't just copy one over the other or it leads to syntax/quoting > > > > errors. We end up running make prepare anyway in all our tests so drop > > > > this copy for fix 5.19 kernels. > > > > > > It is safe to stop doing this manipulation now. > > > > > > But I am curious, I was running on-target tests for 5.19+ and didn't > > > have any issues, what is breaking that I managed to miss ? > > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/5629/steps/13/logs/stdio > > > > Basically any kernelmodule.KernelModuleTest.test_kernel_module which > > was run was failing (which would run on any image with kernel-devsrc in > > and a toolchain). It failed everywhere, all sdk qemu machines equally. > > odd. I never did see that same error, and I built and installed kernel > module for all the architectures. Did you use the oeqa code or something else? It seems odd you didn't see it as it was quite consistent on my local system and the autobuilders :/ Cheers, Richard ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 14:08 ` Richard Purdie @ 2022-08-09 14:22 ` Bruce Ashfield 2022-08-09 14:26 ` Richard Purdie 0 siblings, 1 reply; 7+ messages in thread From: Bruce Ashfield @ 2022-08-09 14:22 UTC (permalink / raw) To: Richard Purdie; +Cc: openembedded-core On Tue, Aug 9, 2022 at 10:08 AM Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > > On Tue, 2022-08-09 at 09:29 -0400, Bruce Ashfield wrote: > > On Tue, Aug 9, 2022 at 9:26 AM Richard Purdie > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > On Tue, 2022-08-09 at 09:16 -0400, Bruce Ashfield wrote: > > > > On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie > > > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > > > > > After this commit in the mainline kernel, the quoting is different in auto.conf > > > > > compared to .config: > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > > > > > > > > > We therefore can't just copy one over the other or it leads to syntax/quoting > > > > > errors. We end up running make prepare anyway in all our tests so drop > > > > > this copy for fix 5.19 kernels. > > > > > > > > It is safe to stop doing this manipulation now. > > > > > > > > But I am curious, I was running on-target tests for 5.19+ and didn't > > > > have any issues, what is breaking that I managed to miss ? > > > > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/5629/steps/13/logs/stdio > > > > > > Basically any kernelmodule.KernelModuleTest.test_kernel_module which > > > was run was failing (which would run on any image with kernel-devsrc in > > > and a toolchain). It failed everywhere, all sdk qemu machines equally. > > > > odd. I never did see that same error, and I built and installed kernel > > module for all the architectures. > > Did you use the oeqa code or something else? It seems odd you didn't > see it as it was quite consistent on my local system and the > autobuilders :/ I run everything by hand, oeqa has always taken too long in my infrastructure, so I run the kernel tests myself. Bruce > > Cheers, > > Richard > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kern-devsrc: Drop auto.conf creation 2022-08-09 14:22 ` Bruce Ashfield @ 2022-08-09 14:26 ` Richard Purdie 0 siblings, 0 replies; 7+ messages in thread From: Richard Purdie @ 2022-08-09 14:26 UTC (permalink / raw) To: Bruce Ashfield; +Cc: openembedded-core On Tue, 2022-08-09 at 10:22 -0400, Bruce Ashfield wrote: > On Tue, Aug 9, 2022 at 10:08 AM Richard Purdie > <richard.purdie@linuxfoundation.org> wrote: > > > > On Tue, 2022-08-09 at 09:29 -0400, Bruce Ashfield wrote: > > > On Tue, Aug 9, 2022 at 9:26 AM Richard Purdie > > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > > > On Tue, 2022-08-09 at 09:16 -0400, Bruce Ashfield wrote: > > > > > On Tue, Aug 9, 2022 at 7:14 AM Richard Purdie > > > > > <richard.purdie@linuxfoundation.org> wrote: > > > > > > > > > > > > After this commit in the mainline kernel, the quoting is different in auto.conf > > > > > > compared to .config: > > > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=129ab0d2d9f38b9d43df35235fc66c6740d6928b > > > > > > > > > > > > We therefore can't just copy one over the other or it leads to syntax/quoting > > > > > > errors. We end up running make prepare anyway in all our tests so drop > > > > > > this copy for fix 5.19 kernels. > > > > > > > > > > It is safe to stop doing this manipulation now. > > > > > > > > > > But I am curious, I was running on-target tests for 5.19+ and didn't > > > > > have any issues, what is breaking that I managed to miss ? > > > > > > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/5629/steps/13/logs/stdio > > > > > > > > Basically any kernelmodule.KernelModuleTest.test_kernel_module which > > > > was run was failing (which would run on any image with kernel-devsrc in > > > > and a toolchain). It failed everywhere, all sdk qemu machines equally. > > > > > > odd. I never did see that same error, and I built and installed kernel > > > module for all the architectures. > > > > Did you use the oeqa code or something else? It seems odd you didn't > > see it as it was quite consistent on my local system and the > > autobuilders :/ > > I run everything by hand, oeqa has always taken too long in my > infrastructure, so I run the kernel tests myself. You could do something like: IMAGE_CLASSES += "testimage" TEST_SUTITES = "ping ssh kernelmodule" and then bitbake core-image-sato-sdk bitbake core-image-sato-sdk -c testimage just to run that one. (I might have missed a test dependency which would need to be added to the list in TEST_SUITES but you get the idea) Cheers, Richard ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-08-09 14:26 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-08-09 11:14 [PATCH] kern-devsrc: Drop auto.conf creation Richard Purdie 2022-08-09 13:16 ` Bruce Ashfield 2022-08-09 13:26 ` Richard Purdie 2022-08-09 13:29 ` Bruce Ashfield 2022-08-09 14:08 ` Richard Purdie 2022-08-09 14:22 ` Bruce Ashfield 2022-08-09 14:26 ` Richard Purdie
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).