[edk2-devel] [PATCH] FSP_TEMP_RAM_INIT API call must follow X64 Calling Convention

2022-05-11 Thread cbduggap
This API accept one parameter using RCX and this is consumed in mutiple sub functions. Signed-off-by: cbduggap --- IntelFsp2Pkg/FspSecCore/X64/FspApiEntryT.nasm | 23 +++--- .../Include/SaveRestoreSseAvxNasm.inc | 82 +++ 2 files changed, 95 insertions(+), 10 deletions(-)

Re: [edk2-devel] [PATCH 0/4] Refactor MpInitLib

2022-05-11 Thread Ni, Ray
Can you please help to review the changes and give a Reviewed-by? > -Original Message- > From: Tom Lendacky > Sent: Tuesday, May 10, 2022 11:13 PM > To: Ni, Ray ; devel@edk2.groups.io > Subject: Re: [edk2-devel] [PATCH 0/4] Refactor MpInitLib > > On 5/10/22 09:44, Tom Lendacky wrote: >

Re: [edk2-devel] [edk2:PATCH v2] MdePkg/Acpi62: Add type 7 NFIT Platform Capabilities Structure support

2022-05-11 Thread Zhiguang Liu
Reviewed-by: Zhiguang Liu > -Original Message- > From: Shindo, Miki > Sent: Thursday, May 12, 2022 9:28 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Gao, Liming > ; Liu, Zhiguang ; Ni, Ray > > Subject: [edk2:PATCH v2] MdePkg/Acpi62: Add type 7 NFIT Platform Capabilities >

[edk2-devel] [edk2:PATCH v2] MdePkg/Acpi62: Add type 7 NFIT Platform Capabilities Structure support

2022-05-11 Thread Miki Shindo
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3915 This commit adds a definition type 7 Platform Capabilities Structure for NFIT Table Structure Types. The type has been added since ACPI Specification Version 6.2A. Signed-off-by: Miki Shindo Cc: Michael D Kinney Cc: Liming Gao Cc:

Re: [edk2-devel] [PATCH V7 36/37] UefiCpuPkg: Setting initial-count register as the last step

2022-05-11 Thread Min Xu
On May 11, 2022 10:06 PM, Lendacky, Thomas wrote: > On 5/10/22 21:00, Xu, Min M wrote: > > On May 11, 2022 4:30 AM, Tom Lendacky wrote: > >> I'm replying to this patch since I can't find patch V12 46/47 > >> anywhere in my email. > >> > >> I've bisected a regression in the Linux kernel to this

[edk2-devel] [PATCH] IntelFsp2Pkg: Add FSP 2.3 header support

2022-05-11 Thread Tung Lun
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3921 This patch adds a couple of fields supported in FSP 2.3 header from both header generation and tool support perspective. Signed-off-by: Loo Tung Lun Cc: Nate DeSimone Cc: Star Zeng Cc: Chasel Chiu ---

Re: [edk2-devel] [PATCH 1/1] MdePkg/BaseLib: Add CRC16-ANSI and CRC32c implementations

2022-05-11 Thread Pedro Falcato
Ping. Could someone review it? On Mon, Apr 25, 2022 at 6:13 PM Pedro Falcato via groups.io wrote: > Ping. If someone could take a look, it would be much appreciated. > > On Thu, Apr 7, 2022 at 11:02 PM Pedro Falcato > wrote: > >> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3871 >> >>

Re: [edk2-devel] [PATCH edk2-platforms 0/3] Ext4Pkg: Add ext2/3 support and move crc16/32c to BaseLib

2022-05-11 Thread Pedro Falcato
Ping. Could someone review these patches? On Mon, Apr 25, 2022 at 6:14 PM Pedro Falcato via groups.io wrote: > Ping. If someone could take a look, it would be much appreciated. > > On Thu, Apr 7, 2022 at 11:01 PM Pedro Falcato > wrote: > >> BZ:

[edk2-devel] EDK2 fuzzing and possible roadmap

