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

2022-05-11 Thread gaoliming
Yes. This patch has passed code review before soft feature freeze. So, it can be merged for this stable tag. Thanks Liming > -邮件原件- > 发件人: Rebecca Cran > 发送时间: 2022年5月12日 1:37 > 收件人: Ard Biesheuvel > 抄送: edk2-devel-groups-io ; Leif Lindholm > ; Ard Biesheuvel ; > Sami Mujawar ; Gerd Hof

[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 > Str

[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: Zhigu

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 patc

[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 --- IntelFsp2Pkg/Tools/ConfigEditor/Con

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 >> >> Add

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: https://bugzilla.tianocore.org/show_bug.cgi?id=374

[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 i

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: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D3711&data=05%7C01%7Cthomas.lendacky%40amd.com%7Cce8cd

[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 +-- .../Feature/VTd/IntelVTdDmarPei/IntelVTdDmarP

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 > S

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] Supp

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] CryptoPkg/opens

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 conditionall

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", "/sm2/"=>"PcdO

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 process_file