In case PcdFirmwareReleaseDateString is not set use a valid date
as fallback. Using "unknown" makes Windows unhappy.
Fixes: 4cb94f20b002 ("OvmfPkg/SmbiosPlatformDxe: use PcdFirmware*")
Reported-by: ruifeng@intel.com
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDx
Ping. Two reviews and the freeze is over. Merge this now?
On Wed, Feb 15, 2023 at 03:37:32PM +0100, Gerd Hoffmann wrote:
> SetMem arguments 2+3 are in the wrong order, resulting in
> the call having no effect because Length is zero.
>
> Fix this by using ZeroMem instead.
>
> Bugzilla: https://
On Thu, 9 Mar 2023 at 10:16, Gerd Hoffmann wrote:
>
> Ping. Two reviews and the freeze is over. Merge this now?
>
Pong. Thanks for the reminder - I'll go and merge this.
> On Wed, Feb 15, 2023 at 03:37:32PM +0100, Gerd Hoffmann wrote:
> > SetMem arguments 2+3 are in the wrong order, resulting
On Thu, 9 Mar 2023 at 10:02, Gerd Hoffmann wrote:
>
> In case PcdFirmwareReleaseDateString is not set use a valid date
> as fallback. Using "unknown" makes Windows unhappy.
>
> Fixes: 4cb94f20b002 ("OvmfPkg/SmbiosPlatformDxe: use PcdFirmware*")
> Reported-by: ruifeng@intel.com
> Signed-off-by
Looking at adding support for Risc-V for UPL. In the process realised
there are several initialisation sequences which need not be part of
common UPL entry flow. The flow should be agnostic to both BL and Arch
with required hooks provided along the boot path. This patch set
addresses 2 such instanc
According to UPL spec BL should initialize FP init meaning UPL
does not need to initialize it. Besides this is arch specific init
and needs to be moved out of UPL common flow. In order to not break
current BL implementations, for now just moving the init to later
point of time but for both x32 and
8259 is very arch specific programming. It needs to be moved out to
the respective arch flow. Added in both x64 and x32 paths
Test: Able to boot UEFI shell with Coreboot Tianocore payload on
x86 qemu
Cc: Guo Dong
Cc: Ray Ni
Cc: Sean Rhodes
Cc: James Lu
Cc: Gua Guo
Signed-off-by: Dhaval Shar
Hi Girish,
Thank you for your feedback.
Please find my response inline marked [SAMI].
Regards,
Sami Mujawar
On 08/03/2023, 17:41, "Girish Mahadevan" mailto:gmahade...@nvidia.com>> wrote:
Hi Sami
Thanks for v2, I will apply these to my tree and test it out.
One small comment before I revie
Hi Girish,
On 08/03/2023, 19:21, "Girish Mahadevan" mailto:gmahade...@nvidia.com>> wrote:
Hi Sami,
Thanks for your response.
I will update my patch series based on v2 of your dispatcher code and
point you to v3. (mid/late next week)
Can you also merge the Smbios String related patch (th
v3 changes:
- rebase to latest master.
- enable crypto driver only for SMM + DXE.
- CI passes now \o/
v2 changes:
- turn on crypto driver support by default.
- left the config option in for now as fallback option.
When all goes as planned remove it one or two releases
later.
- fix vario
Makes it easier to reuse the predefined config sets in other places.
Signed-off-by: Gerd Hoffmann
---
.../Include/Dsc/CryptoServicePcd.all.dsc.inc | 29 +++
.../Dsc/CryptoServicePcd.min_dxe_smm.dsc.inc | 35 +
.../Dsc/CryptoServicePcd.min_pei.dsc.inc | 20 +
CryptoPkg/Cryp
Contains only hash functions needed for measurements.
Signed-off-by: Gerd Hoffmann
---
CryptoPkg/Include/Dsc/CryptoServicePcd.hash_only.dsc.inc | 8
1 file changed, 8 insertions(+)
create mode 100644 CryptoPkg/Include/Dsc/CryptoServicePcd.hash_only.dsc.inc
diff --git a/CryptoPkg/Inclu
Enable Sha384 + Sha512 + Sm3.
Signed-off-by: Gerd Hoffmann
---
CryptoPkg/Include/Dsc/CryptoServicePcd.min_dxe_smm.dsc.inc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/CryptoPkg/Include/Dsc/CryptoServicePcd.min_dxe_smm.dsc.inc
b/CryptoPkg/Include/Dsc/CryptoServicePcd.mi
Signed-off-by: Gerd Hoffmann
---
BaseTools/Conf/tools_def.template | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/BaseTools/Conf/tools_def.template
b/BaseTools/Conf/tools_def.template
index 471eb67c0c83..7f73ed817cd9 100755
--- a/BaseTools/Conf/tools_def.template
+++
Create include files for crypto support, so the configuration can be
shared for all OVMF build variants. Also add support for using the
Crypto Driver.
The Crypto Driver is by default for enabled SMM + DXE and disabled for
PEI. This can be changed using the {PEI,SMM,DXE}_USE_CRYPTO_DRIVER
options
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/OvmfPkgIa32X64.dsc | 22 +-
OvmfPkg/OvmfPkgIa32X64.fdf | 6 ++
2 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
inde
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/OvmfPkgX64.dsc | 22 +-
OvmfPkg/OvmfPkgX64.fdf | 6 ++
2 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
index 04d50704c736..
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/Microvm/MicrovmX64.dsc | 24 +++-
OvmfPkg/Microvm/MicrovmX64.fdf | 7 +++
2 files changed, 18 insertions(+), 13 deletions(-)
diff --git a/OvmfPkg/Microvm/MicrovmX64.dsc b/OvmfPkg/Microvm/
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/AmdSev/AmdSevX64.dsc | 12
OvmfPkg/AmdSev/AmdSevX64.fdf | 6 ++
2 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/OvmfPkg/AmdSev/AmdSevX64.dsc b/OvmfPkg/AmdSev/AmdSevX64.dsc
index f
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/IntelTdx/IntelTdxX64.dsc | 15 +--
OvmfPkg/IntelTdx/IntelTdxX64.fdf | 5 +
2 files changed, 14 insertions(+), 6 deletions(-)
diff --git a/OvmfPkg/IntelTdx/IntelTdxX64.dsc b/OvmfPkg/IntelTdx/Intel
Use the new crypto support include files.
Signed-off-by: Gerd Hoffmann
---
OvmfPkg/OvmfPkgIa32.dsc | 22 +-
OvmfPkg/OvmfPkgIa32.fdf | 6 ++
2 files changed, 15 insertions(+), 13 deletions(-)
diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
index 22dc29330d
Hi Pierre,
On Tue, Oct 4, 2022 at 01:22 AM, PierreGondois wrote:
>
>
>>
>>> +EFI_STATUS
>>> +EFIAPI
>>> +StringTablePublishStringSet (
>>> + IN STRING_TABLE *CONST StrTable,
>>> + IN CHAR8 *CONST SmbiosStringAreaStart,
>>> + IN CONST UINTN SmbiosStringAreaSize
>>> + )
>>> +{
>>> + UINT8 Index;
Reviewed-by: Gua Guo
-Original Message-
From: Dhaval Sharma
Sent: Thursday, March 9, 2023 6:37 PM
To: devel@edk2.groups.io
Cc: Dong, Guo ; Ni, Ray ; Sean Rhodes
; Lu, James ; Guo, Gua
Subject: [PATCH v3 2/2] UefiPayloadPkg: Move INT prog outside common flow
8259 is very arch specif
We are not updating the BMC status in the switch()( *default case* ), which may
lead to installing the IPMI protocol for the failure case too. Could you please
initialize the BMC status in the default case also?
switch ( SelfTestResult->Result ) {
case IPMI_APP_SELFTEST_NO_ERROR:
case IPMI_APP
Thanks Christine.
Reviewed-by: Bob Feng
-Original Message-
From: Chen, Christine
Sent: Thursday, March 9, 2023 10:55 AM
To: devel@edk2.groups.io
Cc: Kinney, Michael D ; Feng, Bob C
; Gao, Liming
Subject: [PATCH] Maintainers: Add 'Yuwei Chen' as BaseTools maintainer.
Add myself as ma
Hi Abner,
I'm not sure if I understand this module completely.
This module is another IPMI solution? we will utilize either
ManageabilityPkg (using IPMI protocol specified in the EDK2 repo) or
IpmiFeaturePkg (using IpmiTransportProtocol). How do I use
IpmiCommandLib
(IpmiFeaturePkg/Library/I
From: Pierre Gondois
The topology of a platform is represented in ACPI using the PPTT
table. It is possible to append information to CPUs/processor
containers using their associated AML nodes in a SSDT
table.
A platform might have multiple 'physical packages' (or top-level
nodes) in their PPTT to
From: Abner Chang
BZ #: 4282
Implement the platform Redfish host interface library that
uses IPMI commands to get the network information of
BMC-exposed USB NIC for building up the SMBIOS type 42
record.
The prerequisites: BZ #4213, 4214 and 4231
Check "Platform with BMC and the BMC-Exposed USB
From: Abner Chang
Update Redfish DSC for
PlatformHostInterfaceBmcUsbNicLib.
Signed-off-by: Abner Chang
Cc: Nickle Wang
Cc: Igor Kulchytskyy
---
RedfishPkg/RedfishPkg.dsc | 7 +++
1 file changed, 7 insertions(+)
diff --git a/RedfishPkg/RedfishPkg.dsc b/RedfishPkg/RedfishPkg.dsc
index cf2
From: Abner Chang
BMC exposed USB NIC platform Redfish Host Interface
library implementation.
Signed-off-by: Abner Chang
Cc: Nickle Wang
Cc: Igor Kulchytskyy
---
RedfishPkg/RedfishPkg.dec | 10 +
.../PlatformHostInterfaceBmcUsbNicLib.inf | 48 +
RedfishPkg/Include
Could I get some reviews on this please?
Thanks.
Rebecca Cran
On 2/16/23 8:50 PM, Rebecca Cran wrote:
Currently, the BaseTools Makefiles use BUILD_CC and BUILD_CXX, which
doesn't allow users to override the compiler to use in the expected way
by running e.g. "make CC=clang-17 CXX=clang++-17"
Function defination should match with declaration.
[-Wlto-type-mismatch]
Cc: Chasel Chiu
Cc: Nate DeSimone
Cc: Sai Chaganty
Cc: Star Zeng
Signed-off-by: Ashraf Ali S
---
IntelFsp2Pkg/Library/BaseFspPlatformLib/FspPlatformNotify.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/IntelFsp2
Thanks for fixing this bug!
Reviewed-by: Chasel Chiu
> -Original Message-
> From: S, Ashraf Ali
> Sent: Thursday, March 9, 2023 8:06 AM
> To: devel@edk2.groups.io
> Cc: S, Ashraf Ali ; Chiu, Chasel
> ;
> Desimone, Nathaniel L ; Chaganty, Rangasai
> V ; Zeng, Star
> Subject: [PATCH
Reviewed-by: Michael D Kinney
> -Original Message-
> From: Chen, Christine
> Sent: Wednesday, March 8, 2023 6:55 PM
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D ; Feng, Bob C
> ; Gao, Liming
>
> Subject: [PATCH] Maintainers: Add 'Yuwei Chen' as BaseTools maintainer.
>
> Add myse
Hi All,Any updates on this patch? Thanks, From: Tuan Phan via groups.ioSent: Monday, March 6, 2023 9:11 AMTo: devel@edk2.groups.ioCc: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn; zhiguang@intel.com; suni...@ventanamicro.com; g...@danielschaefer.me; Tuan PhanSubject: [edk2-devel] [PATCH
Hi All,Any updates on this series? Thanks, From: Tuan PhanSent: Monday, March 6, 2023 9:33 AMTo: devel@edk2.groups.ioCc: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn; zhiguang@intel.com; suni...@ventanamicro.com; g...@danielschaefer.me; andrei.warken...@intel.com; Tuan PhanSubject: [PAT
From: Sean Brogan
Create SECURITY.md security policy for tianocore edk2 leveraging CVD and
the Github Private Vulnerability Reporting process.
Co-authored-by: Sean Brogan
Signed-off-by: Kun Qin
---
SECURITY.md | 33
1 file changed, 33 insertions(+)
diff --git a/SECURITY.
This change added a markdown file as a policy guideline for Tianocore EDK2
community to handle security sensitive reports.
Patch v1 branch: https://github.com/kuqin12/edk2/tree/patch-1
Cc: Andrew Fish
Cc: Leif Lindholm
Cc: Michael D Kinney
Cc: Miki Demeter
Cc: Sean Brogan
Sean Brogan (1):
Hi Tuan,
Could you share a GitHub link to a branch with the patch set? Somehow my email
client is mangling one of your patches where it’s all one giant line of code.
A
From: Tuan Phan
Sent: Thursday, March 9, 2023 1:20 PM
To: devel@edk2.groups.io
Cc: Kinney, Michael D ; Gao, Liming
; Liu, Zhi
Reviewed-by: Nate DeSimone
-Original Message-
From: S, Ashraf Ali
Sent: Thursday, March 9, 2023 8:06 AM
To: devel@edk2.groups.io
Cc: S, Ashraf Ali ; Chiu, Chasel
; Desimone, Nathaniel L
; Chaganty, Rangasai V
; Zeng, Star
Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Fu
Reviewed-by: Sai Chaganty
-Original Message-
From: S, Ashraf Ali
Sent: Thursday, March 09, 2023 8:06 AM
To: devel@edk2.groups.io
Cc: S, Ashraf Ali ; Chiu, Chasel
; Desimone, Nathaniel L
; Chaganty, Rangasai V
; Zeng, Star
Subject: [PATCH] [IntelFsp2Pkg]: Fix GCC Compiler warning
Fu
Reviewed-by: Star Zeng with typo defination -> definition
fixed in commit message.
-Original Message-
From: Chaganty, Rangasai V
Sent: Friday, March 10, 2023 8:27 AM
To: S, Ashraf Ali ; devel@edk2.groups.io
Cc: Chiu, Chasel ; Desimone, Nathaniel L
; Zeng, Star
Subject: RE: [PATCH] [I
Tuan:
Can __FUNCTION__ be used? If yes, please still keep it.
Thanks
Liming
发件人: devel@edk2.groups.io 代表 Tuan Phan
发送时间: 2023年3月10日 3:19
收件人: devel@edk2.groups.io
抄送: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn;
zhiguang@intel.com; suni...@ventanamicro.com; g...@danielschaef
Dun:
Is there any code impact with this change?
Thanks
Liming
> -邮件原件-
> 发件人: Dun Tan
> 发送时间: 2023年3月9日 11:40
> 收件人: devel@edk2.groups.io
> 抄送: Michael D Kinney ; Liming Gao
> ; Zhiguang Liu
> 主题: [Patch V3 1/2] MdePkg: modify the wrong 'AcpiId' to 'ApicId' in
> MpWakeupStructure
>
>
PR https://github.com/tianocore/edk2/pull/4127 is created for this patch.
Thanks
Liming
发件人: devel@edk2.groups.io 代表 gaoliming via groups.io
发送时间: 2023年3月9日 8:55
收件人: devel@edk2.groups.io; ray...@intel.com; 'Tan, Lean Sheng'
抄送: 'Wang, Jian J'
主题: 回复: [edk2-devel] [PATCH 1/3] MdeModulePk
FVMAIN_COMPACT is not reported to DXE. So, it can’t be found in DXE phase.
Thanks
Liming
发件人: devel@edk2.groups.io 代表 Alireza Banejad
发送时间: 2023年3月2日 20:16
收件人: devel@edk2.groups.io; alibanejad1...@gmail.com
主题: Re: [edk2-devel] GetSectionFromAnyFv doesnt find file from FV
So, I managed
Pawel:
The change in MdeModulePkg looks good to me. Reviewed-by: Liming Gao
Thanks
Liming
> -邮件原件-
> 发件人: devel@edk2.groups.io 代表 Pawe?
> Po?awski
> 发送时间: 2023年2月17日 21:02
> 收件人: devel@edk2.groups.io
> 主题: [edk2-devel] PATCH v3 0/3 Add xterm resize support to OVMF
>
> This set of pat
Rebecca:
This patch pass code review. It can be merged now. The second change to using
__func__ is only for MdePkg. Do you expect the change in MdePkg is first merged?
Thanks
Liming
> -邮件原件-
> 发件人: Rebecca Cran
> 发送时间: 2023年3月7日 0:39
> 收件人: devel@edk2.groups.io; quic_rc...@quicinc.com;
Rebecca:
I add my comments below.
> -邮件原件-
> 发件人: Rebecca Cran
> 发送时间: 2023年2月15日 3:07
> 收件人: devel@edk2.groups.io; Michael D Kinney
> ; Liming Gao ;
> Zhiguang Liu
> 抄送: Rebecca Cran
> 主题: [PATCH 1/1] MdePkg: Add new JedecJep106Lib to fetch JEDEC JEP106
> manufacturer
>
> Add a new
Rebecca:
I think these changes are good. Reviewed-by: Liming Gao
Besides, which tool is used to convert function comments to Doxygen
format?
Thanks
Liming
> -邮件原件-
> 发件人: Rebecca Cran
> 发送时间: 2023年2月25日 8:54
> 收件人: devel@edk2.groups.io; Bob Feng ; Liming
> Gao ; Yuwei Chen
> 抄送: Re
Rebecca:
Why new file name is not SdramSpdDdr5.h?
Thanks
Liming
> -邮件原件-
> 发件人: devel@edk2.groups.io 代表 Rebecca Cran
> 发送时间: 2023年2月14日 23:37
> 收件人: Kinney, Michael D ;
> devel@edk2.groups.io; Gao, Liming
> 抄送: 'Liming Gao' ; Liu, Zhiguang
>
> 主题: Re: [edk2-devel] [PATCH 1/1] MdePkg:
On Fri, Mar 10, 2023 at 10:13:42AM +0800, gaoliming via groups.io wrote:
> Tuan:
>
> Can __FUNCTION__ be used? If yes, please still keep it.
>
Hi Liming,
I had suggested this. There are efforts to move to using __func__.
https://edk2.groups.io/g/devel/message/99904
I think, we better start
Fix patch has been pushed with commit message typo and format correction:
https://github.com/tianocore/edk2/commit/8820767fb3bad09eeedecc3030d75c9e0cd4cab7
Thanks,
Chasel
> -Original Message-
> From: S, Ashraf Ali
> Sent: Thursday, March 9, 2023 8:06 AM
> To: devel@edk2.groups.io
> C
+ @UefiCpuPkg Maintainers
Requesting an estimated ETA on integration of this 1 character patch.
I understand the patch queue is huge, just looking to provide a realistic
expectation to those who are blocked by this and waiting for me to provide an
update.
Warm Regards,
Darbin
_
Reviewed-by: Eric Dong
-Original Message-
From: devel@edk2.groups.io On Behalf Of Darbin Reyes
Sent: Wednesday, March 8, 2023 7:04 AM
To: devel@edk2.groups.io
Cc: Reyes, Darbin ; Narey, Jacob
Subject: [edk2-devel] [PATCH] UefiCpuPkg/MicrocodeMeasurementDxe: Fix exception
REF: https://b
Merged https://github.com/tianocore/edk2/pull/4128
> -Original Message-
> From: Sheng, W
> Sent: Tuesday, February 28, 2023 10:44 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Yao, Jiewen
> ; Lu, Xiaoyu1 ; Jiang,
> Guomin
> Subject: [PATCH v2] CryptoPkg/OpensslLib: Upgrade OpenSSL t
On Mon, Mar 06, 2023 at 09:10:58AM -0800, Tuan Phan wrote:
> When the range instruction cache invalidating not supported, the whole
> instruction cache should be invalidated instead.
>
> Signed-off-by: Tuan Phan
> ---
> MdePkg/Library/BaseCacheMaintenanceLib/RiscVCache.c | 5 -
> 1 file chan
Hi Liming,
I've checked edk2 and edk2-platforms repo. This code change doesn't impacted
any code.
The MultiprocessorWakeupMailbox structure has not been used yet.
Thanks,
Dun
-Original Message-
From: gaoliming
Sent: Friday, March 10, 2023 10:16 AM
To: Tan, Dun ; devel@edk2.groups.io
On Fri, Mar 03, 2023 at 12:04:08PM -0600, Andrei Warkentin wrote:
> Add a few more definitions and make SbiCall and TranslateError
> usable (not static) by library users.
>
> Cc: Daniel Schaefer
> Cc: Sunil V L
> Acked-by: Gerd Hoffmann
> Signed-off-by: Andrei Warkentin
> ---
Reviewed-by: Sun
On Fri, Mar 03, 2023 at 12:04:10PM -0600, Andrei Warkentin wrote:
> If the SerialPortLib had any initialization needed, this
> would be skipped in the RiscVVirt Sec. Follow the example
> seen elsewhere (ArmVirtPkg PrePi).
>
> Seen with BaseSerialPortLibRiscVSbi not using DBCN in Sec, yet
> using D
Merged
PR: https://github.com/tianocore/edk2/pull/4129
Commit:
https://github.com/tianocore/edk2/commit/fd1820b7ea09e53e404d6f56bb8bc4b51a5dd83e
Mike
From: devel@edk2.groups.io On Behalf Of Darbin Reyes
Sent: Thursday, March 9, 2023 8:31 PM
To: devel@edk2.groups.io; Reyes, Darbin ; Dong, Eric
On Fri, Mar 03, 2023 at 12:04:09PM -0600, Andrei Warkentin wrote:
> These are implementations of SerialPortLib using SBI console services.
> - SecPeiRiscVSerialPortLibSbi is appropriate for SEC/PEI (XIP) environments
> - PrePiDxeRiscVSerialPortLibSbi is appropriate for PrePI/DXE environments
>
> T
Hello Sami,
On 3/9/23 12:11, Sami Mujawar wrote:
Hi Pierre,
On Tue, Oct 4, 2022 at 01:22 AM, PierreGondois wrote:
+EFI_STATUS
+EFIAPI
+StringTablePublishStringSet (
+ IN STRING_TABLE *CONST StrTable,
+ IN CHAR8 *CONST SmbiosStringArea
63 matches
Mail list logo