BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
With this change, global variable _fltused will not be removed by LTO
Signed-off-by: Liming Gao
Reviewed-by: Jian J Wang
Reviewed-by: Philippe Mathieu-Daude
---
CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c | 10 --
1 file cha
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
Signed-off-by: Liming Gao
Reviewed-by: Hao A Wu
---
MdeModulePkg/Universal/RegularExpressionDxe/RegularExpressionDxe.inf | 3 +++
1 file changed, 3 insertions(+)
diff --git
a/MdeModulePkg/Universal/RegularExpressionDxe/RegularExpressionD
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
1. Apply CLANG9 Linker option.
2. Exclude -mno-mmx -mno-sse compiler option for CLANG9
These two options will cause CLANG Linker crush.
Signed-off-by: Liming Gao
Reviewed-by: Laszlo Ersek
---
OvmfPkg/OvmfPkgIa32.dsc| 4 +++-
OvmfPkg/Ov
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
1. Add WIN_SEC_BUILD macro check for CLANG9 tool chain
build -p EmulatorPkg\EmulatorPkg.dsc -a IA32 -DWIN_SEC_BUILD=TRUE -t CLANG9
build -p EmulatorPkg\EmulatorPkg.dsc -a X64 -DWIN_SEC_BUILD=TRUE -t CLANG9
2. Append CLANG CC and LINK flags to
In v3, add the detail commit message for patch 3.
Update the fix in EmulatorPkg based on the comments.
In v2, drop patch 12, and update commit message for 5 & 11.
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
Code: https://github.com/lgao4/edk2/tree/CLANG9
Wiki: https://github.com/lgao
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
Disable warning reported from CLANG9.
Signed-off-by: Liming Gao
Reviewed-by: Jian J Wang
---
CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf| 1 +
CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf | 1 +
CryptoPkg/Library/BaseCryptLib
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
CLANG9 tool chain defines __clang__ macro only,
doesn't define __GNUC__ macro. But, it uses some same definitions with GCC.
So, update base definition for CLANG9 tool chain.
Signed-off-by: Liming Gao
Cc: Michael Kinney
Reviewed-by: Philippe
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
Define the same macro in the different OS. It can make CLANG generate the same
image in the different host OS.
Signed-off-by: Liming Gao
Reviewed-by: Hao A Wu
---
MdeModulePkg/Library/LzmaCustomDecompressLib/Sdk/C/7zTypes.h | 2 +-
1 file
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
Signed-off-by: Liming Gao
Cc: Bob Feng
---
BaseTools/Conf/build_rule.template | 26 ++
BaseTools/Conf/tools_def.template | 102 +
2 files changed, 118 insertions(+), 10 deletions(-)
diff --git
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
__inline__ has no functional difference effect with the GCC48 / GCC49 / GCC5
toolchains, but it breaks the build with CLANG9. Remove __inline__.
Signed-off-by: Liming Gao
Acked-by: Laszlo Ersek
Cc: Michael Kinney
Reviewed-by: Philippe Math
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
$(DEST_DIR_DEBUG) path is in Include directory.
It is not required to be specified again.
Signed-off-by: Liming Gao
Reviewed-by: Bob Feng
---
BaseTools/Conf/tools_def.template | 22 +++---
1 file changed, 11 insertions(+),
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1603
CLANG9 generated PE image exposes below two issues.
1. SectionSize is used to copy PE section data. It should be smaller than
section raw size.
2. The real data is required to be copied. So, copy the min size of
VirtualSize and SizeOfRawData.
Reviewed-by: Nate DeSimone
-Original Message-
From: Agyeman, Prince
Sent: Tuesday, October 15, 2019 3:51 PM
To: devel@edk2.groups.io
Cc: Sinha, Ankit ; Desimone, Nathaniel L
; Kubacki, Michael A
Subject: [edk2-platforms] [PATCH v2 1/4] BoardModulePkg: Add BIOS Info HOB
Added gBiosIn
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Agyeman, Prince
Sent: Tuesday, October 15, 2019 3:51 PM
To: devel@edk2.groups.io
Cc: Sinha, Ankit ; Desimone, Nathaniel L
; Kubacki, Michael A
Subject: [edk2-devel] [edk2-platforms] [PATCH v2 4/4] Pl
DeviceIndex is used as index in Slot[]. The max size of Slot[]
is SD_PEIM_MAX_SLOTS. So DeviceIndex should be checked before used.
Cc: Hao A Wu
Cc: Ray Ni
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Bus/Sd/SdBlockIoPei/SdBlockIoPei.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
The size of mHiiEfiColors is 16.
mHiiEfiColors[Private->Attribute >> 4] may be out of boundary.
So add a check for that.
Cc: Dandan Bi
Cc: Eric Dong
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/HiiDatabaseDxe/Font.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff
> >
> >1. Why move the assignment to a later place? Can you explain a bit more?
> The hang issue here.
>
> I investigate this issue more. The root cause is that GetProcessAffinityMask()
> API requires the 64bit pointer as the input for 64bit ARCH. So, the correct
> fix
> should declare ProcessAff
Reviewed-by: Ray Ni
> -Original Message-
> From: Wang, Iwen Evelyn
> Sent: Thursday, October 17, 2019 11:21 AM
> To: devel@edk2.groups.io
> Cc: Huang, Jenny ; Shih, More
> ; Ni, Ray ; Chaganty, Rangasai V
> ; Yao, Jiewen
> Subject: [v7 v7] IntelSiliconPkg-Vtd: A new PMR interface
>
> R
Push
@b8fc5c03e5b5551b083f7efe91a051faa00c1e27..e797a806a8d090a09a782fe10ba2765154579d4b
Please close BZ 2202.
Thanks
Liming
>-Original Message-
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Gao, Zhichao
>Sent: Wednesday, October 16, 2019 8:45 AM
>To: Abner Chang
Ray:
>-Original Message-
>From: Ni, Ray
>Sent: Wednesday, October 16, 2019 5:32 PM
>To: devel@edk2.groups.io; Gao, Liming
>Cc: 'Andrew Fish (af...@apple.com)'
>Subject: RE: [edk2-devel] [Patch v2 10/11] EmulatorPkg: Enable CLANG9 tool
>chain
>
>Liming,
>You missed the Cc tag so Andrew a
Entry and RetEntry might be NULL before used.
Cc: Jian J Wang
Cc: Hao A Wu
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdSymbol.c | 2 +-
MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c| 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff
EsrtRepository might be NULL. So return EFI_OUT_OF_RESOURCES
when it is NULL.
Cc: Hao A Wu
Cc: Liming Gao
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/EsrtDxe/EsrtImpl.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/MdeModulePkg/Universal/EsrtDxe/EsrtImpl.c
b/Mde
The returned value from GetBufferForValue might be NULL, so add a
check for that before it is used.
Cc: Jian J Wang
Cc: Hao A Wu
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/SetupBrowserDxe/Expression.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/MdeModulePkg/Universal
If the target string doesn't appear in the searched string,
StringPtr will be NULL. So add a check for that.
Cc: Dandan Bi
Cc: Eric Dong
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c | 4
1 file changed, 4 insertions(+)
diff --git a/MdeModulePkg/
The variables and return value might be NULL.
So add check for them before they are used.
Cc: Jian J Wang
Cc: Hao A Wu
Cc: Dandan Bi
Cc: Eric Dong
Cc: Hao A Wu
Cc: Liming Gao
Shenglei Zhang (4):
MdeModulePkg/EbcDebugger: Add check for Entry and RetEntry
MdeModulePkg/HiiDatabaseDxe: Add c
IndexTable->Length is used as index in array IndexTable->Index[].
So IndexTable->Length needs to be checked, which should be less than
the array size.
Cc: Hao A Wu
Cc: Liming Gao
Signed-off-by: Shenglei Zhang
---
MdeModulePkg/Universal/Variable/Pei/Variable.c | 2 +-
1 file changed, 1 insertio
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Abner Chang
> Sent: Tuesday, October 08, 2019 4:04 PM
> To: devel@edk2.groups.io
> Cc: abner.ch...@hpe.com; Wu, Hao A; Wang, Jian J; Leif Lindholm; Gilbert
> Chen
> Subject: [edk2-devel] [PATCH v5
> -Original Message-
> From: Wu, Hao A
> Sent: Wednesday, October 16, 2019 10:14 AM
> To: Ashish Singhal; devel@edk2.groups.io; Ni, Ray; jbra...@nvidia.com
> Subject: RE: [PATCH v4 2/2] MdeModulePkg/XhciPei: Fix Aligned Page
> Allocation
>
> > -Original Message-
> > From: Ashish Si
Evelyn,
GetGlobalVtdPmrAlignment() is the only API listed in the library header file
but I saw all
the three functions in the library C file have the EFIAPI prefix.
Only the public library API needs the EFIAPI prefix.
Thanks,
Ray
> -Original Message-
> From: Wang, Iwen Evelyn
> Sent: Th
Reviewed-by: Ray Ni
> -Original Message-
> From: Wang, Iwen Evelyn
> Sent: Thursday, October 17, 2019 7:02 AM
> To: devel@edk2.groups.io
> Cc: Huang, Jenny ; Shih, More
> ; Ni, Ray ; Chaganty, Rangasai V
> ; Yao, Jiewen
> Subject: [PATCH] IntelSiliconPkg-Vtd: Code Optimization
>
> REF:
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Leif Lindholm
> Sent: Friday, October 4, 2019 12:32 AM
> To: devel@edk2.groups.io; Chen, Gilbert
> Cc: Palmer Dabbelt
> Subject: Re: [edk2-devel] [plaforms/devel-riscv-v2 PATCHv2 10/14]
> U50
OK. I will update the commit message to describe this issue.
This change is to make sure section size is in section raw data scope.
>-Original Message-
>From: Feng, Bob C
>Sent: Wednesday, October 16, 2019 5:27 PM
>To: devel@edk2.groups.io; Gao, Liming
>Subject: RE: [edk2-devel] [Patch
Yes. I will add it on next version patch.
Thanks
Liming
>-Original Message-
>From: Feng, Bob C
>Sent: Wednesday, October 16, 2019 5:19 PM
>To: Gao, Liming ; devel@edk2.groups.io
>Cc: Yao, Jiewen ; Shi, Steven
>; Justen, Jordan L ; Laszlo
>Ersek ; Andrew Fish ; Ni, Ray
>; Ard Biesheuvel ;
Hi Phillippe,
Yeah, I forgot to fill my example. Thanks for the catch.
It is exactly as Bob mentioned, #include MACRO(xxx.h).
There are silicon code use macro to separate definitions, and use the approach
to reuse code. Like:
#define PATH(x) // in platform 1
#define PATH(x) // in platform 2
#
Just expecting consistency for the rule, I found that for function
VariableServiceGetNextVariableName(), there are 4 occurrences and the local
variable is introduced.
But for VariableExLibFindNextVariable(), there are 5 occurrences but no local
variable is introduced.
Best Regards,
Hao Wu
>
> The "Data" parameter is marked OPTIONAL in FindVariableInRuntimeCache () for
> essentially the same reason
> it is labeled OPTIONAL in the GetVariable () API in the UEFI specification.
> Callers
> expect that they can pass a
> NULL as an actual parameter for Data and get back the size of a buffe
Your threshold is slightly lower than mine for adding the local variable. My
test plan is already
done on this patch series. Do you consider this mandatory?
Thanks,
Michael
> -Original Message-
> From: Wu, Hao A
> Sent: Wednesday, October 16, 2019 6:01 PM
> To: Kubacki, Michael A ;
> de
The "Data" parameter is marked OPTIONAL in FindVariableInRuntimeCache () for
essentially the same reason
it is labeled OPTIONAL in the GetVariable () API in the UEFI specification.
Callers expect that they can pass a
NULL as an actual parameter for Data and get back the size of a buffer needed
f
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Laszlo Ersek
> Sent: Wednesday, October 16, 2019 9:32 PM
> To: Philippe Mathieu-Daude; devel@edk2.groups.io
> Cc: Supreeth Venkatesh; Dong, Eric; Gao, Liming; Achin Gupta; Bi, Dandan;
> Wu, Jiaxin
> -Original Message-
> From: Kubacki, Michael A
> Sent: Thursday, October 17, 2019 12:37 AM
> To: Wu, Hao A; devel@edk2.groups.io
> Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney,
> Michael D; Ni, Ray; Wang, Jian J; Yao, Jiewen
> Subject: RE: [edk2-devel] [PAT
> -Original Message-
> From: Kubacki, Michael A
> Sent: Tuesday, October 15, 2019 7:30 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney,
> Michael D; Ni, Ray; Wang, Jian J; Wu, Hao A; Yao, Jiewen
> Subject: [PATCH V4 04/10] MdeMod
I think one of the example would be like this:
#include MACRO(xxx.h)
I think this patch is good. Thanks for caching and fixing this bug.
Reviewed-by: Bob Feng
Thanks,
Bob
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Philippe
Mathieu-Daud
reviewed-by: jiewen Yao
thank you!
Yao, Jiewen
> 在 2019年10月17日,上午7:47,Gao, Liming 写道:
>
> commit a7e2d20193e853020a1415c25b53280955055394 introduces the code to
> get PcdTpm2AcpiTableRev in the driver entry point. This PCD is designed as
> DynamicHii or DynamicHiiEx PCD. So, this PCD depends
commit a7e2d20193e853020a1415c25b53280955055394 introduces the code to
get PcdTpm2AcpiTableRev in the driver entry point. This PCD is designed as
DynamicHii or DynamicHiiEx PCD. So, this PCD depends on Variable service.
To make sure PcdTpm2AcpiTableRev value be got, add Variable service as Depex.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1770
1) IOMMU PMR feature should be generic to support different hardware
architecture. Platforms may request no overlap between PMR regions
and system reserve memory. Create an interface to control PLMR/PHMR
regions. It allows silicon code to ad
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1770
1) DisableDMAr Function Code Optimization
Optimize the flow to follow the VT-d spec requirements.
2) Renamed InitDmar() to InitGlobalVtd()
The oringal function name is misleading
Signed-off-by: Evelyn Wang
Cc: Jenny Huang
Cc: More Shih
On Wed, Oct 16, 2019 at 06:50:29PM +0200, Philippe Mathieu-Daudé wrote:
> On 10/16/19 5:18 PM, Leif Lindholm wrote:
> > For this reason *I* would not use said script, but I'm not going to
> > say we shouldn't have it. I will say it shouldn't be enabled by
> > default by SetupGit.py though.
>
> Wel
Thanks Ray, this was a good discussion. I will leave the library instances
as-is for now.
> -Original Message-
> From: Ni, Ray
> Sent: Tuesday, October 15, 2019 11:30 PM
> To: Kubacki, Michael A ;
> devel@edk2.groups.io
> Cc: Chaganty, Rangasai V
> Subject: RE: [edk2-platforms][PATCH V1
Platform\Intel\KabylakeOpenBoardPkg\GalagoPro3\OpenBoardPkg.dsc
* Please move $(PLATFORM_BOARD_PACKAGE)/BiosInfo/BiosInfo.inf under
the "Board Package" section.
Platform\Intel\KabylakeOpenBoardPkg\KabylakeRvp3\OpenBoardPkg.dsc:
* Please move $(PLATFORM_BOARD_PACKAGE)/BiosInfo/BiosInfo.inf under
t
Reviewed-by: Michael Kubacki
> -Original Message-
> From: Agyeman, Prince
> Sent: Tuesday, October 15, 2019 3:51 PM
> To: devel@edk2.groups.io
> Cc: Sinha, Ankit ; Desimone, Nathaniel L
> ; Kubacki, Michael A
>
> Subject: [edk2-platforms] [PATCH v2 4/4] Platform/Intel: Add FIT generatio
Platform\Intel\WhiskeylakeOpenBoardPkg\WhiskeylakeURvp\OpenBoardPkg.dsc:
* Please move $(PLATFORM_BOARD_PACKAGE)/BiosInfo/BiosInfo.inf under
the "Board Package" section.
Thanks,
Michael
> -Original Message-
> From: Agyeman, Prince
> Sent: Tuesday, October 15, 2019 3:51 PM
> To: devel@edk
Reviewed-by: Michael Kubacki
> -Original Message-
> From: Agyeman, Prince
> Sent: Tuesday, October 15, 2019 3:51 PM
> To: devel@edk2.groups.io
> Cc: Sinha, Ankit ; Desimone, Nathaniel L
> ; Kubacki, Michael A
>
> Subject: [edk2-platforms] [PATCH v2 1/4] BoardModulePkg: Add BIOS Info
> H
On 10/16/19 5:18 PM, Leif Lindholm wrote:
Hi Phil,
This is definitely a manual step that could do with automation, so
many thanks for having a go. ...But, I have a few reservations
regarding the actual solution.
First of all, doing it as a .sh means it will not work for those
developers working
AtRuntime () is implemented in VariableSmmRuntimeDxe.c because it is called by
VariableParsing.c
which is linked to both VariableSmmRuntimeDxe and VariableSmm. VariableSmm
cannot directly
invoke EfiAtRuntime (), so prior to this change, VariableSmm stores the runtime
status in the global
variabl
The InitVariableParsing () function has already been removed entirely in V3+ to
prevent issues like this. I strongly believe this is the right direction. It is
only a
matter of time before someone else modifies the global and forgets to call
InitVariableParsing ().
Furthermore, VariablePei has al
On 10/14/19 20:36, Laszlo Ersek wrote:
> On 10/14/19 17:03, Pete Batard wrote:
>> Changes from previous version:
>> - Additional empty comment lines above and below, to keep with existing
>> comment
>> style
>> - Similar logic is applied to ArmVirtPkg/PlatformBootManagerLib, since the
>> same
>
On Wed, 2019-10-16 at 16:43 +0200, Laszlo Ersek wrote:
> Regarding the current edk2 patch set, I think we should do the following:
>
> - use X509_VERIFY_PARAM_set1_ip() rather than
> X509_VERIFY_PARAM_set1_ip_asc()
>
> - incorporate "StdLib/BsdSocketLib/inet_pton.c" from the edk2-libc
> project (
Hi Phil,
This is definitely a manual step that could do with automation, so
many thanks for having a go. ...But, I have a few reservations
regarding the actual solution.
First of all, doing it as a .sh means it will not work for those
developers working primarily with Visual Studio. Rewriting it
Hi Phil,
On 10/16/19 16:09, Philippe Mathieu-Daude wrote:
> Hi,
>
> This series contains trivial fixes to the GetMaintainer.py script,
> then adds a new script which allow git-send-email to automatically
> fills the email addresses relevant to each patch, using the new
> Maintainer.txt format (wi
On 10/16/19 15:35, David Woodhouse wrote:
> On Wed, 2019-10-16 at 13:41 +0200, Laszlo Ersek wrote:
>> Anyway: we still have the issue that X509_VERIFY_PARAM_set_ip_asc()
>> appears to reject IPv4 address literals. Could you check that please?
>>
>> (Using a hosted (Linux userspace) program like "s
On 10/16/19 4:09 PM, Philippe Mathieu-Daude wrote:
Add a new script GitCcCmd.sh that git-send-email can use to
automatically determine which addresses to cc on patch submission.
Signed-off-by: Philippe Mathieu-Daude
---
BaseTools/Scripts/GitCcCmd.sh | 38 +++
Add a new script GitCcCmd.sh that git-send-email can use to
automatically determine which addresses to cc on patch submission.
Signed-off-by: Philippe Mathieu-Daude
---
BaseTools/Scripts/GitCcCmd.sh | 38 +++
1 file changed, 38 insertions(+)
create mode 100755 Ba
The GetMaintainer.py script use the "if __name__ == '__main__'"
evaluation so it expects to be run as a standalone program.
We have other Python scripts with the executable permission, let
this one be executable too.
Signed-off-by: Philippe Mathieu-Daude
---
BaseTools/Scripts/GetMaintainer.py |
To have GetMaintainer.py script output easily parsable by
scripts, add the '--quiet' option. Output will be formatted
as one email per line.
Signed-off-by: Philippe Mathieu-Daude
---
BaseTools/Scripts/GetMaintainer.py | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/B
Hi,
This series contains trivial fixes to the GetMaintainer.py script,
then adds a new script which allow git-send-email to automatically
fills the email addresses relevant to each patch, using the new
Maintainer.txt format (with the GetMaintainer.py script).
I sent this series adding:
[sendemai
Reviewed-by: jiewen@intel.com
> -Original Message-
> From: Philippe Mathieu-Daude
> Sent: Wednesday, October 16, 2019 9:26 PM
> To: devel@edk2.groups.io
> Cc: Supreeth Venkatesh ; Dong, Eric
> ; Gao, Liming ; Achin Gupta
> ; Bi, Dandan ; Wu, Jiaxin
> ; Wu, Hao A ; Ni, Ray
> ; Fu, Siyu
On Wed, 2019-10-16 at 13:41 +0200, Laszlo Ersek wrote:
> On 10/16/19 11:19, David Woodhouse wrote:
> > In the end you did actually solve it all for yourself -- based on the
> > pointer I'd given in bugzilla, and then ignoring my subsequent
> > misdirection about callbacks and my overly complex atte
On 2019.10.16 14:24, Leif Lindholm wrote:
On Wed, Oct 16, 2019 at 01:56:15PM +0200, Philippe Mathieu-Daudé wrote:
On 10/16/19 1:43 PM, Pete Batard wrote:
DEBUG_WARNING was invalid. DEBUG_WARN is the proper channel name to use.
Oops.
+1
Yeah, my bad here.
I was under the impression that
On 10/16/19 15:25, Philippe Mathieu-Daude wrote:
> Some "processing/processed" words are written with an extra 's'.
> Properly write them.
>
> v2: Rebased
>
> Philippe Mathieu-Daudé (4):
> MdeModulePkg DxeCore: Fix typo in function name
> MdeModulePkg/PiSmmCore: Fix typo in function name
>
An extra 's' slipped into the FvIsBeingProcessed function
name. Drop it to fix the typo.
Reviewed-by: Laszlo Ersek
Reviewed-by: Hao A Wu
Reviewed-by: Liming Gao
Signed-off-by: Philippe Mathieu-Daude
---
MdeModulePkg/Core/PiSmmCore/Dispatcher.c | 4 ++--
1 file changed, 2 insertions(+), 2 dele
An extra 's' slipped into the 'processing' word.
Drop it to fix the typo.
Reviewed-by: Laszlo Ersek
Reviewed-by: Siyuan Fu
Signed-off-by: Philippe Mathieu-Daude
---
NetworkPkg/Ip6Dxe/Ip6Option.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/NetworkPkg/Ip6Dxe/Ip6Option.c b
An extra 's' slipped into the FvIsBeingProcessed function
name. Drop it to fix the typo.
Reviewed-by: Laszlo Ersek
Signed-off-by: Philippe Mathieu-Daude
---
StandaloneMmPkg/Core/Dispatcher.c | 4 ++--
StandaloneMmPkg/Core/FwVol.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
An extra 's' slipped into the FvIsBeingProcessed function
name. Drop it to fix the typo.
Reviewed-by: Laszlo Ersek
Reviewed-by: Hao A Wu
Reviewed-by: Liming Gao
Signed-off-by: Philippe Mathieu-Daude
---
MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c | 4 ++--
1 file changed, 2 insertions(+), 2
Some "processing/processed" words are written with an extra 's'.
Properly write them.
v2: Rebased
Philippe Mathieu-Daudé (4):
MdeModulePkg DxeCore: Fix typo in function name
MdeModulePkg/PiSmmCore: Fix typo in function name
StandaloneMmPkg/Core: Fix typo in function name
NetworkPkg/Ip6Dxe
On Wed, Oct 16, 2019 at 01:56:15PM +0200, Philippe Mathieu-Daudé wrote:
> On 10/16/19 1:43 PM, Pete Batard wrote:
> > DEBUG_WARNING was invalid. DEBUG_WARN is the proper channel name to use.
>
> Oops.
+1
> Reviewed-by: Philippe Mathieu-Daude
And
Reviewed-by: Leif Lindholm
Pushed as bd26e413
On 10/16/19 1:43 PM, Pete Batard wrote:
DEBUG_WARNING was invalid. DEBUG_WARN is the proper channel name to use.
Oops.
Reviewed-by: Philippe Mathieu-Daude
Signed-off-by: Pete Batard
---
Platform/RaspberryPi/RPi3/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c | 2 +-
1 file changed, 1 in
Hi Derek,
On 10/16/19 8:17 AM, Lin, Derek (HPS SW) wrote:
When c/h file use macro after #include, for example,
Apparently you forgot to write your example.
In this case, GenMake is not able to create a healthy dependency for the c
file. GenMake used to add $(FORCE_REBUILD) dependency in the
On 10/15/19 1:07 PM, Ard Biesheuvel wrote:
DynamicTablesPkg can be built for ARM as well as for AARCH64, but on
the former, doing so will result in a build failure due to the lack
of 64-bit division helpers provided by the ArmPkg intrinsics library.
So add the missing reference, for both ARM and
DEBUG_WARNING was invalid. DEBUG_WARN is the proper channel name to use.
Signed-off-by: Pete Batard
---
Platform/RaspberryPi/RPi3/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/Platform/RaspberryPi/RPi3/Drivers/PlatformSmbiosDxe
On 10/16/19 11:19, David Woodhouse wrote:
> On Wed, 2019-10-16 at 09:36 +0200, Laszlo Ersek wrote:
>> On 10/16/19 07:18, Wu, Jiaxin wrote:
>>>In some cases, the URI is specified as an IP address rather than
>>>a Hostname . In this case, the iPAddress subjectAltName must be
>>>present i
On Wed, 2019-10-16 at 01:08 +0200, Laszlo Ersek wrote:
> Notes:
> Unfortunately, there are two problems with this patch:
>
> (1) X509_VERIFY_PARAM_set1_ip_asc() does not accept IPv4 addresses in
> dot-decimal notation (unless I messed up the code). My log file
> contain
On 10/16/19 11:40, David Woodhouse wrote:
> On Tue, 2019-10-15 at 19:34 +0200, Laszlo Ersek wrote:
>> Ehh, I failed to ask the actual question.
>>
>> Is it OK to call X509_VERIFY_PARAM_set1*() multiple times -- basically,
>> every time just before we call X509_verify_cert()?
>>
>> My concern is not
Thanks for catching this, Laszlo.
Thanks for the suggestion and information, Zhichao. Next time, I will
definitely take care of this in the first place.
Hi Jian and Hao,
I forgot to add both of you in Cc. Sorry about that. I will send you guys a new
patch with the updated commit message below
On Tue, 2019-10-15 at 19:34 +0200, Laszlo Ersek wrote:
> Ehh, I failed to ask the actual question.
>
> Is it OK to call X509_VERIFY_PARAM_set1*() multiple times -- basically,
> every time just before we call X509_verify_cert()?
>
> My concern is not with the crypto functionality, but whether we c
Liming,
You missed the Cc tag so Andrew and I were not CCed.
3 comments in below.
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Liming
> Gao
> Sent: Tuesday, October 15, 2019 8:27 AM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [Patch v2 10/11] EmulatorPkg: Enable
Hi Liming,
Would you please add more description in commit message? This patch is in the
patch set of CLANG9 enabling, but it's hard for me to see the direct
relationship between CLANG9 and this fix and why the original code is wrong.
Thanks
Bob
-Original Message-
From: devel@edk2.grou
On Wed, 2019-10-16 at 09:36 +0200, Laszlo Ersek wrote:
> On 10/16/19 07:18, Wu, Jiaxin wrote:
> >In some cases, the URI is specified as an IP address rather than a
> >Hostname . In this case, the iPAddress subjectAltName must be
> >present in the certificate and must exactly match the
Hi Liming,
Would you please add the BZ link to each of the patches? So that we can see it
in git commit log.
Thanks,
Bob
-Original Message-
From: Gao, Liming
Sent: Tuesday, October 15, 2019 8:27 AM
To: devel@edk2.groups.io
Cc: Yao, Jiewen ; Shi, Steven ;
Justen, Jordan L ; Laszlo Erse
Reviewed-by: Bob Feng < bob.c.f...@intel.com >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#49080): https://edk2.groups.io/g/devel/message/49080
Mute This Topic: https://groups.io/mt/34540582/21656
Group Owner: devel+ow...@edk2.groups.
On Wed, 2019-10-16 at 05:18 +, Wu, Jiaxin wrote:
> Fortunately, I get my wanted answer in RFC6125, SECTION 3.1 :
>
>If a subjectAltName extension of type dNSName is present, that MUST
>be used as the identity. Otherwise, the (most specific) Common Name
>field in the Subject field
Sunny,
For the OS recovery question, I had the code change in
https://github.com/niruiyu/edk2/tree/bds/osrecovery.
Due to there is no OS support for OS recovery yet, the code was not pushed.
Do you see any needs of the OS recovery?
Thanks,
Ray
> -Original Message-
> From: Wang, Sunny (
MdeModulePkg/BdsDxe: Do PlatformRecovery regardless of OsIndications
According to the UEFI specification section 3.4.1 and 3.4.2,
the OsRecovery and PlatformRecovery options should still be
processed regardless of the value of OsIndications variable
if processing of BootOrder does not result in su
pushed at 1e509eb07d5fa4ce93653cd4410b04cf6c815ea5
-Original Message-
From: devel@edk2.groups.io On Behalf Of Eric Jin
Sent: Saturday, October 12, 2019 12:57 PM
To: devel@edk2.groups.io; supreeth.venkat...@arm.com
Subject: Re: [edk2-devel] [edk2-test][PATCH v1 1/1] SctPkg: Convert files f
On 10/16/19 09:36, Laszlo Ersek wrote:
> On 10/16/19 07:18, Wu, Jiaxin wrote:
>> Comment2: do we really need the app_verify_callback function setting?
>> Why not call X509_VERIFY_PARAM_set1_ip_asc (TlsConn->Ssl->param,
>> HostName) in TlsSetVerifyHost directly? anything I missed in the
>> discussi
On 10/16/19 09:40, Wang, Sunny (HPS SW) wrote:
> According to the UEFI specification section 3.4.1 and 3.4.2, the OsRecovery
> and PlatformRecovery options should still be processed regardless of the
> value of OsIndications variable if processing of BootOrder does not result in
> success. There
> -Original Message-
> From: Kubacki, Michael A
> Sent: Tuesday, October 15, 2019 7:30 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney,
> Michael D; Ni, Ray; Wang, Jian J; Wu, Hao A; Yao, Jiewen
> Subject: [PATCH V4 10/10] MdeMod
> -Original Message-
> From: Kubacki, Michael A
> Sent: Tuesday, October 15, 2019 7:30 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney,
> Michael D; Ni, Ray; Wang, Jian J; Wu, Hao A; Yao, Jiewen
> Subject: [PATCH V4 08/10] MdeMod
> -Original Message-
> From: Kubacki, Michael A
> Sent: Tuesday, October 15, 2019 7:30 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan; Ard Biesheuvel; Dong, Eric; Laszlo Ersek; Gao, Liming; Kinney,
> Michael D; Ni, Ray; Wang, Jian J; Wu, Hao A; Yao, Jiewen
> Subject: [PATCH V4 06/10] MdeMod
Again, I would like to ask for help from other reviewers to look at this patch
(patch 7/10) and the next one (patch 8/10) (at least from the security
perspective). Any help will be appreciated, thanks in advance.
One comment inherited from the feedback on the V2 series:
I saw AtRuntime() is still
On Wed, 2019-10-16 at 09:54 +0200, Laszlo Ersek wrote:
> Anyway: once David's PR#9201 is merged into OpenSSL, the example code in
> the wiki page that calls SSL_set1_host() will become right.
Crypto libraries should make it hard for users to get it wrong.
Film at 11.
-=-=-=-=-=-=-=-=-=-=-=-
G
1 - 100 of 109 matches
Mail list logo