uesday, February 23, 2021 1:44 PM
To: devel@edk2.groups.io; Ni, Ray ; Ma, Maurice
; Patrick Rudolph ; Dong,
Guo ; Dong, Eric
Cc: You, Benjamin
Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Ray,
BSP's GDT table is setup in CpuDxe and then MpInitLib re
, Eric
Cc: You, Benjamin
Subject: RE: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Maurice,
It’s doable 😊
I prefer to accept today’s below 4G limitation to maintain simple
implementation.
Thanks,
Ray
From: Ma, Maurice mailto:maurice...@intel.com>>
Sent: T
, Benjamin
Subject: RE: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Could we decouple BSP and AP GDT in early waking up stage ?
For example, AP uses temporary GDT (below 4GB) just for mode switching. Once
AP is in the final stage, AP can reload GDT to match BSP
Sent: Monday, February 22, 2021 21:22
To: devel@edk2.groups.io; fanjianf...@byosoft.com.cn; Ma, Maurice
; Patrick Rudolph ; Dong,
Guo ; Dong, Eric
Cc: You, Benjamin
Subject: RE: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
“But we could allocate room under 4G for
byosoft.com.cn; Ma, Maurice; Patrick
Rudolph; Dong, Guo; Dong, Eric
CC: You, Benjamin
Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
“But we could allocate room under 4G for GDT table directly in CpuDxe.”
The GDT pre-allocated is re-used by AP. Why do you sugges
, Maurice
; Patrick Rudolph ; Dong,
Guo ; Dong, Eric
Cc: devel@edk2.groups.io; You, Benjamin
Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Ray,
Yes. You are right. Acutally, x64 IDT table cannot work correctly on protected
mode. :-)
But for GDT
Date: 2021-02-23 10:52
To: fanjianf...@byosoft.com.cn; devel; Ma, Maurice; Patrick Rudolph; Dong, Guo;
Dong, Eric
CC: devel@edk2.groups.io; You, Benjamin
Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Jeff,
You are right that BSP’s GDT and IDT tables are
.groups.io; You, Benjamin
Subject: Re: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
4GB memory WA
we will save the current BSP's GDT and IDT for APs at first time APs are waken
by BSP as below. APs will start from real mode to protected mode and then to
long mode. D
Eric help me to confirm it?
Jeff
From: Ma, Maurice
Date: 2021-02-23 00:49
To: Patrick Rudolph; Dong, Guo; Dong, Eric; Ni, Ray
CC: devel@edk2.groups.io; You, Benjamin
Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove 4GB
memory WA
Hi, Ray and Eric,
Is there any reason
Cc: Ma, Maurice ; Dong, Eric ;
> Ni, Ray ; devel@edk2.groups.io; You, Benjamin
>
> Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> 4GB memory WA
>
> Hi Guo,
> I tested it and it works fine with your patch. The crash is gone.
>
> BTW: Are there a
; Dong, Eric ; Ni, Ray
> >
> > Cc: devel@edk2.groups.io; You, Benjamin
> > Subject: RE: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> > 4GB memory WA
> >
> > Hi, Ray and Eric,
> >
> > Is there any reason why the GDT base was typecast t
gt; > To: Dong, Guo
> > Cc: devel@edk2.groups.io; Ma, Maurice ; You,
> > Benjamin
> > Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry:
> > Remove 4GB memory WA
> >
> > Hi Guo,
> > I tested on 078400ee15e7b250e4dfafd840c2e0c19835e16b and
;
Thanks
Maurice
> -Original Message-
> From: Patrick Rudolph
> Sent: Monday, February 22, 2021 7:43
> To: Dong, Guo
> Cc: devel@edk2.groups.io; Ma, Maurice ; You,
> Benjamin
> Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry:
> Remove 4GB memory WA
Ma, Maurice
> > Cc: Dong, Guo ; You, Benjamin
> >
> > Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> > 4GB memory WA
> >
> > This patch breaks booting on master.
> > In CpuDxe.efi / InitGlobalDescriptorTable as the GDT poi
jamin
>
> Subject: Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> 4GB memory WA
>
> This patch breaks booting on master.
> In CpuDxe.efi / InitGlobalDescriptorTable as the GDT pointer is casted
> to 32bits.
>
> Regards,
> Patrick
>
&
> > From: Dong, Guo
> > Sent: Sunday, February 14, 2021 21:13
> > To: devel@edk2.groups.io
> > Cc: Ma, Maurice ; You, Benjamin
> >
> > Subject: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> > 4GB memory WA
> >
> > Previous i
Reviewed-by: Maurice Ma
Regards
Maurice
> -Original Message-
> From: Dong, Guo
> Sent: Sunday, February 14, 2021 21:13
> To: devel@edk2.groups.io
> Cc: Ma, Maurice ; You, Benjamin
>
> Subject: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadEntry: Remove
> 4GB
Previous it would hang in CpuDxe if DXE drivers are dispatched above 4GB.
Now remove the work around since the fixed in CpuDxe are merged.
Signed-off-by: Guo Dong
---
UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/UefiPayloadPkg/UefiPay
18 matches
Mail list logo
