Re: [edk2-devel] [PATCH v3 12/28] AmpereAltraPkg: Add Ac01PcieLib library instance

2021-10-07 Thread Leif Lindholm
On Wed, Oct 06, 2021 at 19:55:53 +0700, Nhi Pham wrote: > > Would it be possible for PHYLib to link in ArmArchTimerLib directly > > and wrap this there instead? Something about this integration just > > feels kind of backwards to me. > > Thanks Leif. That's great idea. > > I will wrap it and re-s

Re: [edk2-devel] [PATCH v3 12/28] AmpereAltraPkg: Add Ac01PcieLib library instance

2021-10-07 Thread Nhi Pham via groups.io
On 07/10/2021 16:06, Leif Lindholm wrote: On Wed, Oct 06, 2021 at 19:55:53 +0700, Nhi Pham wrote: Would it be possible for PHYLib to link in ArmArchTimerLib directly and wrap this there instead? Something about this integration just feels kind of backwards to me. Thanks Leif. That's great idea.

Re: [edk2-devel] [RFC] [PATCH 0/2] Proposal to add EFI_MP_SERVICES_PROTOCOL support for AARCH64

2021-10-07 Thread Leif Lindholm
Ard/Sami - any comments? / Leif On Tue, Sep 28, 2021 at 12:14:35 +0100, Leif Lindholm wrote: > On Fri, Sep 24, 2021 at 20:17:50 -0600, Rebecca Cran wrote: > > I'd like to propose adding EFI_MP_SERVICES_PROTOCOL support for > > AARCH64 systems. I've attached two patches to implement support fo

Re: [edk2-devel] [RFC] [PATCH 0/2] Proposal to add EFI_MP_SERVICES_PROTOCOL support for AARCH64

2021-10-07 Thread Ard Biesheuvel
On Thu, 7 Oct 2021 at 11:41, Leif Lindholm wrote: > > Ard/Sami - any comments? > The code changes by itself look fine to me. The only problem I see is that we cannot run arbitrary code on other cores with the MMU off, given that in that case, they don't comply with the UEFI AArch64 platform requi

Re: [edk2-devel] [PATCH v6 1/1] MdePkg/BaseLib: Add QuickSort function on BaseLib

2021-10-07 Thread Marvin Häuser
On 06/10/2021 02:30, IanX Kuo wrote: From: IanX Kuo REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3675 Add QuickSort function into BaseLib Cc: Ray Ni Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: IanX Kuo --- MdePkg/Include/Library/BaseLib.h |

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Leif Lindholm
Hi Michael, Apologies, I've owed you a response (promised off-list) for a while now. First, let me say I hugely appreciate this effort. Apart from aligning the codebase(s), this will reduce manual reviewing effort substantially, as well as cutting down on number of rework cycles for developers.

Re: [edk2-devel] [RFC] [PATCH 0/2] Proposal to add EFI_MP_SERVICES_PROTOCOL support for AARCH64

2021-10-07 Thread Leif Lindholm
On Thu, Oct 07, 2021 at 12:03:30 +0200, Ard Biesheuvel wrote: > On Thu, 7 Oct 2021 at 11:41, Leif Lindholm wrote: > > > > Ard/Sami - any comments? > > > > The code changes by itself look fine to me. The only problem I see is > that we cannot run arbitrary code on other cores with the MMU off, > g

Re: [edk2-devel] [PATCH v1 7/7] DynamicTablesPkg: SSDT Pci express generator

2021-10-07 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. I have some feedback marked inline as [SAMI]. Regards, Sami Muajwar On 23/06/2021 12:58 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois This generator allows to generate a SSDT table describing a Pci express Bus. It uses the following CmObj: - E

[edk2-devel] Event: TianoCore edk2-test Bug Triage Meeting - 10/07/2021 #cal-reminder

2021-10-07 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:Asia/Shanghai LAST-MODIFIED:20201011T015911Z TZURL:http://tzurl.org/zoneinfo-outlook/Asia/Shanghai X-LIC-LOCAT

[edk2-devel] Now: TianoCore edk2-test Bug Triage Meeting - 10/07/2021 #cal-notice

