Re: [PATCH v3 3/5] MdePkg/DxeServicesLib: introduce AllocatePeiAccessiblePages routine

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 29 May 2018 at 18:09, Kinney, Michael D <michael.d.kinney@intel.com> wrote:
> Jiewen,
>
> I see what you mean.  It is not the submitting of
> capsules you are referring to.  It is the processing
> if capsules in the PEI phase that depends on some
> things being setup in DXE phase and DXE phase needs
> to know if PEI is in IA32 mode.
>
> So I agree that the commit message could add capsule
> processing to the list of features that can use this
> new service.
>
> The logic in that file is using PcdDxeIplSwitchToLongMode.
> That PCD is TRUE when PEI is IA32 and DXE is X64.  But it
> is ignored when  PEI and DXE are both X64 and could be
> TRUE or FALSE.  Is there a logic issue here with using that
> PCD when PEI and DXE are both X64?
>

Hi all,

The reason I disregarded this particular case in my series is that
this code is already specific to X64. However, given the discussion
you are having, I guess the logic of looking at EfiFreeMemoryTop [at
runtime] combined with the various [build time] PCD values could help
to refine this case as well.