On Thu, Jan 26, 2023 at 5:10 PM Richard Purdie
<richard.pur...@linuxfoundation.org> wrote:
>
> With the 6.1 kernel we've seen reproducibility/determinism issues where the 
> kernel
> seems to rebuild headers referencing the hostname or local user. kernel-devsrc
> building after the kernel seemed to trigger it in some cases.
>
> Moving the definitions to the bbclass used by all the kernel recipe code 
> including
> kernel-devsrc seems to be the best way to ensure this doesn't happen.
>

This is the right thing to do.

I haven't been able to root cause how the 6.1 kernel file generation was
being triggered by devsrc, and how it raced with the main kernel to change
the build-artifacts .. but it most definitely did do it.

At the very least, this rules out a lot of paths to the generation  of the
header files, and if it happens again, it will be easier to track down.

Bruce

> Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
> ---
>  meta/classes-recipe/kernel.bbclass            | 3 ---
>  meta/classes-recipe/linux-kernel-base.bbclass | 4 ++++
>  2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/meta/classes-recipe/kernel.bbclass 
> b/meta/classes-recipe/kernel.bbclass
> index bd3b6471c6b..4fbe84c4e42 100644
> --- a/meta/classes-recipe/kernel.bbclass
> +++ b/meta/classes-recipe/kernel.bbclass
> @@ -210,9 +210,6 @@ PACKAGES_DYNAMIC += "^${KERNEL_PACKAGE_NAME}-firmware-.*"
>
>  export OS = "${TARGET_OS}"
>  export CROSS_COMPILE = "${TARGET_PREFIX}"
> -export KBUILD_BUILD_VERSION = "1"
> -export KBUILD_BUILD_USER ?= "oe-user"
> -export KBUILD_BUILD_HOST ?= "oe-host"
>
>  KERNEL_RELEASE ?= "${KERNEL_VERSION}"
>
> diff --git a/meta/classes-recipe/linux-kernel-base.bbclass 
> b/meta/classes-recipe/linux-kernel-base.bbclass
> index cb2212c9485..65cc48f304d 100644
> --- a/meta/classes-recipe/linux-kernel-base.bbclass
> +++ b/meta/classes-recipe/linux-kernel-base.bbclass
> @@ -43,5 +43,9 @@ def linux_module_packages(s, d):
>      suffix = ""
>      return " ".join(map(lambda s: "kernel-module-%s%s" % 
> (s.lower().replace('_', '-').replace('@', '+'), suffix), s.split()))
>
> +export KBUILD_BUILD_VERSION = "1"
> +export KBUILD_BUILD_USER ?= "oe-user"
> +export KBUILD_BUILD_HOST ?= "oe-host"
> +
>  # that's all
>
> --
> 2.37.2
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#176432): 
https://lists.openembedded.org/g/openembedded-core/message/176432
Mute This Topic: https://lists.openembedded.org/mt/96554078/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to