2021-10-07 Thread devel@edk2.groups.io Calendar
*TianoCore edk2-test Bug Triage Meeting* *When:* 10/07/2021 10:00pm to 11:00pm (UTC+08:00) Asia/Shanghai *Where:* https://armltd.zoom.us/j/93809865843?pwd=dU1hSzk4NHM2RGhaRDRyWWZxUzY5dz09&from=addon *Organizer:* Edhaya Chandran edhaya.chand...@arm.com ( edhaya.chand...@arm.com?subject=Re:%20Eve

[edk2-devel] [[edk2-test] PATCH v1 1/1] SctPkg: Fix X64 build errors for GCC toolchain

2021-10-07 Thread Abdul Lateef Attar via groups.io
From: Abdul Lateef Attar From: Your Name Signed-off-by: Abdul Lateef Attar --- uefi-sct/SctPkg/TestCase/UEFI/EFI/Protocol/DebugSupport/BlackBoxTest/DebugSupportBBTestMain.h | 1 + uefi-sct/SctPkg/TestCase/UEFI/EFI/Protocol/DevicePathFromText/BlackBoxTest/DevicePathFromTex

[edk2-devel] [edk2-test] [PATCH v1 0/1] SctPkg: Fix X64 build errors for GCC toolchain

2021-10-07 Thread Abdul Lateef Attar via groups.io
Removes duplicate function definations. Adds EFIAPI to function definations/declerations. Abdul Lateef Attar (1): SctPkg: Fix X64 build errors for GCC toolchain uefi-sct/SctPkg/TestCase/UEFI/EFI/Protocol/DebugSupport/BlackBoxTest/DebugSupportBBTestMain.h | 1 + uefi-sct/Sc

