public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-built binaries
       [not found] <176A8D14D53D9014.14466@groups.io>
@ 2023-06-21  6:01 ` Ni, Ray
  2023-06-22 12:54   ` Dong, Eric
  0 siblings, 1 reply; 2+ messages in thread
From: Ni, Ray @ 2023-06-21  6:01 UTC (permalink / raw)
  To: Liu, Zhiguang, Dong, Eric; +Cc: devel@edk2.groups.io, Ni, Ray

I updated the patch per Zhiguang's comments to remove ALIGN 16 requirement.
Can you please review again?

Only the 2nd patch is updated.

Thanks,
Ray

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray
> Sent: Wednesday, June 21, 2023 10:55 AM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-
> built binaries
> 
> Ray Ni (3):
>   UefiCpuPkg: Include ResetVector in DSC
>   UefiCpuPkg/ResetVector: Add guidance of FDF ffs rule
>   UefiCpuPkg/ResetVector: Remove pre-built binaries
> 
>  .../Vtf0/Bin/IA32/ResetVector.ia32.port80.raw | Bin 532 -> 0 bytes
>  .../Vtf0/Bin/IA32/ResetVector.ia32.raw        | Bin 500 -> 0 bytes
>  .../Vtf0/Bin/IA32/ResetVector.ia32.serial.raw | Bin 900 -> 0 bytes
>  .../ResetVector/Vtf0/Bin/ResetVector.inf      |  31 ------
>  .../ResetVector/Vtf0/Bin/ResetVector.uni      |  16 ----
>  .../ResetVector/Vtf0/Bin/ResetVector1G.inf    |  31 ------
>  .../ResetVector/Vtf0/Bin/ResetVectorExtra.uni |  12 ---
>  .../PageTable1G/ResetVector.x64.port80.raw    | Bin 12292 -> 0 bytes
>  .../Bin/X64/PageTable1G/ResetVector.x64.raw   | Bin 12292 -> 0 bytes
>  .../PageTable1G/ResetVector.x64.serial.raw    | Bin 12292 -> 0 bytes
>  .../PageTable2M/ResetVector.x64.port80.raw    | Bin 28676 -> 0 bytes
>  .../Bin/X64/PageTable2M/ResetVector.x64.raw   | Bin 28676 -> 0 bytes
>  .../PageTable2M/ResetVector.x64.serial.raw    | Bin 28676 -> 0 bytes
>  UefiCpuPkg/ResetVector/Vtf0/Build.py          |  89 ------------------
>  UefiCpuPkg/ResetVector/Vtf0/ReadMe.txt        |  27 ++----
>  .../Vtf0/Tools/FixupForRawSection.py          |  20 ----
>  UefiCpuPkg/ResetVector/Vtf0/Vtf0.inf          |  28 +++++-
>  UefiCpuPkg/UefiCpuPkg.dsc                     |   4 +-
>  18 files changed, 36 insertions(+), 222 deletions(-)
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.port80.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.serial.raw
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.uni
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector1G.inf
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVectorExtra.uni
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.port80
> .raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.serial.r
> aw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.port8
> 0.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.serial.r
> aw
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Build.py
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Tools/FixupForRawSection.py
> 
> --
> 2.39.1.windows.1
> 
> 
> 
> 
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-built binaries
  2023-06-21  6:01 ` [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-built binaries Ni, Ray
@ 2023-06-22 12:54   ` Dong, Eric
  0 siblings, 0 replies; 2+ messages in thread
From: Dong, Eric @ 2023-06-22 12:54 UTC (permalink / raw)
  To: Ni, Ray, Liu, Zhiguang; +Cc: devel@edk2.groups.io

Reviewed-by: Eric Dong <eric.dong@intel.com>

-----Original Message-----
From: Ni, Ray <ray.ni@intel.com> 
Sent: Wednesday, June 21, 2023 2:01 PM
To: Liu, Zhiguang <zhiguang.liu@intel.com>; Dong, Eric <eric.dong@intel.com>
Cc: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>
Subject: RE: [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-built binaries

I updated the patch per Zhiguang's comments to remove ALIGN 16 requirement.
Can you please review again?

Only the 2nd patch is updated.

Thanks,
Ray

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray
> Sent: Wednesday, June 21, 2023 10:55 AM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove 
> pre- built binaries
> 
> Ray Ni (3):
>   UefiCpuPkg: Include ResetVector in DSC
>   UefiCpuPkg/ResetVector: Add guidance of FDF ffs rule
>   UefiCpuPkg/ResetVector: Remove pre-built binaries
> 
>  .../Vtf0/Bin/IA32/ResetVector.ia32.port80.raw | Bin 532 -> 0 bytes
>  .../Vtf0/Bin/IA32/ResetVector.ia32.raw        | Bin 500 -> 0 bytes
>  .../Vtf0/Bin/IA32/ResetVector.ia32.serial.raw | Bin 900 -> 0 bytes
>  .../ResetVector/Vtf0/Bin/ResetVector.inf      |  31 ------
>  .../ResetVector/Vtf0/Bin/ResetVector.uni      |  16 ----
>  .../ResetVector/Vtf0/Bin/ResetVector1G.inf    |  31 ------
>  .../ResetVector/Vtf0/Bin/ResetVectorExtra.uni |  12 ---
>  .../PageTable1G/ResetVector.x64.port80.raw    | Bin 12292 -> 0 bytes
>  .../Bin/X64/PageTable1G/ResetVector.x64.raw   | Bin 12292 -> 0 bytes
>  .../PageTable1G/ResetVector.x64.serial.raw    | Bin 12292 -> 0 bytes
>  .../PageTable2M/ResetVector.x64.port80.raw    | Bin 28676 -> 0 bytes
>  .../Bin/X64/PageTable2M/ResetVector.x64.raw   | Bin 28676 -> 0 bytes
>  .../PageTable2M/ResetVector.x64.serial.raw    | Bin 28676 -> 0 bytes
>  UefiCpuPkg/ResetVector/Vtf0/Build.py          |  89 ------------------
>  UefiCpuPkg/ResetVector/Vtf0/ReadMe.txt        |  27 ++----
>  .../Vtf0/Tools/FixupForRawSection.py          |  20 ----
>  UefiCpuPkg/ResetVector/Vtf0/Vtf0.inf          |  28 +++++-
>  UefiCpuPkg/UefiCpuPkg.dsc                     |   4 +-
>  18 files changed, 36 insertions(+), 222 deletions(-)  delete mode 
> 100644 
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.port80.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/IA32/ResetVector.ia32.serial.raw
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.uni
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector1G.inf
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVectorExtra.uni
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.port80
> .raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable1G/ResetVector.x64.serial
> .r
> aw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.port8
> 0.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.raw
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Bin/X64/PageTable2M/ResetVector.x64.serial
> .r
> aw
>  delete mode 100644 UefiCpuPkg/ResetVector/Vtf0/Build.py
>  delete mode 100644
> UefiCpuPkg/ResetVector/Vtf0/Tools/FixupForRawSection.py
> 
> --
> 2.39.1.windows.1
> 
> 
> 
> 
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-06-22 12:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <176A8D14D53D9014.14466@groups.io>
2023-06-21  6:01 ` [edk2-devel] [PATCH v2 0/3] UefiCpuPkg/ResetVector: Remove pre-built binaries Ni, Ray
2023-06-22 12:54   ` Dong, Eric

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox