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@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Oliver Steffen Sent: Tuesday, January 17, 2023 6:37 PM To: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com> Cc: Andrew Fish <af...@apple.com>; Ard Biesheuvel <ardb+tianoc...@kernel.org>; Feng, Bob C <bob.c.f...@intel.com>; Bi, Dandan <dandan...@intel.com>; Gerd Hoffmann <kra...@redhat.com>; Wang, Jian J <jian.j.w...@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com>; Leif Lindholm <quic_llind...@quicinc.com>; Gao, Liming <gaolim...@byosoft.com.cn>; Kinney, Michael D <michael.d.kin...@intel.com>; Michael Kubacki <mikub...@linux.microsoft.com>; Ni, Ray <ray...@intel.com>; Sami Mujawar <sami.muja...@arm.com>; Sean Brogan <sean.bro...@microsoft.com>; Chen, Christine <yuwei.c...@intel.com>; Pawel Polawski <ppola...@redhat.com>; chris.fern...@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....@intel.com<mailto:jiewen....@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 <ostef...@redhat.com<mailto:ostef...@redhat.com>> > Sent: Tuesday, January 17, 2023 1:41 AM > To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > Cc: Andrew Fish <af...@apple.com<mailto:af...@apple.com>>; Ard Biesheuvel > <ardb+tianoc...@kernel.org<mailto:ardb%2btianoc...@kernel.org>>; Feng, Bob C > <bob.c.f...@intel.com<mailto:bob.c.f...@intel.com>>; Bi, > Dandan <dandan...@intel.com<mailto:dandan...@intel.com>>; Gerd Hoffmann > <kra...@redhat.com<mailto:kra...@redhat.com>>; > Wang, Jian J <jian.j.w...@intel.com<mailto:jian.j.w...@intel.com>>; Yao, > Jiewen <jiewen....@intel.com<mailto:jiewen....@intel.com>>; > Justen, Jordan L > <jordan.l.jus...@intel.com<mailto:jordan.l.jus...@intel.com>>; Leif Lindholm > <quic_llind...@quicinc.com<mailto:quic_llind...@quicinc.com>>; Gao, Liming > <gaolim...@byosoft.com.cn<mailto:gaolim...@byosoft.com.cn>>; > Kinney, Michael D > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Michael > Kubacki > <mikub...@linux.microsoft.com<mailto:mikub...@linux.microsoft.com>>; Ni, Ray > <ray...@intel.com<mailto:ray...@intel.com>>; Sami > Mujawar <sami.muja...@arm.com<mailto:sami.muja...@arm.com>>; Sean Brogan > <sean.bro...@microsoft.com<mailto:sean.bro...@microsoft.com>>; Chen, > Christine <yuwei.c...@intel.com<mailto:yuwei.c...@intel.com>>; > Pawel Polawski <ppola...@redhat.com<mailto:ppola...@redhat.com>>; > chris.fern...@outlook.com<mailto:chris.fern...@outlook.com>; Oliver > Steffen <ostef...@redhat.com<mailto:ostef...@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 > <ostef...@redhat.com<mailto:ostef...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
