From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: "Richard Purdie" <richard.purdie@linuxfoundation.org>,
"Enrico Jörns" <ejo@pengutronix.de>,
openembedded-core@lists.openembedded.org
Cc: Alexander Kanavin <alex.kanavin@gmail.com>, yocto@pengutronix.de
Subject: Re: [yocto] [PATCH v7 0/7] Add barebox bootloader support (and testing)
Date: Wed, 9 Oct 2024 08:45:54 +0200 [thread overview]
Message-ID: <2553e06f-11de-413f-b705-dfa6f19c23a1@pengutronix.de> (raw)
In-Reply-To: <739cdd73-470e-4fa4-8901-750c246a28bd@pengutronix.de>
Hello,
On 04.10.24 12:14, Ahmad Fatoum wrote:
> On 02.10.24 10:17, Ahmad Fatoum wrote:
>> I am building for MACHINE = "qemuarm64" already, but something else catches
>> my eye: QEMU_USE_KVM = 'True'
>>
>> I wasn't aware that I need to manually set this. I do this now and it fails
>> for other reasons: qemu-system-aarch64: KVM does not support GICv3 emulation
>>
>> I'll look into how to resolve this and maybe I am lucky and can reproduce
>> this issue, once KVM is enabled.
>
> Good news: I managed to reproduce this issue and write a few lines long reproducer
> that I sent in a bug report to the KVM/QEMU development mailing lists:
>
> https://lore.kernel.org/all/89f184d6-5b61-4c77-9f3b-c0a8f6a75d60@pengutronix.de/
TL;DR: Turns out this is not a bug, but a known quirk of KVM and software that
wishes to run under KVM needs to adapt the instructions it uses to access MMIO
regions accordingly.
For barebox, I sent a series adding KVM compatibility here:
https://lore.kernel.org/barebox/20241009060511.4121157-1-a.fatoum@pengutronix.de
For U-Boot, I reported the issue here:
https://lore.kernel.org/u-boot/2211f9f0-cd7d-4b55-860d-a34c04877e7b@pengutronix.de
> As far as this series here is concerned, both barebox and U-Boot function correctly
> with KVM disabled, so could we just disable KVM for only the U-Boot/barebox runqemu tests
> to unblock merge of this series?
I am more certain now that we should disable KVM for bootloader testing.
Full system tests with Linux are much more demanding, so KVM absolutely
makes sense there.
In the bootloader however, normal emulation should suffice, especially when
their upstream isn't KVM-compatible yet anyway.
Cheers,
Ahmad
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
prev parent reply other threads:[~2024-10-09 6:45 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-19 12:06 [PATCH v7 0/7] Add barebox bootloader support (and testing) Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 1/7] barebox: add initial support Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 2/7] barebox-tools: add initial barebox tools support Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 3/7] barebox: set default BAREBOX_CONFIG for qemu machines Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 4/7] oeqa/utils/qemurunner: support ignoring vt100 escape sequences Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 5/7] oeqa: support passing custom boot patterns to runqemu Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 6/7] oeqa/selftest/cases: add basic u-boot test Enrico Jörns
2024-09-19 12:06 ` [PATCH v7 7/7] oeqa/selftest/cases: add basic barebox tests Enrico Jörns
2024-09-19 20:56 ` [PATCH v7 0/7] Add barebox bootloader support (and testing) Richard Purdie
2024-09-20 10:37 ` Enrico Jörns
2024-10-01 14:35 ` [yocto] " Ahmad Fatoum
2024-10-01 14:56 ` Richard Purdie
2024-10-02 8:17 ` Ahmad Fatoum
2024-10-04 10:14 ` Ahmad Fatoum
2024-10-09 6:45 ` Ahmad Fatoum [this message]
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=2553e06f-11de-413f-b705-dfa6f19c23a1@pengutronix.de \
--to=a.fatoum@pengutronix.de \
--cc=alex.kanavin@gmail.com \
--cc=ejo@pengutronix.de \
--cc=openembedded-core@lists.openembedded.org \
--cc=richard.purdie@linuxfoundation.org \
--cc=yocto@pengutronix.de \
/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