[OE-core] [PATCH] wic: Use custom kernel path if provided
Richard,
I don't think there are any additional test cases that need to be created for
this condition. It is a fairly unique set of circumstances that cause an error
in our build system and would be difficult to reproduce outside of it.
As for your concern about this being the right thing to do, honestly I'm not
sure, but it does seem wrong to me that there is a code path that ignores the
value in options.kernel_dir, which is why I proposed the patch (and of course
our build system was being affected by it). I'm not sure if it is right or
not, but it seems "more right" to me.
Hope that helps,
Bill
From: Richard Purdie
Sent: Wednesday, March 2, 2022 5:02 PM
To: Bill Pittman ;
openembedded-core@lists.openembedded.org
Subject: [EXTERNAL] Re: [OE-core] [PATCH] wic: Use custom kernel path if
provided
On Tue, 2022-02-15 at 16:54 +, Bill Pittman wrote:
> If the custom kernel path is provided in options, then
> use that path instead of the default path.
>
> Signed-off-by: Bill Pittman
> ---
> scripts/wic | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/scripts/wic b/scripts/wic
> index a741aed364..29a157bf37 100755
> --- a/scripts/wic
> +++ b/scripts/wic
> @@ -159,6 +159,9 @@ def wic_create_subcommand(options, usage_str):
> "(Use -e/--image-name to specify it)")
> native_sysroot = options.native_sysroot
>
> +if options.kernel_dir:
> +kernel_dir = options.kernel_dir
> +
> if not options.vars_dir and (not native_sysroot or not
> os.path.isdir(native_sysroot)):
> logger.info("Building wic-tools...\n")
> subprocess.check_call(["bitbake", "wic-tools"])
The concern with these kinds of changes is I can't really test it or tell if it
is the right thing to do and I worry about it regressing in the future.
Is there part of the test coverage we should be improving too? wic does have
fairly good coverage from oe-selftest -r wic
Cheers,
Richard
INTERNAL - NI CONFIDENTIAL
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162855):
https://lists.openembedded.org/g/openembedded-core/message/162855
Mute This Topic: https://lists.openembedded.org/mt/89165206/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH] wic: Use custom kernel path if provided
Andrei, In our build, we use a two stage build, the first stage builds the IPKs and packages files, the second stage builds the various images from the feeds created in the first stage. We've run into a corner case when the kernel repo gets an update that bumps the version, if that happens, the kernel will rebuild during the second stage build (the kernel recipe is on AUTOREV). Unfortunately, that now creates a new kernel with a newer version that the one in the pre-built feed. The image recipe is then assembled using the prebuilt feed (and the older kernel), however the wic script picks up the new kernel. Then when the wic image is booted, the new kernel in the boot portion of the wic image is unable to find the right version of modules and it panics. This is only an issue during development while the kernel recipe is on autorev, but its causing other teams (and us) some grief and this change allows the wic script to always use a custom kernel path. In this case it allows the wic script to use the path to the kernel installed in the image recipe. The patch I'm proposing allows options.kernel_dir to always be used if its provided. The current code in wic_create_subcommand has a code path where it is presently ignored. Cheers, Bill From: Andrei Gherzan Sent: Wednesday, March 2, 2022 4:06 PM To: Bill Pittman Cc: openembedded-core@lists.openembedded.org Subject: [EXTERNAL] Re: [OE-core] [PATCH] wic: Use custom kernel path if provided Feb 15, 2022 16:54:28 Bill Pittman : > If the custom kernel path is provided in options, then > use that path instead of the default path. Just to understand this a bit, what is your usecase here? -- Andrei Gherzan gpg: rsa4096/D4D94F67AD0E9640 INTERNAL - NI CONFIDENTIAL -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#162854): https://lists.openembedded.org/g/openembedded-core/message/162854 Mute This Topic: https://lists.openembedded.org/mt/89165206/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] wic: Use custom kernel path if provided
On Tue, 2022-02-15 at 16:54 +, Bill Pittman wrote:
> If the custom kernel path is provided in options, then
> use that path instead of the default path.
>
> Signed-off-by: Bill Pittman
> ---
> scripts/wic | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/scripts/wic b/scripts/wic
> index a741aed364..29a157bf37 100755
> --- a/scripts/wic
> +++ b/scripts/wic
> @@ -159,6 +159,9 @@ def wic_create_subcommand(options, usage_str):
> "(Use -e/--image-name to specify it)")
> native_sysroot = options.native_sysroot
>
> + if options.kernel_dir:
> + kernel_dir = options.kernel_dir
> +
> if not options.vars_dir and (not native_sysroot or not
> os.path.isdir(native_sysroot)):
> logger.info("Building wic-tools...\n")
> subprocess.check_call(["bitbake", "wic-tools"])
The concern with these kinds of changes is I can't really test it or tell if it
is the right thing to do and I worry about it regressing in the future.
Is there part of the test coverage we should be improving too? wic does have
fairly good coverage from oe-selftest -r wic
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#162628):
https://lists.openembedded.org/g/openembedded-core/message/162628
Mute This Topic: https://lists.openembedded.org/mt/89165206/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] wic: Use custom kernel path if provided
Feb 15, 2022 16:54:28 Bill Pittman : > If the custom kernel path is provided in options, then > use that path instead of the default path. Just to understand this a bit, what is your usecase here? -- Andrei Gherzan gpg: rsa4096/D4D94F67AD0E9640 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#162627): https://lists.openembedded.org/g/openembedded-core/message/162627 Mute This Topic: https://lists.openembedded.org/mt/89165206/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] wic: Use custom kernel path if provided
Hey all, Gentle ping on this patch, any comments? Thanks, Bill From: Bill Pittman Sent: Tuesday, February 15, 2022 10:54 AM To: openembedded-core@lists.openembedded.org Subject: [PATCH] wic: Use custom kernel path if provided If the custom kernel path is provided in options, then use that path instead of the default path. INTERNAL - NI CONFIDENTIAL -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#162618): https://lists.openembedded.org/g/openembedded-core/message/162618 Mute This Topic: https://lists.openembedded.org/mt/89165206/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH] wic: Use custom kernel path if provided
If the custom kernel path is provided in options, then
use that path instead of the default path.
Signed-off-by: Bill Pittman
---
scripts/wic | 3 +++
1 file changed, 3 insertions(+)
diff --git a/scripts/wic b/scripts/wic
index a741aed364..29a157bf37 100755
--- a/scripts/wic
+++ b/scripts/wic
@@ -159,6 +159,9 @@ def wic_create_subcommand(options, usage_str):
"(Use -e/--image-name to specify it)")
native_sysroot = options.native_sysroot
+if options.kernel_dir:
+kernel_dir = options.kernel_dir
+
if not options.vars_dir and (not native_sysroot or not
os.path.isdir(native_sysroot)):
logger.info("Building wic-tools...\n")
subprocess.check_call(["bitbake", "wic-tools"])
--
2.25.1
INTERNAL - NI CONFIDENTIAL
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#161756):
https://lists.openembedded.org/g/openembedded-core/message/161756
Mute This Topic: https://lists.openembedded.org/mt/89165206/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