2022-05-11 Thread Pedro Falcato
Hi all, Reposting (https://bugzilla.tianocore.org/show_bug.cgi?id=3927) as this is more likely to gain attention and meaningful discussion here. Fuzzing EDK2 could provide a greater insight into the firmware and its safety/stability, by using automated guided testing. I took the time to look at

Re: [edk2-devel] [PATCH 00/10] Multiple packages: Remove RVCT support

2022-05-11 Thread Rebecca Cran
Could we get this pushed soon, so it's in plenty of time for the new stable tag? -- Rebecca Cran On 5/3/22 13:13, Ard Biesheuvel wrote: On Tue, 3 May 2022 at 20:48, Rebecca Cran wrote: RVCT is obsolete, no longer supported and only supports 32-bit systems. Since it's no longer used, remove

Re: [edk2-devel] [PATCH V7 36/37] UefiCpuPkg: Setting initial-count register as the last step

2022-05-11 Thread Lendacky, Thomas via groups.io
On 5/10/22 21:00, Xu, Min M wrote: On May 11, 2022 4:30 AM, Tom Lendacky wrote: On 2/28/22 01:21, Min Xu via groups.io wrote: BZ:

[edk2-devel] [PATCH] IntelSiliconPkg/VTd: Add PCD for VTd Abort DMA Mode Support

2022-05-11 Thread Sheng Wei
PcdVTdSupportAbortDmaMode is used to enable/disable VTd Abort DMA Mode. Signed-off-by: Sheng Wei Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Jenny Huang Cc: Robert Kowalewski --- .../Feature/VTd/IntelVTdDmarPei/IntelVTdDmar.c | 7 +--

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-11 Thread Yao, Jiewen
Merged https://github.com/tianocore/edk2/pull/2877 > -Original Message- > From: Xu, Min M > Sent: Monday, May 9, 2022 8:45 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Brijesh > Singh > ; Aktas, Erdem ; James > Bottomley ; Yao, Jiewen ; Tom > Lendacky ; Gerd Hoffmann >

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-11 Thread Yao, Jiewen
Acked-by: Jiewen Yao > -Original Message- > From: Xu, Min M > Sent: Monday, May 9, 2022 8:45 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Brijesh > Singh > ; Aktas, Erdem ; James > Bottomley ; Yao, Jiewen ; Tom > Lendacky ; Gerd Hoffmann > Subject: RE: [PATCH V2 0/6]

Re: [edk2-devel] [PATCH V4 0/5] CryptoPkg/openssl: Enable EC conditionally.

2022-05-11 Thread Yao, Jiewen
Merged https://github.com/tianocore/edk2/pull/2871 > -Original Message- > From: Li, Yi1 > Sent: Tuesday, May 10, 2022 3:19 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian > J ; Lu, Xiaoyu1 ; Jiang, Guomin > ; Gerd Hoffmann > Subject: [PATCH V4 0/5]

Re: [edk2-devel] [PATCH V4 0/5] CryptoPkg/openssl: Enable EC conditionally.

2022-05-11 Thread Yao, Jiewen
Reviewed-by: Jiewen Yao > -Original Message- > From: Li, Yi1 > Sent: Tuesday, May 10, 2022 3:19 PM > To: devel@edk2.groups.io > Cc: Li, Yi1 ; Yao, Jiewen ; Wang, Jian > J ; Lu, Xiaoyu1 ; Jiang, Guomin > ; Gerd Hoffmann > Subject: [PATCH V4 0/5] CryptoPkg/openssl: Enable EC

Re: [edk2-devel] [PATCH V2 2/5] CryptoPkg: Separate auto-generated openssl config and edk2 openssl config

2022-05-11 Thread yi1 li
It will change it. For example if we want to make SM2 conditional also, we can change config list to: my %conditional_feature = ("PcdOpensslEcEnabled"=>["EC", "ECDH", "ECDSA", "TLS1_3"], "PcdOpensslSM2Enabled=>["SM2"]); my %conditional_feature_dir = ("/ec/"=>"PcdOpensslEcEnabled",

Re: [edk2-devel] [PATCH V2 2/5] CryptoPkg: Separate auto-generated openssl config and edk2 openssl config

2022-05-11 Thread Gerd Hoffmann
On Tue, May 10, 2022 at 11:43:54AM +, Li, Yi1 wrote: > Just for convenience. > There are too many places need to configure, so I think putting all the > configuration options close together, all in process_file.pl, will be more > clear to user ;) Well, I find it confusing to let