Re: [edk2] [PATCH 10/10] ArmPkg/MmCommunicationDxe: signal architected PI events into MM context

2019-03-05 Thread Felix Polyudov
There is an architectural difference between EndOfDxe and SmmReadyToLock events. It's important to have both of them. Here is what PI specification says: == Transition from the environment where all of the components are under the authority of the platform manufacturer to the environment where thi

Re: [edk2] [Patch] Document: Update the INF spec about [Depex] section

2019-02-28 Thread Felix Polyudov
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Feng, > Bob C > Sent: Wednesday, February 27, 2019 9:12 PM > To: edk2-devel@lists.01.org > Cc: Liming Gao > Subject: [edk2] [Patch] Document: Update the INF spec about [Depex] section > > ::= {"

Re: [edk2] [Patch] BaseTools: Enable component override functionality

2019-02-22 Thread Felix Polyudov
Bob, Do you know if DSC specification has been updated to document this new behavior? If not, it should be updated. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of BobCF Sent: Thursday, January 10, 2019 9:39 PM To: edk2-devel@lists.01.org Cc: Cars

Re: [edk2] [Patch] BaseTools: Correct the value assignment for StructurePcd

2019-02-18 Thread Felix Polyudov
Bob, I think silent trimming of input data is not a good idea. The build tool should either report an error (my preference) or a warning that DSC or DEC input value is too long. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Feng, Bob C Sent: S

Re: [edk2] [edk2-announce] Community Meeting Minutes

2019-02-15 Thread Felix Polyudov
> -Original Message- > From: Laszlo Ersek > Sent: Friday, February 15, 2019 9:23 AM > > Just a short comment below. > > (Not changing my stance in any way that I've presented thus far; the > comment is only meant in addition to / as a clarification for that.) > > On 02/15/19 09:43, Ard B

Re: [edk2] History question about Base.h and its alternate parallel name space.... Should we change it?

2019-01-18 Thread Felix Polyudov
Mike, I think EFI_GUID and EFI_STATUS should cover most of the use cases. -Original Message- From: Kinney, Michael D [mailto:michael.d.kin...@intel.com] Sent: Thursday, January 17, 2019 3:04 PM To: Felix Polyudov; 'Andrew Fish'; Kinney, Michael D Cc: edk2-devel@lists.01.org S

Re: [edk2] History question about Base.h and its alternate parallel name space.... Should we change it?

2019-01-16 Thread Felix Polyudov
I agree with Andrew. In my opinion, moving alias types to Base.h will help to overcome certain practical inconveniences without a significant impact on the ability to detect poorly written Base libraries. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Beha

[edk2] [Patch v2] BaseTools: Fix incorrect formatting of GenFds command dictionary

2019-01-16 Thread Felix Polyudov
EFI_SOURCE references Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- BaseTools/Source/Python/AutoGen/AutoGen.py | 52 +- 1 file changed, 51 insertions(+), 1 deletion(-) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b

[edk2] [Patch] BaseTools: Fix incorrect formatting of GenFds command dictionary

2019-01-15 Thread Felix Polyudov
-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- BaseTools/Source/Python/AutoGen/AutoGen.py | 54 +- 1 file changed, 53 insertions(+), 1 deletion(-) diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen

Re: [edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table pointer on AP

2018-12-05 Thread Felix Polyudov
k it makes sense to apply the patch. Thanks Felix From: af...@apple.com [mailto:af...@apple.com] Sent: Friday, November 30, 2018 3:37 PM To: Felix Polyudov Cc: Ni, Ruiyu; edk2-devel@lists.01.org; ler...@redhat.com; Dong, Eric Subject: Re: [edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table p

Re: [edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table pointer on AP

2018-11-30 Thread Felix Polyudov
Sent: Thursday, November 29, 2018 10:43 PM To: Felix Polyudov; edk2-devel@lists.01.org Cc: ler...@redhat.com; Dong, Eric Subject: Re: [edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table pointer on AP Felix, I disagree:) Sorry about that. :) The commit you mentioned might be made by me (d

[edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table pointer on AP

2018-11-29 Thread Felix Polyudov
PeiServicesTablePointerLib or PeiServicesLib on AP caused CPU exception. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib

Re: [edk2] [Patch] BaseTools: Replace the sqlite database with list

2018-11-08 Thread Felix Polyudov
Bob, Does it mean that after this patch the build data is no longer saved to a file and is recreated on every build? Do you have any data regarding build process performance improvements after applying the patch? Does this patch improve full build time and incremental build time? Thanks Felix

[edk2] [Patch v3] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file

2018-03-02 Thread Felix Polyudov
. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- MdePkg/Include/IndustryStandard/PciExpress40.h | 89 ++ 1 file changed, 89 insertions(+) create mode 100644 MdePkg/Include/IndustryStandard/PciExpress40.h diff --git a/MdePkg

Re: [edk2] [Patch v2] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file

2018-02-28 Thread Felix Polyudov
devel@lists.01.org Cc: Felix Polyudov; michael.d.kin...@intel.com; Manickavasakam Karpagavinayagam; liming@intel.com Subject: RE: [edk2] [Patch v2] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file Good day, Please consider for compatibility with some toolchains, byte-packed struct

[edk2] [Patch v2] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file

2018-02-27 Thread Felix Polyudov
1.1 Signed-off-by: Felix Polyudov --- MdePkg/Include/IndustryStandard/PciExpress40.h | 89 ++ 1 file changed, 89 insertions(+) create mode 100644 MdePkg/Include/IndustryStandard/PciExpress40.h diff --git a/MdePkg/Include/IndustryStandard/PciExpress40.h b/MdePkg/Include

[edk2] [PATCH] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file

2018-02-26 Thread Felix Polyudov
The header includes Physical Layer PCI Express Extended Capability definitions described in section 7.7.5 of PCI Express Base Specification rev. 4.0. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- MdePkg/Include/IndustryStandard/PciExpress40.h | 69

[edk2] [PATCH] MdePkg/Include/IndustryStandard: Add PCI Express 4.0 header file

2018-02-15 Thread Felix Polyudov
The header includes Physical Layer PCI Express Extended Capability definitions described in section 7.7.5 of PCI Express Base Specification rev. 4.0. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Felix Polyudov --- MdePkg/Include/IndustryStandard/PciExpress40.h | 69