diff mbox series

[v4,2/4] qemuarm64.conf: allow overriding QB_OPT_APPEND

Message ID 20250526083548.413834-2-mikko.rapeli@linaro.org
State New
Headers show
Series [v4,1/4] u-boot: disable CONFIG_BLOBLIST on genericarm64 and qemuarm64 | expand

Commit Message

Mikko Rapeli May 26, 2025, 8:35 a.m. UTC
It enables qemu USB support which causes a reset
loop currently with u-boot.

Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
---
 meta/conf/machine/qemuarm64.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini May 28, 2025, 12:52 a.m. UTC | #1
On Mon, May 26, 2025 at 11:35:45AM +0300, Mikko Rapeli wrote:

> It enables qemu USB support which causes a reset
> loop currently with u-boot.
> 
> Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
> ---
>  meta/conf/machine/qemuarm64.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

This too feel like something that should be brought up on the U-Boot
side.
Mikko Rapeli May 28, 2025, 8:02 a.m. UTC | #2
Hi,

On Tue, May 27, 2025 at 06:52:47PM -0600, Tom Rini wrote:
> On Mon, May 26, 2025 at 11:35:45AM +0300, Mikko Rapeli wrote:
> 
> > It enables qemu USB support which causes a reset
> > loop currently with u-boot.
> > 
> > Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
> > ---
> >  meta/conf/machine/qemuarm64.conf | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> This too feel like something that should be brought up on the U-Boot
> side.

We saw these issues in 2023 on qemu but thought they were specific to our
Trusted Substrate config with tf-a, OP-TEE and u-boot. Some public info is
in: https://linaro.atlassian.net/browse/LEDGE-435

I know USB support is really tricky with timings etc so I don't expect
it to work well with bootloaders. Hence did not dig deeper.

Now that I saw this reset loop with KVM I will note the details down
with reproduction steps and send to u-boot list.

But I think overriding QB_OPT_APPEND is handy, wether it has USB setup
or something else. Other layers in yocto ecosystem use qemuarm64 as basis
machine with testing and may want to change this.

Cheers,

-Mikko
Tom Rini May 28, 2025, 2:15 p.m. UTC | #3
On Wed, May 28, 2025 at 11:02:44AM +0300, Mikko Rapeli wrote:
> Hi,
> 
> On Tue, May 27, 2025 at 06:52:47PM -0600, Tom Rini wrote:
> > On Mon, May 26, 2025 at 11:35:45AM +0300, Mikko Rapeli wrote:
> > 
> > > It enables qemu USB support which causes a reset
> > > loop currently with u-boot.
> > > 
> > > Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
> > > ---
> > >  meta/conf/machine/qemuarm64.conf | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > This too feel like something that should be brought up on the U-Boot
> > side.
> 
> We saw these issues in 2023 on qemu but thought they were specific to our
> Trusted Substrate config with tf-a, OP-TEE and u-boot. Some public info is
> in: https://linaro.atlassian.net/browse/LEDGE-435
> 
> I know USB support is really tricky with timings etc so I don't expect
> it to work well with bootloaders. Hence did not dig deeper.
> 
> Now that I saw this reset loop with KVM I will note the details down
> with reproduction steps and send to u-boot list.
> 
> But I think overriding QB_OPT_APPEND is handy, wether it has USB setup
> or something else. Other layers in yocto ecosystem use qemuarm64 as basis
> machine with testing and may want to change this.

I would say there are two parts to this. On the U-Boot side, USB is
expected to work and we have active custodians dealing with it. Please
work with Ilias and others to get a reproduceable set of steps with
vanilla QEMU and report them to the U-Boot list with appropriate CC. On
the OE side, it's all complicated enough that I would defer to Richard.
diff mbox series

Patch

diff --git a/meta/conf/machine/qemuarm64.conf b/meta/conf/machine/qemuarm64.conf
index d310445a34..f086729641 100644
--- a/meta/conf/machine/qemuarm64.conf
+++ b/meta/conf/machine/qemuarm64.conf
@@ -20,7 +20,7 @@  QB_SMP ?= "-smp 4"
 QB_CPU_KVM = "-cpu host -machine gic-version=3"
 # For graphics to work we need to define the VGA device as well as the necessary USB devices
 QB_GRAPHICS = "-device virtio-gpu-pci"
-QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-kbd"
+QB_OPT_APPEND ?= "-device qemu-xhci -device usb-tablet -device usb-kbd"
 # Virtio Networking support
 QB_TAP_OPT = "-netdev tap,id=net0,ifname=@TAP@,script=no,downscript=no"
 QB_NETWORK_DEVICE = "-device virtio-net-pci,netdev=net0,mac=@MAC@"