* [PATCHv2] wic/plugins: fix kernel version detection for bios
@ 2021-02-04 16:39 Henning Schild
2021-02-04 16:51 ` IMPORTANT " Henning Schild
2021-02-05 8:41 ` Baurzhan Ismagulov
0 siblings, 2 replies; 5+ messages in thread
From: Henning Schild @ 2021-02-04 16:39 UTC (permalink / raw)
To: isar-users; +Cc: Vijai Kumar K, Henning Schild
From: Henning Schild <henning.schild@siemens.com>
The mechanism used to "cut off the end" seems to cut off too much.
>>> "vmlinuz-4.19.0-14-amd64".strip('-' + 'amd64')
'vmlinuz-4.19.0-1'
But indeed we would hope for 'vmlinuz-4.19.0-14'. This is a fix that
should be aplied asap, it seems to affect all "even" kernel revisions,
"odd" is odly enough no problem.
Signed-off-by: Henning Schild <henning.schild@siemens.com>
---
meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py
index 493615b481..ece08efa76 100644
--- a/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py
+++ b/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py
@@ -135,7 +135,7 @@ class BootimgPcbiosIsarPlugin(SourcePlugin):
kernel_name = get_bitbake_var("KERNEL_NAME")
rootfs_dir = get_bitbake_var("IMAGE_ROOTFS")
kernel = os.path.basename(os.path.realpath(os.path.join(rootfs_dir, kernel_file)))
- kernel_version = kernel.strip('-' + kernel_name).strip(kernel_file + '-')
+ kernel_version = kernel[len(kernel_file)+1:-(len(kernel_name)+1)]
initrd = "initrd.img-%s-%s" % (kernel_version, kernel_name)
syslinux_conf += "KERNEL " + kernel + "\n"
@@ -165,7 +165,7 @@ class BootimgPcbiosIsarPlugin(SourcePlugin):
kernel_name = get_bitbake_var("KERNEL_NAME")
rootfs_dir = rootfs_dir['ROOTFS_DIR']
kernel = os.path.basename(os.path.realpath(os.path.join(rootfs_dir, kernel_file)))
- kernel_version = kernel.strip('-' + kernel_name).strip(kernel_file + '-')
+ kernel_version = kernel[len(kernel_file)+1:-(len(kernel_name)+1)]
initrd = "initrd.img-%s-%s" % (kernel_version, kernel_name)
config = "config-%s-%s" % (kernel_version, kernel_name)
mapfile = "System.map-%s-%s" % (kernel_version, kernel_name)
--
2.26.2
^ permalink raw reply [flat|nested] 5+ messages in thread* IMPORTANT Re: [PATCHv2] wic/plugins: fix kernel version detection for bios
2021-02-04 16:39 [PATCHv2] wic/plugins: fix kernel version detection for bios Henning Schild
@ 2021-02-04 16:51 ` Henning Schild
2021-02-04 17:04 ` Baurzhan Ismagulov
2021-02-05 8:41 ` Baurzhan Ismagulov
1 sibling, 1 reply; 5+ messages in thread
From: Henning Schild @ 2021-02-04 16:51 UTC (permalink / raw)
To: isar-users, Baurzhan Ismagulov; +Cc: Vijai Kumar K
Seems like a kernel update in buster found a bug in our wic plugin for
legacy bios. I would like to ask for a quick review of this, since it
currently breaks building bios wic images.
Henning
Am Thu, 4 Feb 2021 17:39:12 +0100
schrieb Henning Schild <henning.schild@siemens.com>:
> From: Henning Schild <henning.schild@siemens.com>
>
> The mechanism used to "cut off the end" seems to cut off too much.
>
> >>> "vmlinuz-4.19.0-14-amd64".strip('-' + 'amd64')
> 'vmlinuz-4.19.0-1'
>
> But indeed we would hope for 'vmlinuz-4.19.0-14'. This is a fix that
> should be aplied asap, it seems to affect all "even" kernel revisions,
> "odd" is odly enough no problem.
>
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
> meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git
> a/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py
> b/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py index
> 493615b481..ece08efa76 100644 ---
> a/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py +++
> b/meta/scripts/lib/wic/plugins/source/bootimg-pcbios-isar.py @@
> -135,7 +135,7 @@ class BootimgPcbiosIsarPlugin(SourcePlugin):
> kernel_name = get_bitbake_var("KERNEL_NAME") rootfs_dir =
> get_bitbake_var("IMAGE_ROOTFS") kernel =
> os.path.basename(os.path.realpath(os.path.join(rootfs_dir,
> kernel_file)))
> - kernel_version = kernel.strip('-' +
> kernel_name).strip(kernel_file + '-')
> + kernel_version =
> kernel[len(kernel_file)+1:-(len(kernel_name)+1)] initrd =
> "initrd.img-%s-%s" % (kernel_version, kernel_name)
> syslinux_conf += "KERNEL " + kernel + "\n"
> @@ -165,7 +165,7 @@ class BootimgPcbiosIsarPlugin(SourcePlugin):
> kernel_name = get_bitbake_var("KERNEL_NAME")
> rootfs_dir = rootfs_dir['ROOTFS_DIR']
> kernel =
> os.path.basename(os.path.realpath(os.path.join(rootfs_dir,
> kernel_file)))
> - kernel_version = kernel.strip('-' +
> kernel_name).strip(kernel_file + '-')
> + kernel_version =
> kernel[len(kernel_file)+1:-(len(kernel_name)+1)] initrd =
> "initrd.img-%s-%s" % (kernel_version, kernel_name) config =
> "config-%s-%s" % (kernel_version, kernel_name) mapfile =
> "System.map-%s-%s" % (kernel_version, kernel_name)
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: IMPORTANT Re: [PATCHv2] wic/plugins: fix kernel version detection for bios
2021-02-04 16:51 ` IMPORTANT " Henning Schild
@ 2021-02-04 17:04 ` Baurzhan Ismagulov
2021-02-04 19:37 ` Henning Schild
0 siblings, 1 reply; 5+ messages in thread
From: Baurzhan Ismagulov @ 2021-02-04 17:04 UTC (permalink / raw)
To: Henning Schild; +Cc: isar-users, Vijai Kumar K, Anton Mikanovich
On Thu, Feb 04, 2021 at 05:51:23PM +0100, Henning Schild wrote:
> Seems like a kernel update in buster found a bug in our wic plugin for
> legacy bios. I would like to ask for a quick review of this, since it
> currently breaks building bios wic images.
Thanks for heads up, will check.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: IMPORTANT Re: [PATCHv2] wic/plugins: fix kernel version detection for bios
2021-02-04 17:04 ` Baurzhan Ismagulov
@ 2021-02-04 19:37 ` Henning Schild
0 siblings, 0 replies; 5+ messages in thread
From: Henning Schild @ 2021-02-04 19:37 UTC (permalink / raw)
To: Baurzhan Ismagulov; +Cc: isar-users, Vijai Kumar K, Anton Mikanovich
Am Thu, 4 Feb 2021 18:04:57 +0100
schrieb Baurzhan Ismagulov <ibr@radix50.net>:
> On Thu, Feb 04, 2021 at 05:51:23PM +0100, Henning Schild wrote:
> > Seems like a kernel update in buster found a bug in our wic plugin
> > for legacy bios. I would like to ask for a quick review of this,
> > since it currently breaks building bios wic images.
>
> Thanks for heads up, will check.
I would assume all pipelines to fail without it, since "yesterday".
Applying it should hopefully fix that. If it gets merged people will
hopefully find it and bump to latest next or backport to whichever isar
version they might be on.
Henning
> With kind regards,
> Baurzhan.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCHv2] wic/plugins: fix kernel version detection for bios
2021-02-04 16:39 [PATCHv2] wic/plugins: fix kernel version detection for bios Henning Schild
2021-02-04 16:51 ` IMPORTANT " Henning Schild
@ 2021-02-05 8:41 ` Baurzhan Ismagulov
1 sibling, 0 replies; 5+ messages in thread
From: Baurzhan Ismagulov @ 2021-02-05 8:41 UTC (permalink / raw)
To: Henning Schild; +Cc: isar-users, Vijai Kumar K
On Thu, Feb 04, 2021 at 05:39:12PM +0100, Henning Schild wrote:
> From: Henning Schild <henning.schild@siemens.com>
>
> The mechanism used to "cut off the end" seems to cut off too much.
>
> >>> "vmlinuz-4.19.0-14-amd64".strip('-' + 'amd64')
> 'vmlinuz-4.19.0-1'
>
> But indeed we would hope for 'vmlinuz-4.19.0-14'. This is a fix that
> should be aplied asap, it seems to affect all "even" kernel revisions,
> "odd" is odly enough no problem.
>
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
Regarding the commit message, I'd suggest:
* Mentioning the user-visible failure.
* Dropping the "This is a fix that should be aplied asap" part.
* "odly" -> "oddly".
We'll provide an update regarding the code.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-02-05 8:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-04 16:39 [PATCHv2] wic/plugins: fix kernel version detection for bios Henning Schild
2021-02-04 16:51 ` IMPORTANT " Henning Schild
2021-02-04 17:04 ` Baurzhan Ismagulov
2021-02-04 19:37 ` Henning Schild
2021-02-05 8:41 ` Baurzhan Ismagulov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox