Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg failure

["Michael D Kinney via groups.io" <michael.d.kinney=intel.com@groups.io>]

Yes.  We have had issues with Windows and Linux with the virtual address ranges used.

It is a fragile aspect of EmulatorPkg.

The challenge is that value needs to be FixedAtBuild because modules need the
value before dynamic PCD services are available.

We could consider alternate storage for this value and allow Windows and Linux
init code to allocate this special buffer anywhere and store the address in
a global location in HW (such as CMOS).

Mike

> -----Original Message-----
> From: Michael Kubacki <mikuback@linux.microsoft.com>
> Sent: Wednesday, December 18, 2024 1:27 PM
> To: devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>; osde@linux.microsoft.com;
> leif.lindholm@oss.qualcomm.com
> Cc: Oliver Smith-Denny <osde@microsoft.com>
> Subject: Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg failure
> 
> It looks like there was a similar issue in the past that led to a tweak
> to the PCD value -
> https://github.com/tianocore/edk2/commit/868254ace3a0729c9ed6eb1a8646f64
> c77ee84f9.
> 
> Mike, do you recall that rationale / debug details for that change? I'm
> also making some pipeline changes for the new names. Understanding this
> is tangential to a change dropping IA32 VS EmulatorPkg in the pipeline
> but that could be considered as well and I could add that if desired.
> 
> Thanks,
> Michael
> 
> On 12/18/2024 3:44 PM, Michael Kubacki wrote:
> > Hi Mike,
> >
> > I've also had trouble reproducing it consistently locally but I was
> able
> > to with at least one build. With that build, simply rerunning the same
> > WinHost.exe binary would sometimes work and fail.
> >
> > Thanks,
> > Michael
> >
> > On 12/18/2024 3:33 PM, Michael D Kinney via groups.io wrote:
> >> I also just did a rerun on a job that failed, and the rerun passed.
> >>
> >>      https://github.com/tianocore/edk2/pull/6542
> >>
> >> Either there was an OS change/patch since the failed run or the
> >> failure is non deterministic.
> >>
> >> Mike
> >>
> >>> -----Original Message-----
> >>> From: Kinney, Michael D <michael.d.kinney@intel.com>
> >>> Sent: Wednesday, December 18, 2024 12:31 PM
> >>> To: devel@edk2.groups.io; mikuback@linux.microsoft.com;
> >>> osde@linux.microsoft.com; leif.lindholm@oss.qualcomm.com
> >>> Cc: Oliver Smith-Denny <osde@microsoft.com>; Kinney, Michael D
> >>> <michael.d.kinney@intel.com>
> >>> Subject: RE: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg
> failure
> >>>
> >>> Here are some more details.  Looks like the upgrade to VS2022 also
> >>> change the runner image which included an OS version change as well.
> >>>
> >>> Passing Azure Pipelines CI Agent details:
> >>>      Image: windows-2019
> >>>      Version: 20241125.1.0
> >>>      Included Software: https://github.com/actions/runner-
> >>> images/blob/win19/20241125.1/images/windows/Windows2019-Readme.md
> >>>          OS Version: 10.0.17763 Build 6532
> >>>          Image Version: 20241125.1.0
> >>>      Image Release: https://github.com/actions/runner-
> >>> images/releases/tag/win19%2F20241125.1
> >>>
> >>> Failing Azure Pipelines CI Agent details:
> >>>      Runner Image
> >>>      Image: windows-2022
> >>>      Version: 20241215.1.0
> >>>      Included Software: https://github.com/actions/runner-
> >>> images/blob/win22/20241215.1/images/windows/Windows2022-Readme.md
> >>>          OS Version: 10.0.20348 Build 2966
> >>>          Image Version: 20241215.1.0
> >>>      Image Release: https://github.com/actions/runner-
> >>> images/releases/tag/win22%2F20241215.1
> >>>
> >>>> -----Original Message-----
> >>>> From: Kinney, Michael D <michael.d.kinney@intel.com>
> >>>> Sent: Wednesday, December 18, 2024 9:54 AM
> >>>> To: devel@edk2.groups.io; mikuback@linux.microsoft.com;
> >>>> osde@linux.microsoft.com; leif.lindholm@oss.qualcomm.com
> >>>> Cc: Oliver Smith-Denny <osde@microsoft.com>; Kinney, Michael D
> >>>> <michael.d.kinney@intel.com>
> >>>> Subject: RE: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg
> failure
> >>>>
> >>>> I can not reproduce locally either.
> >>>>
> >>>> I installed VS2022 build tools.
> >>>>
> >>>> The error message is related to the ability to allocate a specific
> >>> range
> >>>> of virtual memory.
> >>>>
> >>>> If that fails on CI agent, but not local envs, it could be related
> to
> >>>> the
> >>>> CI agent windows image being used.
> >>>>
> >>>> Mike
> >>>>
> >>>>
> >>>>> -----Original Message-----
> >>>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> >>> Michael
> >>>>> Kubacki via groups.io
> >>>>> Sent: Wednesday, December 18, 2024 8:48 AM
> >>>>> To: devel@edk2.groups.io; osde@linux.microsoft.com;
> >>>>> leif.lindholm@oss.qualcomm.com; Kinney, Michael D
> >>>>> <michael.d.kinney@intel.com>
> >>>>> Cc: Oliver Smith-Denny <osde@microsoft.com>
> >>>>> Subject: Re: [edk2-devel] CI blockage: VS2019 Ia32 EmulatorPkg
> >>> failure
> >>>>>
> >>>>> I'll update the pipeline names later today.
> >>>>>
> >>>>> Thanks,
> >>>>> Michael
> >>>>>
> >>>>> On 12/18/2024 10:42 AM, Oliver Smith-Denny via groups.io wrote:
> >>>>>> Hi Leif,
> >>>>>>
> >>>>>> On 12/18/2024 3:13 AM, Leif Lindholm via groups.io wrote:
> >>>>>>> PR CI jobs are currently failing on Ia32 EmulatorPkg RELEASE
> >>>> (passes
> >>>>>>> on DEBUG) with
> >>>>>>> ---
> >>>>>>> ERROR : Could not allocate PeiServicesTablePage @ 13000000
> >>>>>>> ---
> >>>>>>> during the "run to shell" test.
> >>>>>>>
> >>>>>>> I note that the build step for
> >>>>>>> PlatformCI_EmulatorPkg_Windows_VS2019_PR now uses VS2022,
> >>>>>>> presumably as of https://github.com/tianocore/edk2/pull/6501
> >>>>>>>
> >>>>>>> Can someone with access to a VS environment help investigate?
> >>>>>>
> >>>>>> I tried to repro this locally yesterday, but unfortunately was
> not
> >>>>>> getting one. I will try again today.
> >>>>>>
> >>>>>>> We should probably also get the CI job names updated to reflect
> >>> the
> >>>>>>> toolchain version they're using.
> >>>>>>
> >>>>>> Yes, I asked Mike Kinney to update the pipelines (I unfortunately
> >>> do
> >>>>>> not have permission to do so, it has to be done by an admin, Mike
> >>>>> Kinney
> >>>>>> or Michael Kubacki I believe will do it. The plan is to also
> >>> change
> >>>>> the
> >>>>>> name of the GCC5 pipeline (Gerd requested this) and instead have
> >>>> more
> >>>>>> generically named pipelines so when they are updated in the
> future
> >>>> we
> >>>>>> don't have this name mismatch.
> >>>>>>
> >>>>>> Thanks,
> >>>>>> Oliver
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>>
> >>
> >>
> >>
> >> 
> >>
> >>
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120919): https://edk2.groups.io/g/devel/message/120919
Mute This Topic: https://groups.io/mt/110178683/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-