[edk2-devel] [edk2platforms][PATCH v1 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-07 Thread Maciej Czajkowski
Added IntelDieInfoProtocol header into IntelSiliconPkg tree. The purpose is to have generic and unified interface for getting information about dies installed in the system. It will be implemented by silicon code. Change-Id: I219f238a6245eb81dbaebe6b826304baf3f78dc2 Cc: Ray Ni Cc: Rangasai V Chag

[edk2-devel] [edk2platforms][PATCH v1 0/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-07 Thread Maciej Czajkowski
This commit will add a protocol/PPI definition which will provide a generic and unified way to get information about dies installed in the system(PCH, SoC, CPU). It will be implemented per generation in silicon packages. Cc: Ray Ni Cc: Rangasai V Chaganty Maciej Czajkowski (1): IntelSilicon

[edk2-devel] MOM: TianoCore edk2-test Bug Triage Meeting

2021-10-07 Thread G Edhaya Chandran
Hello All, Thank you for attending the meeting edk2-test Bug Triage Meeting. Below is the MOM of the proceedings: Closed Issues from the previous triage meeting: ID▼

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Andrew Fish via groups.io
> On Oct 7, 2021, at 6:48 AM, Leif Lindholm wrote: > > Hi Michael, > > Apologies, I've owed you a response (promised off-list) for a while > now. > > First, let me say I hugely appreciate this effort. Apart from aligning > the codebase(s), this will reduce manual reviewing effort > substantia

[edk2-devel] [PATCH v2 00/21] Create a SSDT CPU topology generator

2021-10-07 Thread PierreGondois
From: Pierre Gondois v2: - Update or error handling. [Sami/Pierre] - Various comments. [Sami] and New patches: DynamicTablesPkg: Remove unnecessary includes DynamicTablesPkg: Add missing parameter check DynamicTablesPkg: Add AddSsdtAcpiHeader() DynamicTablesPkg: Add AmlRdSetEndTagChecksum() Dynam

[edk2-devel] [PATCH v2 02/21] DynamicTablesPkg: Add missing parameter check

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add missing check of 'AcpiTableInfo' in AddAcpiHeader(). Signed-off-by: Pierre Gondois --- Notes: v2: - New patch. [Pierre] DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/DynamicTablesPkg/Library

[edk2-devel] [PATCH v2 06/21] DynamicTablesPkg: Set EndTag's Checksum if RdList is modified

2021-10-07 Thread PierreGondois
From: Pierre Gondois ACPI 6.4, s6.4.2.9 "End Tag": "This checksum is generated such that adding it to the sum of all the data bytes will produce a zero sum." "If the checksum field is zero, the resource data is treated as if the checksum operation succeeded. Configuration proceeds normally." To

[edk2-devel] [PATCH v2 07/21] DynamicTablesPkg: Clear pointer in node creation fcts

2021-10-07 Thread PierreGondois
From: Pierre Gondois The following functions: - AmlCreateRootNode() - AmlCreateObjectNode() - AmlCreateDataNode() create a node and return it by populating a pointer. This pointer should only be considered/used if the function returns successfully. Otherwise, the value stored in this pointer shou

[edk2-devel] [PATCH v2 03/21] DynamicTablesPkg: Add AddSsdtAcpiHeader()

2021-10-07 Thread PierreGondois
From: Pierre Gondois To fetch the Oem information from the ConfigurationManagerProtocol and the AcpiTableInfo, and populate the SSDT ACPI header when creating a RootNode via the AmlLib, create AddSsdtAcpiHeader(). Signed-off-by: Pierre Gondois --- .../Include/Library/TableHelperLib.h

[edk2-devel] [PATCH v2 01/21] DynamicTablesPkg: Remove unnecessary includes

2021-10-07 Thread PierreGondois
From: Pierre Gondois Some includes are not necessary. Remove then. Signed-off-by: Pierre Gondois --- Notes: v2: - New patch. [Pierre] .../Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c| 2 -- .../Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c | 3 --- .../Com

[edk2-devel] [PATCH v2 05/21] DynamicTablesPkg: Add AmlSetRdListCheckSum()

2021-10-07 Thread PierreGondois
From: Pierre Gondois Lists of Resource Data elements end with an EndTag (most of the time). This function finds the EndTag (if present) in a list of Resource Data elements and sets the checksum. ACPI 6.4, s6.4.2.9 "End Tag": "This checksum is generated such that adding it to the sum of all the d

[edk2-devel] [PATCH v2 04/21] DynamicTablesPkg: Add AmlRdSetEndTagChecksum()

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlRdSetEndTagChecksum(), setting the CheckSum value contained in a Resource Data element. ACPI 6.4, s6.4.2.9 "End Tag": "This checksum is generated such that adding it to the sum of all the data bytes will produce a zero sum." "If the checksum field is zero, the resourc

[edk2-devel] [PATCH v2 09/21] DynamicTablesPkg: Make AmlNodeGetIntegerValue public

2021-10-07 Thread PierreGondois
From: Pierre Gondois Remove the STATIC qualifier for the AmlUtility function AmlNodeGetIntegerValue() and add the definition to the header file so that it can be used by other AmlLib sub-modules. Reviewed-by: Sami Mujawar Signed-off-by: Pierre Gondois --- .../Library/Common/AmlLib/Utils/AmlUt

[edk2-devel] [PATCH v2 10/21] DynamicTablesPkg: AML Code generation for Register()

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlCodeGenRegister() to generate AML code for the Generic Register Resource Descriptor. This function is equivalent to the ASL macro Register(). Reviewed-by: Sami Mujawar Signed-off-by: Pierre Gondois --- .../AmlLib/CodeGen/AmlResourceDataCodeGen.c | 87

[edk2-devel] [PATCH v2 11/21] DynamicTablesPkg: AML Code generation for Resource data EndTag

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add a helper function AmlCodeGenEndTag() to generate AML Resource Data EndTag. The EndTag resource data is automatically generated by the ASL compiler at the end of a list of resource data elements. Therefore, an equivalent function is not present in ASL. However, AmlCodeGen

[edk2-devel] [PATCH v2 08/21] DynamicTablesPkg: Update error handling for node creation

2021-10-07 Thread PierreGondois
From: Pierre Gondois The node creation functions: - AmlCreateRootNode() - AmlCreateObjectNode() - AmlCreateDataNode() are now resetting the input pointer where the created node is stored. Thus, it is not necessary to set some local variables to NULL or check a node value before trying to delete i

[edk2-devel] [PATCH v2 13/21] DynamicTablesPkg: Helper function to compute package length

2021-10-07 Thread PierreGondois
From: Pierre Gondois Some AML object have a PkgLen which indicates the size of the AML object. The package length can be encoded in 1 to 4 bytes. The bytes used to encode the PkgLen is itself counted in the PkgLen value. So, if an AML object's size increments/decrements, the number of bytes used

[edk2-devel] [PATCH v2 12/21] DynamicTablesPkg: AML code generation for a Package

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlCodeGenPackage() to generate AML code for declaring a Package() object. This function generates an empty package node. New elements can then be added to the package's variable argument list. Signed-off-by: Pierre Gondois --- .../Common/AmlLib/CodeGen/AmlCodeGen.c

[edk2-devel] [PATCH v2 14/21] DynamicTablesPkg: AML code generation for a ResourceTemplate

2021-10-07 Thread PierreGondois
From: Pierre Gondois ASL provides a ResourceTemplate macro that creates a Buffer in which resource descriptor macros can be listed. The ResourceTemplate macro automatically generates an End descriptor and calculates the checksum for the resource template. Therefore, add AmlCodeGenResourceTemplat

[edk2-devel] [PATCH v2 15/21] DynamicTablesPkg: AML code generation for a Method

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlCodeGenMethod() to generate code for a control method. AmlCodeGenMethod ("MET0", 1, TRUE, 3, ParentNode, NewObjectNode) is equivalent of the following ASL code: Method(MET0, 1, Serialized, 3) {} Signed-off-by: Pierre Gondois --- .../Common/AmlLib/CodeGen/AmlCod

[edk2-devel] [PATCH v2 16/21] DynamicTablesPkg: AML code generation to Return a NameString

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlCodeGenReturnNameString() to generate AML code for a Return object node, returning the object as a NameString. AmlCodeGenReturn ("NAM1", ParentNode, NewObjectNode) is equivalent of the following ASL code: Return(NAM1) Signed-off-by: Pierre Gondois --- .../Common/

[edk2-devel] [PATCH v2 18/21] DynamicTablesPkg: AML code generation for a _LPI object

2021-10-07 Thread PierreGondois
From: Pierre Gondois _LPI object provides a method to describe Low Power Idle states that define the local power states for each node in a hierarchical processor topology. Therefore, add AmlCreateLpiNode() to generate code for a _LPI object. AmlCreateLpiNode ("_LPI", 0, 1, ParentNode, &LpiNode)

[edk2-devel] [PATCH v2 17/21] DynamicTablesPkg: AML code generation for a Method returning a NS

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlCodeGenMethodRetNameString() to generate AML code to create a Method returning a NameString (NS). AmlCodeGenMethodRetNameString ( "MET0", "_CRS", 1, TRUE, 3, ParentNode, NewObjectNode ); is equivalent of the following ASL code: Method(MET0, 1, Serialized, 3) { R

[edk2-devel] [PATCH v2 19/21] DynamicTablesPkg: AML code generation to add an _LPI state

2021-10-07 Thread PierreGondois
From: Pierre Gondois Add AmlAddLpiState() to generates AML code to add an _LPI state to an _LPI object created using AmlCreateLpiNode(). AmlAddLpiState increments the count of LPI states in the LPI node by one, and adds the following package: Package() { MinResidency, WorstCaseWakeLate

[edk2-devel] [PATCH v2 20/21] DynamicTablesPkg: Add CM_ARM_LPI_INFO object

2021-10-07 Thread PierreGondois
From: Pierre Gondois Introduce the CM_ARM_LPI_INFO CmObj in the ArmNameSpaceObjects. This allows to describe LPI state information, as described in ACPI 6.4, s8.4.4.3 "_LPI (Low Power Idle States)". Reviewed-by: Sami Mujawar Signed-off-by: Pierre Gondois --- .../Include/ArmNameSpaceObjects.h

[edk2-devel] [PATCH v2 21/21] DynamicTablesPkg: SSDT CPU topology and LPI state generator

2021-10-07 Thread PierreGondois
From: Pierre Gondois In the GIC interrupt model, logical processors are required to have a Processor Device object in the DSDT and must convey each processor's GIC information to the OS using the GICC structure. Additionally, _LPI objects may be needed as they provide a method to describe Low Pow

[edk2-devel] Event: TianoCore Community Meeting - EMEA / NAMO - 10/07/2021 #cal-reminder

2021-10-07 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20201011T015911Z TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angeles

[edk2-devel] Event: TianoCore Community Meeting - EMEA / NAMO - 10/07/2021 #cal-reminder

2021-10-07 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20201011T015911Z TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angeles

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Andrew Fish via groups.io
On the git-blame front we might be able to build a recipe that walks past the code style and line ending commits that would could add to an FAQ or Wiki. For bonus points we could add a git command that does this. You can add a Python git command to the repo by adding a git- script in the path.

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Marvin Häuser
Good day, +1, but while you're at it, can we have arguments not align to the function name... Status = Test ( a ); ... but to the next natural indentation level? Status = Test ( a ); Basically no IDE I have seen supports EDK II's style, and I wouldn't b

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Michael D Kinney
Hi Michael, I am looking at your uncrustify_poc_3 branch, and I see an unexpected change related to use of IN/OUT in a function declaration. File: MdeModulePkg\Logo\Logo.c Before Uncrustify = EFI_STATUS EFIAPI GetImage ( IN EDKII_PLATFORM_LOGO_PROTOCOL *This, IN

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Michael Kubacki
Thanks Mike, I will look into that. For others, I recently published an "uncrustify_poc_3" branch on my edk2 fork that demonstrates the latest set of results I've obtained with Uncrustify tool changes and the configuration file. Most of the edk2-specific issues have been mitigated and I was ab

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Michael D Kinney
Hi Leif, Thanks for the feedback. Adjusting the EDK II C Coding Style should absolutely be part of this discussion. We need to work towards specific proposals. Here are the current open bugs against the EDK II C Coding Style. Please update or add new for the changes you are suggesting. htt

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Michael D Kinney
Hi Marvin, Some comments below. Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of Marvin Häuser > Sent: Thursday, October 7, 2021 11:31 AM > To: Leif Lindholm ; devel@edk2.groups.io; > mikub...@linux.microsoft.com > Cc: rebe...@nuviainc.com; Michael Kubacki ; > Bret

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Andrew Fish via groups.io
> On Oct 7, 2021, at 1:19 PM, Michael D Kinney > wrote: > > Hi Marvin, > > Some comments below. > > Mike > >> -Original Message- >> From: devel@edk2.groups.io >> mailto:devel@edk2.groups.io>> On Behalf Of Marvin >> Häuser >> Sent: Thursday, October 7,

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Marvin Häuser
Hey Mike, Hey Andrew, I'll just reply to both mails at once :) On 07/10/2021 19:36, Andrew Fish wrote: On Oct 7, 2021, at 1:19 PM, Michael D Kinney wrote: Hi Marvin, Some comments below. Mike -Original Message- From:devel@edk2.groups.io On Behalf Of Marvin Häuser Sent: Thurs

Re: [edk2-devel] Progress on getting Uncrustify working for EDK2?

2021-10-07 Thread Andrew Fish via groups.io
> On Oct 7, 2021, at 1:43 PM, Marvin Häuser wrote: > > Hey Mike, > Hey Andrew, > > I'll just reply to both mails at once :) > > On 07/10/2021 19:36, Andrew Fish wrote: >> >> >>> On Oct 7, 2021, at 1:19 PM, Michael D Kinney >>> wrote: >>> >>> Hi Marvin, >>> >>> Some comments below. >>>

[edk2-devel] [PATCH v7 0/1] Add function QuickSort into MdePkg/BaseLib

2021-10-07 Thread IanX Kuo
From: IanX Kuo First change 1. MdePkg/BaseLib: Add QuickSort function It need to seperate to second change 2. MdeModulePkg/SortLib: Use QuickSort instead of QuickSortWorker 3. CryptLib/CryptLib: Remove duplicate QuickSortWorker 4. CpuCacheInfoLib: Remove MdeModulePkg dependency IanX Kuo (1):

[edk2-devel] [PATCH v7 1/1] MdePkg/BaseLib: Add QuickSort function on BaseLib

2021-10-07 Thread IanX Kuo
From: IanX Kuo REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3675 Add QuickSort function into BaseLib Cc: Ray Ni Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: IanX Kuo --- MdePkg/Include/Library/BaseLib.h | 49 MdePkg/Library/BaseLib/Base

Re: [edk2-devel] [PATCH V8 3/3] OvmfPkg: Enable TDX in ResetVector

2021-10-07 Thread Yao, Jiewen
I propose to submit independent patch with independent metadata (2 tables) - don't complicate thing. We can revisit to see if there is need to merge to 1 table or how to merge, as a separate/standalone patch. Thank you Yao Jiewen From: Singh, Brijesh Sent: Friday, October 1, 2021 1:40 AM To:

回复: [edk2-devel] [PATCH v1 1/1] MdePkg: Fix ACPI memory aggregator/device type mismatch

2021-10-07 Thread gaoliming
Jones: Do you know what impact will be introduced by this change? Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Chris Jones > 发送时间: 2021年10月6日 18:12 > 收件人: devel@edk2.groups.io > 抄送: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn; > zhiguang@intel.com; sami.muja...@a

Re: [edk2-devel] [PATCH] MdeModulePkg/Core/Dxe: Add lock protection in CoreLocateHandleBuffer()

2021-10-07 Thread Dandan Bi
Reviewed-by: Dandan Bi Thanks, Dandan > -Original Message- > From: Ma, Hua > Sent: Wednesday, September 29, 2021 1:49 PM > To: devel@edk2.groups.io > Cc: Ma, Hua ; Wang, Jian J ; > Liming Gao ; Bi, Dandan > Subject: [PATCH] MdeModulePkg/Core/Dxe: Add lock protection in > CoreLocateHa

Re: [edk2-devel] [PATCH] UserAuthFeaturePkg/UserAuthenticationDxeSmm: The SMI to handle the user authentication should be unregister before booting to OS

2021-10-07 Thread Dandan Bi
Reviewed-by: Dandan Bi Thanks, Dandan > -Original Message- > From: Shi, Hao > Sent: Tuesday, September 28, 2021 10:09 AM > To: devel@edk2.groups.io > Cc: Shi, Hao ; Bi, Dandan ; > Liming Gao > Subject: [PATCH] UserAuthFeaturePkg/UserAuthenticationDxeSmm: The > SMI to handle the user

Re: [edk2-devel] [PATCH v1 1/1] MdePkg: Fix ACPI memory aggregator/device type mismatch

2021-10-07 Thread Samer El-Haj-Mahmoud
We did investigate this in the BZ, and the conclusion was it is safer to update the code to match the spec. The only OS implementation we have seen so far is in Linux, and it uses the spec defined values (although for limited usage). See https://bugzilla.tianocore.org/show_bug.cgi?id=3579 >

回复: [edk2-devel] [PATCH v1 1/1] MdePkg: Fix ACPI memory aggregator/device type mismatch

2021-10-07 Thread gaoliming
Samer: Thanks for your information. I agree this change. Reviewed-by: Liming Gao Thanks Liming > -邮件原件- > 发件人: Samer El-Haj-Mahmoud > 发送时间: 2021年10月8日 9:45 > 收件人: devel@edk2.groups.io; gaolim...@byosoft.com.cn; Christopher > Jones > 抄送: michael.d.kin...@intel.com; zhiguang@intel.

[edk2-devel] Event: TianoCore Community Meeting - APAC/NAMO - 10/07/2021 #cal-reminder

2021-10-07 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20201011T015911Z TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angeles

回复: [edk2-devel] [PATCH v7 1/1] MdePkg/BaseLib: Add QuickSort function on BaseLib

2021-10-07 Thread gaoliming
Kuo: I suggest to move SORT_COMPARE definition from MdeModulePkg\Include\Library\SortLib.h to MdePkg\Include\Library\BaseLib.h, and then let SortLib.h include BaseLib.h. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 IanX Kuo > 发送时间: 2021年10月8日 8:27 > 收件人: devel@edk2.groups.io >

回复: 回复: [edk2-devel] Python2.7 is not working with the EDK2 build system

2021-10-07 Thread gaoliming
Cole: Here is the error message https://edk2.groups.io/g/devel/message/80068?p=,,,20,0,0,0::recentpostdate%252Fsticky,,python,20,2,0,85296733 Yes, Python27 has been end of life. But, Edk2 has not announced to drop Python27 support. So, if someone meets with python27 issue, we need to respo

[edk2-devel] Event: TianoCore Community Meeting - APAC/NAMO - 10/07/2021 #cal-reminder

2021-10-07 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20201011T015911Z TZURL:http://tzurl.org/zoneinfo-outlook/America/Los_Angeles

[edk2-devel] [edk2-platforms: PATCH 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH 1/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH 2/9] CometlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH 3/9] KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH 4/9] KabylakeOpenBoardPkg/GalagoPro3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc: N

[edk2-devel] [edk2-platforms: PATCH 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Sign

[edk2-devel] [edk2-platforms: PATCH 7/9] WhiskeylakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Sign

[edk2-devel] [edk2-platforms: PATCH 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH 9/9] WhitleySiliconPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Isaac Oram Cc: Nat

[edk2-devel] 回复: [PATCH] MdeModulePkg/Core/Dxe: Add lock protection in CoreLocateHandleBuffer()

2021-10-07 Thread gaoliming
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Bi, Dandan > 发送时间: 2021年10月8日 9:40 > 收件人: Ma, Hua ; devel@edk2.groups.io > 抄送: Wang, Jian J ; Liming Gao > > 主题: RE: [PATCH] MdeModulePkg/Core/Dxe: Add lock protection in > CoreLocateHandleBuffer() > > Reviewed-by: Dandan Bi > > > > Thanks, >

回复: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to OvmfPkg

2021-10-07 Thread gaoliming
Ard and Abner: I am OK to add these three PCDs PcdPciMmio32Translation, PcdPciMmio64Translation, PcdPciIoTranslation to MdePkg. Thanks Liming 发件人: Chang, Abner (HPS SW/FW Technologist) 发送时间: 2021年10月6日 17:27 收件人: edk2-devel-groups-io ; a...@kernel.org; Chang, Abner (HPS SW/FW Technologis

Re: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to OvmfPkg

2021-10-07 Thread Abner Chang
Thanks Liming, could you please also give the reviewed-by. Hi Ard, also need your reviewed-by for ArmPkg. Then this changes could be upstream and finished. Thanks Abner From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of gaoliming Sent: Friday, October 8, 2021 11:14 AM To: Cha

[edk2-devel] [edk2-platforms: PATCH v2 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
V2: Fix GCC build failures. V1: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where

[edk2-devel] [edk2-platforms: PATCH v2 1/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v2 2/9] CometlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v2 3/9] KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v2 4/9] KabylakeOpenBoardPkg/GalagoPro3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v2 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Sign

[edk2-devel] [edk2-platforms: PATCH v2 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc: N

[edk2-devel] [edk2-platforms: PATCH v2 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v2 9/9] WhitleySiliconPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Isaac Oram Cc: Nat

[edk2-devel] [edk2-platforms: PATCH v2 7/9] WhiskeylakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Sign

[edk2-devel] [PATCH V2 0/3] Introduce TdProtocol into EDK2

2021-10-07 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625 If TD-Guest firmware supports measurement and an event is created, TD-Guest firmware is designed to report the event log with the same data structure in TCG-Platform-Firmware-Profile specification with EFI_TCG2_EVENT_LOG_FORMAT_TCG_2 format.

[edk2-devel] [PATCH V2 1/3] MdePkg: Introduce TdProtocol for TD-Guest firmware

2021-10-07 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625 If TD-Guest firmware supports measurement and an event is created, TD-Guest firmware is designed to report the event log with the same data structure in TCG-Platform-Firmware-Profile specification with EFI_TCG2_EVENT_LOG_FORMAT_TCG_2 format.

[edk2-devel] [PATCH V2 2/3] SecurityPkg: Support TdProtocol in DxeTpm2MeasureBootLib

2021-10-07 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625 DxeTpm2MeasureBootLib supports TPM2 based measure boot. After Td protocol is introduced, TD based measure boot needs to be supported in DxeTpm2MeasureBootLib as well. There are 2 major changes in this commit. 1. MEASURE_BOOT_PROTOCOLS is de

[edk2-devel] [PATCH V2 3/3] SecurityPkg: Support TdProtocol in DxeTpmMeasurementLib

2021-10-07 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625 DxeTpmMeasurementLib supports TPM based measurement in DXE phase. After Td protocol is introduced, TD based measurement needs to be supported in DxeTpmMeasurementLib as well. In TpmMeasureAndLogData, TD based measurement will be first called

[edk2-devel] 回复: [PATCH v1 0/9] Add Raw algorithm support using Arm FW-TRNG interface

2021-10-07 Thread gaoliming
Sami: Here is my comments for this patch set. 1. MdeModulePkg\Include\Guid\ZeroGuid.h has defined gZeroGuid. You don't define it again. 2. MdePkg\Include\Library\TrngLib.h is base type library. It can use RETURN_STATUS instead of EFI_STATUS. Then, it doesn't need to include 3. MdePkg\Include\

[edk2-devel] [PATCH v8 0/1] Add function QuickSort into MdePkg/BaseLib

2021-10-07 Thread IanX Kuo
From: IanX Kuo First change 1. MdePkg/BaseLib: Add QuickSort function It need to seperate to second change 2. MdeModulePkg/SortLib: Use QuickSort instead of QuickSortWorker 3. CryptLib/CryptLib: Remove duplicate QuickSortWorker 4. CpuCacheInfoLib: Remove MdeModulePkg dependency IanX Kuo (1):

[edk2-devel] [PATCH v8 1/1] MdePkg/BaseLib: Add QuickSort function on BaseLib

2021-10-07 Thread IanX Kuo
From: IanX Kuo REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3675 Add QuickSort function into BaseLib Cc: Ray Ni Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: IanX Kuo --- .../Library/BaseCryptLib/SysCall/CrtWrapper.c | 8 +- MdeModulePkg/Include/Library/Sort

Re: [edk2-devel] [PATCH v1] ShellPkg: Update Acpiview HMAT parser to ACPI spec version 6.4

2021-10-07 Thread Gao, Zhichao
Yes, there is a typo. Please correct my email address mailto:zhichao@intel.com>>. Or you can directly copy the name from Maintainers.txt. Thanks, Zhichao From: Sami Mujawar Sent: Wednesday, October 6, 2021 9:25 PM To: Gao; Gao, Zhichao ; devel@edk2.groups.io Subject: Re: [edk2-devel] [PATCH

[edk2-devel] [edk2-platforms: PATCH v3 1/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

[edk2-devel] [edk2-platforms: PATCH v3 0/9] MinPlatformPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
V3: Fix another GCC build failure. V2: Fix GCC build failures. V1: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariabl

[edk2-devel] [edk2-platforms: PATCH v3 3/9] KabylakeOpenBoardPkg/AspireVn7Dash572G:Use same variable name for FspNvsHob

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v3 2/9] CometlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v3 4/9] KabylakeOpenBoardPkg/GalagoPro3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Cc:

[edk2-devel] [edk2-platforms: PATCH v3 5/9] KabylakeOpenBoardPkg/KabylakeRvp3: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Nate DeSimone Sign

[edk2-devel] [edk2-platforms: PATCH v3 6/9] TigerlakeOpenBoardPkg: Use same variable name for FspNvsHob.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 To simplify the implementation the variable Name/GUID has been changed to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the data from FSP_NON_VOLATILE_STORAGE_HOB2 or FSP_NON_VOLATILE_STORAGE_HOB. Cc: Sai Chaganty Cc: N

[edk2-devel] [edk2-platforms: PATCH v3 8/9] WhitleyOpenBoardPkg: Support FSP 2.3 FSP_NON_VOLATILE_STORAGE_HOB2.

2021-10-07 Thread Chiu, Chasel
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 Implementation should search FSP_NON_VOLATILE_STORAGE_HOB2 firstly and only search FSP_NON_VOLATILE_STORAGE_HOB when former one is not found. Also added PeiGetLargeVariable () to support the scenarios where the variable data size is bigger

  1   2   >