BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1872
Since MultipleWorkspace is not imported in FvImageSection,
an error of "global name 'MWS' is not defined" appeared.
This patch is going to fix that issue.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Zhiju.Fan
---
BaseTools/Source/Python/G
Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1926
This problem was found by Rebecca Cran .
REF: https://edk2.groups.io/g/devel/topic/32100684
OpenSSL will automatically load a system config file which configures
default ssl options. In UEFI, It will cause TlsInitialize failed
without
Reviewed-by: Nate DeSimone
-Original Message-
From: Chiu, Chasel
Sent: Sunday, June 16, 2019 7:42 PM
To: devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Zeng, Star
Subject: [PATCH v2] IntelFsp2Pkg: add TempRamExitPpi.h.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1883
Add he
Hi All,
The decision to use FSP_TEMP_RAM_EXIT_PPI for both FSP and non-FSP builds is a
platform level decision. There are several EDK2 based UEFI implementations that
do not use FSP_TEMP_RAM_EXIT_PPI, OvmfPkg, Minnow, and Quark are some of many
examples. There is nothing in the UEFI PI spec or
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678
Use the PcdPlatformRecoverySupport to control the function
of platform recovery in BDS.
First, set the variable's ("OsIndicationsSupported")
EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit base on the pcd.
It would affect the variable "OsIndi
V1:
Add a pcd PcdPlatformRecoverySupport to control the variable
PlatformRecovery and the EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY
bit of variable "OsIndicationsSupported".
V2:
While PcdPlatformRecoverySupport is FALSE, do not set a PlatformRecovery
Variable.
But remain boot from a defau
From: Bret Barkelew
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678
Add a pcd PcdPlatformRecoverySupport to conditionally
set an OS indications bit and set the 'PlatformRecovery'
variable. This pcd would also control whether the BDS
support platform recovery or not.
Update the uni f
Reviewed-by: Nate DeSimone
-Original Message-
From: Chiu, Chasel
Sent: Friday, June 14, 2019 1:25 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
; Gao, Liming
Subject: [PATCH] KabylakeOpenBoardPkg: Remove PeiMain for Dispatch mode.
REF: https://bugzilla.tia
Hi Ray,
Currently we prefer to duplicate header files so we do not have IntelFsp2Pkg
dependency for non-FSP build.
We will review for how to support FSP/non-FSP builds better.
Thanks!
Chasel
> -Original Message-
> From: Ni, Ray
> Sent: Monday, June 17, 2019 11:27 AM
> To: devel@edk2.
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, June 13, 2019 1:32 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
; Gao, Liming ; Feng, Bob
C
Subject: [edk2-devel] [PATCH v2] Platform/Intel:
Hi Chasel,
Please update the comments for the TempRamExit() function declaration to
include all of the FSP specification allowed return values:
EFI_SUCCESS - FSP execution environment was initialized successfully.
EFI_INVALID_PARAMETER - Input parameters are invalid.
EFI_UNSUPPORTED - The FSP ca
Reviewed-by: Eric Dong
> -Original Message-
> From: Ni, Ray
> Sent: Wednesday, June 19, 2019 4:58 PM
> To: devel@edk2.groups.io; Ni, Ray
> Cc: Dong, Eric
> Subject: RE: [edk2-devel] [PATCH] UefiCpuPkg/MpService: GetProcessorInfo
> returns 6-level topology
>
> This change will be pushed
I am so sorry for my mistake.
There are still ECC errors in the change of uni file. The additional string
should be
STR_gEfiMdeModulePkgTokenSpaceGuid_PcdPlatformRecoverySupport_PROMPT
STR_gEfiMdeModulePkgTokenSpaceGuid_PcdPlatformRecoverySupport_HELP
I would update the patch.
Thanks,
Zhichao
>
https://bugzilla.tianocore.org/show_bug.cgi?id=1617
Cc: Chao Zhang
Cc: Jiewen Yao
Cc: "Hernandez Beltran, Jorge"
Cc: Harry Han
Signed-off-by: Jian J Wang
---
SecurityPkg/SecurityPkg.dsc | 5 +
1 file changed, 5 insertions(+)
diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/Securit
> v4: change FV_HASH_FLAG_BOOT_MODE to align with
> HASHED_FV_FLAG_SKIP_BOOT_MODE
> v3: update per Jiewen's comments
> - remove HASHED_FV_MAX_NUMBER
> - update _EDKII_PEI_FIRMWARE_VOLUME_INFO_STORED_HASH_FV_PPI to
> be more flexible
> - update HASHED_FV_INFO to use UINT64 for Base an
>V4: change FV_HASH_FLAG_BOOT_MODE definition
>V3: update per Jiewen's comments
>V2: fix parameter description error found by ECC
https://bugzilla.tianocore.org/show_bug.cgi?id=1617
Cc: Chao Zhang
Cc: Jiewen Yao
Cc: "Hernandez Beltran, Jorge"
Cc: Harry Han
Jian J Wang (3):
SecurityPkg: a
>v3:
> - shrink mHashAlgInfo and add FindHashAlgInfo() to find matched algorithm
> - add ASSERT for illegal use cases
> - add DEBUG print for skipped FV
> - update code to use new definition of HASHED_FV_INFO and
>_EDKII_PEI_FIRMWARE_VOLUME_INFO_STORED_HASH_FV_PPI
> - some small code optim
The library DxeDeferImageLoadLib supports UID feature and it is conflicted
with the driver SecurityStubDxe (Defer3rdPartyImageLoad.c).
And the UID feature is dropped. So it should be removed from SecurityPkg.
https://bugzilla.tianocore.org/show_bug.cgi?id=1919
Cc: Chao Zhang
Cc: Jiewen Yao
Cc: J
Hi Dandan,
Please update the copyright date to 2019 before you check in the code, others
are good to me.
Reviewed-by: Eric Dong
Thanks,
Eric
> -Original Message-
> From: Bi, Dandan
> Sent: Monday, June 17, 2019 4:49 PM
> To: devel@edk2.groups.io
> Cc: Dong, Eric ; Gao, Liming
> Subje
Agree with Laszlo's comment. With it's addressed,
Reviewed-by: Jian J Wang
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Laszlo Ersek
> Sent: Thursday, June 20, 2019 6:20 AM
> To: devel@edk2.groups.io; Lu, XiaoyuX ; Rebecca Cran
>
> Cc:
> -Original Message-
> From: Ni, Ray
> Sent: Wednesday, June 19, 2019 5:00 PM
> To: devel@edk2.groups.io; Xu, Wei6; Wu, Hao A
> Cc: Wang, Jian J; Zhang, Chao B
> Subject: RE: [edk2-devel][Patch v2 3/7] MdeModulePkg: Add
> CapsuleOnDiskLoadPei PEIM.
>
> > -Original Message-
> > From
Reviewed-by: Jian J Wang
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Xiaoyu Lu
> Sent: Wednesday, June 19, 2019 3:19 PM
> To: devel@edk2.groups.io
> Cc: Lu, XiaoyuX ; Laszlo Ersek ;
> Wang, Jian J ; Ye, Ting
> Subject: [edk2-devel] [PAT
Reviewed-by: Jian J Wang
> -Original Message-
> From: Lu, XiaoyuX
> Sent: Wednesday, June 19, 2019 3:21 PM
> To: devel@edk2.groups.io
> Cc: Lu, XiaoyuX ; Wang, Jian J
> Subject: [PATCH 1/1] CryptoPkg/OpensslLib: Fix CR/LF issue
>
> Re-run process_files.py to generate OpensslLib[Crypto].
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1855
UniTool is one python script to generate UQI (Universal Question
Identifier) unicode string for HII question PROMPT string. UQI
string can be used to identify each HII question.
The scripts function will sync up UQI definitions with uni files
On 06/15/19 10:47, Philippe Mathieu-Daudé wrote:
> On 6/10/19 1:54 PM, Leif Lindholm wrote:
>> https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers
>> is a great resource, but it's a lot of manual steps to go through for
>> each repo
On 06/19/19 09:19, Xiaoyu Lu wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1881
>
> Commit(0a1b13fd4d2210e2c3) fix VS2017 build failure
> remove useless file in OpensslLib[Crypto].inf,
> but we use process_files.py to generate files.
> So exclude err_all.c file in process_files.py
>
Hi,
On 06/19/19 09:22, Xiaoyu Lu wrote:
> This problem was found by Rebecca Cran .
> REF: https://edk2.groups.io/g/devel/topic/32100684
>
> OpenSSL will automatically load a system config file which configures
> default ssl options. In UEFI, It will cause TlsInitialize failed
> without OPENSSL_IN
> the driver is thoroughly commented. See especially the
> DriverInitialize() function. Can you determine which one(s) of those
> statements doesn't / don't hold any longer?
>
> Or maybe IncompatiblePciDeviceSupportDxe works as before, but commit
> 065ae7d717f9 ("MdeModulePkg/PciBusDxe: make OPRO
Hi David,
On 06/19/19 14:50, David Woodhouse wrote:
> On Thu, 2016-05-19 at 00:12 +0200, Laszlo Ersek wrote:
>> According to edk2 commit
>>
>> "MdeModulePkg/PciBus: do not improperly degrade resource"
>>
>> and to the EFI_INCOMPATIBLE_PCI_DEVICE_SUPPORT_PROTOCOL definition in the
>> Platform Ini
Hi David,
On 06/17/19 12:52, David Woodhouse wrote:
> On Mon, 2014-05-12 at 20:21 +0400, Mike Maslenkin wrote:
>>>
> + Segment0 = 0;
> + Segment0Pages = 1;
> + Status = gBS->AllocatePages (AllocateAddress, EfiReservedMemoryType,
> + Segment0Pages, &Segment0
The last compiler flag change was for Xcode 5.0, not Xcode 3.2. As such
the HOST_TOOLS should be set to XCODE5.
Also, fix a small typo.
This fixes bug 447:
https://bugzilla.tianocore.org/show_bug.cgi?id=447
Signed-off-by: Stephano Cetola
---
EmulatorPkg/build.sh | 4 ++--
1 file changed, 2 in
On 2019-06-19 02:07, Lu, XiaoyuX wrote:
>
> Thanks for your information. I checked the NOOPT OVMF in linux environment,
> it failed too.
> I think compiler optimization hides this problem.
> By default, OpenSSL will auto load config file. But UEFI don't use it.
> And OpenSSL commit (25eb9299) fir
On 2019-06-19 01:22, Xiaoyu Lu wrote:
> This problem was found by Rebecca Cran .
> REF: https://edk2.groups.io/g/devel/topic/32100684
>
> OpenSSL will automatically load a system config file which configures
> default ssl options. In UEFI, It will cause TlsInitialize failed
> without OPENSSL_INIT_N
On Thu, 2016-05-19 at 00:12 +0200, Laszlo Ersek wrote:
> According to edk2 commit
>
> "MdeModulePkg/PciBus: do not improperly degrade resource"
>
> and to the EFI_INCOMPATIBLE_PCI_DEVICE_SUPPORT_PROTOCOL definition in the
> Platform Init 1.4a specification, a platform can provide such a protoco
Signed-off-by: David Woodhouse
---
OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
index 05e3ffd2bb..69abd06c40 100644
--- a/OvmfPkg/Csm/LegacyBiosDxe/Legacy
Signed-off-by: David Woodhouse
---
They still end up all just called 'Harddisk', but I absolutely do not
want to reproduce all the special cases in BmBootDescription.c. I'm not
even sure I want to export that and use it; it's horrid. Why don't the
disk objects themselves have a protocol which wil
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1781
EDK2 Support for SM3 digest algorithm is needed to enable TPM with SM3 PCR
banks. This digest algorithm is part of the China Crypto algorithm suite.
This integration has dependency on the openssl_1_1_1b integration into
edk2.
This patch add S
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1781
EDK2 Support for SM3 digest algorithm is needed to enable TPM with SM3 PCR
banks. This digest algorithm is part of the China Crypto algorithm suite.
This integration has dependency on the openssl_1_1_1b integration into
edk2.
This patch add S
Hello :
Right now the structure _ISCSI_ATTEMPT_CONFIG_NVDATA does not have the entry
for Initiator Name.
It uses the NV Variable I_NAME for maintain the Initiator Name since the setup
supports single initiator name for all the attempts getting created.
When we have the attempts from the Redfis
Can you please help to confirm the behavior
From: Sivaraman Nainar
Sent: Friday, June 7, 2019 2:48 PM
To: devel@edk2.groups.io
Subject: reg: Multiple Host Name Certificate
Hello:
Can someone help to confirm if EDK2 supports multiple Host Name support.
We need to have an environment where the HT
On Wed, 19 Jun 2019 at 12:09, Ooi, Tzy Way wrote:
>
> Hi Bieshuevel,
>
> In the DmaMap function, there is one line where my code look like this:
>
> "Status = DmaMap (MapOperationBusMasterCommonBuffer,
> Snp->MacDriver.TxdescRing[0], &BufferSize, &Snp->MacDriver.TxdescRingMap[0],
> &Snp->Mapping
Hi Bieshuevel,
In the DmaMap function, there is one line where my code look like this:
"Status = DmaMap (MapOperationBusMasterCommonBuffer,
Snp->MacDriver.TxdescRing[0], &BufferSize, &Snp->MacDriver.TxdescRingMap[0],
&Snp->MappingTxdesc);"
You asked me why the [0] and how many descriptor I am
On Wed, Jun 19, 2019 at 05:09:40AM +, Wu, Hao A wrote:
> Hello all,
>
> As suggested by Ray and Leif, modules (with wildcard) in MdeModulePkg are
> classified to a list of features.
>
> Please note that:
> * The below list is a draft at this moment, please help to provide
> feedbacks/commen
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Xu,
> Wei6
> Sent: Wednesday, June 19, 2019 4:41 PM
> To: Wu, Hao A ; devel@edk2.groups.io
> Cc: Wang, Jian J ; Zhang, Chao B
>
> Subject: Re: [edk2-devel][Patch v2 3/7] MdeModulePkg: Add
> CapsuleOnDiskLoadPei PEIM.
>
> > >
This change will be pushed to edk2-staging repo after it gets Reviewed-by.
The reason is the change requires a protocol interface change in PI spec.
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Ni, Ray
> Sent: Wednesday, June 19, 2019 4:57 PM
> To: devel@edk2.groups.io
>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ray Ni
Cc: Eric Dong
---
MdePkg/Include/Protocol/MpService.h | 50 +---
UefiCpuPkg/Library/MpInitLib/MpLib.c | 19 +++
2 files changed, 64 insertions(+), 5 deletions(-)
diff --git a/MdePkg/In
> > + ASSERT_EFI_ERROR (Status);
> > +
> > + FileNameSize = PcdGetSize (PcdCoDRelocationFileName); Status =
> > + PcdSetPtrS (PcdRecoveryFileName, &FileNameSize, (VOID *)
> > PcdGetPtr(PcdCoDRelocationFileName));
>
>
> Buffer for 'PcdRecoveryFileName' may not be big enough to hold the content
Thank you very much.
Of course, I will follow up all the other comments.
BR,
Wei
> -Original Message-
> From: Wu, Hao A
> Sent: Wednesday, June 19, 2019 4:20 PM
> To: devel@edk2.groups.io; Wu, Hao A ; Xu, Wei6
>
> Cc: Wang, Jian J ; Zhang, Chao B
>
> Subject: RE: [edk2-devel][Patch v2 7/
Forgot to mention, please help to handle other comments in the first reply
to the patch. Thanks in advance.
Best Regards,
Hao Wu
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Wu, Hao A
> Sent: Wednesday, June 19, 2019 4:17 PM
> To: Xu, Wei
>
> No, that's how it is designed.
> PcdCodRelocationDevPath is used by platform to specify a device to store
> temp Cod relocation file.
> If this PCD is not available, it means that platform doesn't have a
> requirement
> to store the file to a specific place.
> Then save the file to the devic
Hi bcran,
> -Original Message-
> From: Rebecca Cran [mailto:rebe...@bluestop.org]
> Sent: Wednesday, June 19, 2019 5:27 AM
> To: Lu, XiaoyuX ; devel@edk2.groups.io
> Cc: Wang, Jian J ; Laszlo Ersek
> Subject: Re: [edk2-devel] Problems using HTTPS boot: Http->Request can't
> find TLS Servi
Thanks a lot for the comments.
Please find my answer bellow.
> -Original Message-
> From: Wu, Hao A
> Sent: Wednesday, June 12, 2019 3:50 PM
> To: devel@edk2.groups.io; Xu, Wei6
> Cc: Wang, Jian J ; Zhang, Chao B
>
> Subject: RE: [edk2-devel][Patch v2 7/7] MdeModulePkg: Add Capsule On
This problem was found by Rebecca Cran .
REF: https://edk2.groups.io/g/devel/topic/32100684
OpenSSL will automatically load a system config file which configures
default ssl options. In UEFI, It will cause TlsInitialize failed
without OPENSSL_INIT_NO_LOAD_CONFIG flag. we don't use this feature,
So
Re-run process_files.py to generate OpensslLib[Crypto].inf.
CryptoPkg/Library/Include/openssl/opensslconf.h is coped from OpenSSL,
So keep the CR/LF style like OpenSSL source file.
Cc: Jian J Wang
Signed-off-by: Xiaoyu Lu
---
CryptoPkg/Library/Include/openssl/opensslconf.h | 80
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1881
Commit(0a1b13fd4d2210e2c3) fix VS2017 build failure
remove useless file in OpensslLib[Crypto].inf,
but we use process_files.py to generate files.
So exclude err_all.c file in process_files.py
Cc: Jian J Wang
Cc: Ting Ye
Signed-off-by: Xia
55 matches
Mail list logo
