[edk2-devel] [PATCH v11 17/17] OfmvPkg: CI: Qemu tests: set CPU to Broadwell

[Yao, Jiewen]

I have seen Gerd sent out V4 patch set for #4251. Is that not enough?

In general, I prefer no work-around, unless there is strong reason.

I would like to have Gerd and Ard to feedback.

Thank you
Yao, Jiewen

From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Oliver Steffen
Sent: Tuesday, January 17, 2023 6:37 PM
To: devel at edk2.groups.io; Yao, Jiewen <jiewen.yao at intel.com>
Cc: Andrew Fish <afish at apple.com>; Ard Biesheuvel <ardb+tianocore at kernel.org>; Feng, Bob C <bob.c.feng at intel.com>; Bi, Dandan <dandan.bi at intel.com>; Gerd Hoffmann <kraxel at redhat.com>; Wang, Jian J <jian.j.wang at intel.com>; Justen, Jordan L <jordan.l.justen at intel.com>; Leif Lindholm <quic_llindhol at quicinc.com>; Gao, Liming <gaoliming at byosoft.com.cn>; Kinney, Michael D <michael.d.kinney at intel.com>; Michael Kubacki <mikuback at linux.microsoft.com>; Ni, Ray <ray.ni at intel.com>; Sami Mujawar <sami.mujawar at arm.com>; Sean Brogan <sean.brogan at microsoft.com>; Chen, Christine <yuwei.chen at intel.com>; Pawel Polawski <ppolawsk at redhat.com>; chris.fernald at outlook.com
Subject: Re: [edk2-devel] [PATCH v11 17/17] OfmvPkg: CI: Qemu tests: set CPU to Broadwell

It is this one:
https://bugzilla.tianocore.org/show_bug.cgi?id=4251
Should have put that into the commit message, sorry.


On Tue, Jan 17, 2023 at 1:16 AM Yao, Jiewen <jiewen.yao at intel.com<mailto:jiewen.yao at intel.com>> wrote:
Not sure the detail.

Could you please add more comment, such as, what issue? Why crash? Why changing to Broadwell can resolve it?
any URL?



> -----Original Message-----
> From: Oliver Steffen <osteffen at redhat.com<mailto:osteffen at redhat.com>>
> Sent: Tuesday, January 17, 2023 1:41 AM
> To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>
> Cc: Andrew Fish <afish at apple.com<mailto:afish at apple.com>>; Ard Biesheuvel
> <ardb+tianocore at kernel.org<mailto:ardb%2Btianocore at kernel.org>>; Feng, Bob C <bob.c.feng at intel.com<mailto:bob.c.feng at intel.com>>; Bi,
> Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Gerd Hoffmann <kraxel at redhat.com<mailto:kraxel at redhat.com>>;
> Wang, Jian J <jian.j.wang at intel.com<mailto:jian.j.wang at intel.com>>; Yao, Jiewen <jiewen.yao at intel.com<mailto:jiewen.yao at intel.com>>;
> Justen, Jordan L <jordan.l.justen at intel.com<mailto:jordan.l.justen at intel.com>>; Leif Lindholm
> <quic_llindhol at quicinc.com<mailto:quic_llindhol at quicinc.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>;
> Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; Michael Kubacki
> <mikuback at linux.microsoft.com<mailto:mikuback at linux.microsoft.com>>; Ni, Ray <ray.ni at intel.com<mailto:ray.ni at intel.com>>; Sami
> Mujawar <sami.mujawar at arm.com<mailto:sami.mujawar at arm.com>>; Sean Brogan
> <sean.brogan at microsoft.com<mailto:sean.brogan at microsoft.com>>; Chen, Christine <yuwei.chen at intel.com<mailto:yuwei.chen at intel.com>>;
> Pawel Polawski <ppolawsk at redhat.com<mailto:ppolawsk at redhat.com>>; chris.fernald at outlook.com<mailto:chris.fernald at outlook.com>; Oliver
> Steffen <osteffen at redhat.com<mailto:osteffen at redhat.com>>
> Subject: [PATCH v11 17/17] OfmvPkg: CI: Qemu tests: set CPU to Broadwell
>
> Work around a bug in Ovmf that causes Ovmf to crash on startup
> when running in recent versions of Qemu.  Setting the cpu model
> to something older, for example "Broadwell" works around the issue.
>
> Signed-off-by: Oliver Steffen <osteffen at redhat.com<mailto:osteffen at redhat.com>>
> ---
>  OvmfPkg/PlatformCI/PlatformBuildLib.py | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/OvmfPkg/PlatformCI/PlatformBuildLib.py
> b/OvmfPkg/PlatformCI/PlatformBuildLib.py
> index bfef9849c749..93f7d7aa13d6 100644
> --- a/OvmfPkg/PlatformCI/PlatformBuildLib.py
> +++ b/OvmfPkg/PlatformCI/PlatformBuildLib.py
> @@ -194,6 +194,7 @@ class PlatformBuilder( UefiBuilder,
> BuildSettingsManager):
>          cmd = "qemu-system-x86_64"
>          args  = "-debugcon stdio"                                           # write messages to
> stdio
>          args += " -global isa-debugcon.iobase=0x402"                        # debug
> messages out thru virtual io port
> +        args += " -cpu Broadwell"                                           # workaround Qvmf
> bug
>          args += " -net none"                                                # turn off network
>          args += f" -drive file=fat:rw:{VirtualDrive},format=raw,media=disk" #
> Mount disk with startup.nsh
>
> --
> 2.39.0








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98679): https://edk2.groups.io/g/devel/message/98679
Mute This Topic: https://groups.io/mt/96311858/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230117/5623745d/attachment-0001.htm>