Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Andrej Valek <andrej.valek@siemens.com>
Cc: openembedded-core@lists.openembedded.org,
	Abdellatif.ElKhlifi@arm.com,
	Walter Schweizer <walter.schweizer@siemens.com>
Subject: Re: [OE-core][PATCH v2 2/2] kernel-fitimage: use correct kernel image
Date: Wed, 13 Oct 2021 11:57:25 +0200	[thread overview]
Message-ID: <YWathVAwfGUafaRL@piout.net> (raw)
In-Reply-To: <20211006070031.156404-2-andrej.valek@siemens.com>

Hello,

This series causes the following failure:

https://autobuilder.yoctoproject.org/typhoon/#/builders/87/builds/2699/steps/15/logs/stdio

2021-10-12 00:31:47,557 - oe-selftest - INFO - fitimage.FitImageTests.test_initramfs_bundle (subunit.RemotedTestCase)
2021-10-12 00:31:47,665 - oe-selftest - INFO -  ... FAIL
Stderr:
2021-10-11 21:55:16,664 - oe-selftest - INFO - Adding: "include selftest.inc" in /home/pokybuild/yocto-worker/oe-selftest-ubuntu/build/build-st-16604/conf/local.conf
2021-10-11 21:55:16,665 - oe-selftest - INFO - Adding: "include bblayers.inc" in bblayers.conf
2021-10-12 00:31:47,666 - oe-selftest - INFO - 10: 3/13 335/444 (2817.06s) (fitimage.FitImageTests.test_initramfs_bundle)
2021-10-12 00:31:47,666 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
  File "/home/pokybuild/yocto-worker/oe-selftest-ubuntu/build/meta/lib/oeqa/selftest/cases/fitimage.py", line 803, in test_initramfs_bundle
    self.assertTrue(test_passed == True,"kernel node does not match expectation")
  File "/home/pokybuild/yocto-worker/oe-selftest-ubuntu/build/buildtools/sysroots/x86_64-pokysdk-linux/usr/lib/python3.9/unittest/case.py", line 682, in assertTrue
    raise self.failureException(msg)
AssertionError: False is not true : kernel node does not match expectation

It failed on all 4 oe-selftest builds

On 06/10/2021 09:00:31+0200, Andrej Valek wrote:
> Even if initramfs_bundle_path was used, a wrong compression was reflected
> in output its template file. Use linux.bin as universal kernel image.
> The linux.bin file covers both cases.
> 
> Signed-off-by: Andrej Valek <andrej.valek@siemens.com>
> Signed-off-by: Walter Schweizer <walter.schweizer@siemens.com>
> ---
>  meta/classes/kernel-fitimage.bbclass | 17 +----------------
>  1 file changed, 1 insertion(+), 16 deletions(-)
> 
> diff --git a/meta/classes/kernel-fitimage.bbclass b/meta/classes/kernel-fitimage.bbclass
> index 886ed13029..8718ce7e16 100644
> --- a/meta/classes/kernel-fitimage.bbclass
> +++ b/meta/classes/kernel-fitimage.bbclass
> @@ -495,22 +495,7 @@ fitimage_assemble() {
>  	fitimage_emit_section_maint $1 imagestart
>  
>  	uboot_prep_kimage
> -
> -	if [ "${INITRAMFS_IMAGE_BUNDLE}" = "1" ]; then
> -		initramfs_bundle_path="arch/"${UBOOT_ARCH}"/boot/"${KERNEL_IMAGETYPE_REPLACEMENT}".initramfs"
> -		if [ -e "$initramfs_bundle_path" ]; then
> -
> -			#
> -			# Include the kernel/rootfs bundle.
> -			#
> -
> -			fitimage_emit_section_kernel $1 $kernelcount "$initramfs_bundle_path" "$linux_comp"
> -		else
> -			bbwarn "$initramfs_bundle_pat not found."
> -		fi
> -	else
> -		fitimage_emit_section_kernel $1 $kernelcount linux.bin "$linux_comp"
> -	fi
> +	fitimage_emit_section_kernel $1 $kernelcount linux.bin "$linux_comp"
>  
>  	#
>  	# Step 2: Prepare a DTB image section
> -- 
> 2.31.1
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#156674): https://lists.openembedded.org/g/openembedded-core/message/156674
> Mute This Topic: https://lists.openembedded.org/mt/86113979/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


  reply	other threads:[~2021-10-13  9:57 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-05 20:22 [OE-core][PATCH 1/2] featimage: refactor style Andrej Valek
2021-10-05 20:22 ` [OE-core][PATCH 2/2] kernel-fitimage: use correct kernel image Andrej Valek
2021-10-06  7:00 ` [OE-core][PATCH v2 1/2] featimage: refactor style Andrej Valek
2021-10-06  7:00 ` [OE-core][PATCH v2 2/2] kernel-fitimage: use correct kernel image Andrej Valek
2021-10-13  9:57   ` Alexandre Belloni [this message]
2021-10-13 12:06 ` [OE-core][PATCH v3 1/2] featimage: refactor style Andrej Valek
2021-10-13 12:06 ` [OE-core][PATCH v3 2/2] kernel-fitimage: use correct kernel image Andrej Valek
2021-10-16 10:47   ` Alexandre Belloni
2021-10-16 20:01 ` [OE-core][PATCH v4 1/2] featimage: refactor style Andrej Valek
2021-10-16 20:01 ` [OE-core][PATCH v4 2/2] kernel-fitimage: use correct kernel image Andrej Valek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YWathVAwfGUafaRL@piout.net \
    --to=alexandre.belloni@bootlin.com \
    --cc=Abdellatif.ElKhlifi@arm.com \
    --cc=andrej.valek@siemens.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=walter.schweizer@siemens.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox