Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
Hi Pierre, Thanks for reviewing the patch series. IORT table has separate specification document and not included as a part of ACPI 6.4. That was the reason for excluding IORT from this series. If required, it will be updated in a separate patch. I've posted V3 of this patch, no major changes, I request you to have a look at that: https://edk2.groups.io/g/devel/message/90399. Regards, Pranav > On 6/22/22 10:57, Pranav Madhu via groups.io wrote:> > Hello Pranav, > > The definitions for the IORT table are still at 6_0, (IORT revision=0): > MdePkg/Include/IndustryStandard/IoRemappingTable.h > and the definitions are used at: > Platform/ARM/SgiPkg/AcpiTables/Iort.aslc > > The IORT spec is now at the version E.d (revision=5). > Does the IORT need to be updated aswell ? > > Regards, > Pierre > > On 6/20/22 18:52, Pranav Madhu via groups.io wrote: > > Hi All, > > > > Could you please let me know if any updates are required for this series of > patch? > > > > Regards, > > Pranav > > > >> -Original Message- > >> From: Thomas Abraham > >> Sent: Tuesday, April 19, 2022 1:40 AM > >> To: devel@edk2.groups.io; Pranav Madhu > >> Cc: Ard Biesheuvel ; Sami Mujawar > >> > >> Subject: Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the > >> ACPI tables for RD platforms > >> > >> > >> On 4/18/22 12:54, Pranav Madhu via groups.io wrote: > >>> Changes since V1: > >>> - Rebased on top of latest master branch. > >>> - Rebased on top of patch to remove SLC cache entries from PPTT (link > >>> for the same in edk2.groups.io is provided below) > >>> > >>> Arm infrastructure reference design platforms uses ACPI tables to > >>> provide the hardware information to the operating system. Currently > >>> the ACPI tables are aligned with ACPI v6.2 and v6.3 specification. > >>> This patch series update the tables to ACPI v6.4 specificaion. > >>> > >>> The first patch in this series update the generic tables which are > >>> common for all platfoms. The subsequent patches in this series > >>> update platform specific ACPI tables to v6.4 for the respective platform. > >>> > >>> This patch series also update the headers in the .aslc files to > >>> match the coding style as per edk2 coding guidelines. > >>> > >>> This patch series should be applied on top of the patch series > >>> https://edk2.groups.io/g/devel/message/88996 > >>> > >>> Link to github branch with the patches in this series - > >>> https://github.com/Pranav-Madhu/edk2- > >> platforms/tree/topics/acpi64_for_ > >>> rd_platforms > >>> > >>> > >>> Pranav Madhu (9): > >>> Platform/Sgi: Update ACPI version to v6.4 > >>> Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform > >>> Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 > >>> platform > >> > >> For this series - > >> Reviewed-by: Thomas Abraham > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, > please notify the sender immediately and do not disclose the contents to any > other person, use it for any purpose, or store or copy the information in any > medium. Thank you. > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#90669): https://edk2.groups.io/g/devel/message/90669 Mute This Topic: https://groups.io/mt/90537675/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
Hello Pranav, The definitions for the IORT table are still at 6_0, (IORT revision=0): MdePkg/Include/IndustryStandard/IoRemappingTable.h and the definitions are used at: Platform/ARM/SgiPkg/AcpiTables/Iort.aslc The IORT spec is now at the version E.d (revision=5). Does the IORT need to be updated aswell ? Regards, Pierre On 6/20/22 18:52, Pranav Madhu via groups.io wrote: Hi All, Could you please let me know if any updates are required for this series of patch? Regards, Pranav -Original Message- From: Thomas Abraham Sent: Tuesday, April 19, 2022 1:40 AM To: devel@edk2.groups.io; Pranav Madhu Cc: Ard Biesheuvel ; Sami Mujawar Subject: Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms On 4/18/22 12:54, Pranav Madhu via groups.io wrote: Changes since V1: - Rebased on top of latest master branch. - Rebased on top of patch to remove SLC cache entries from PPTT (link for the same in edk2.groups.io is provided below) Arm infrastructure reference design platforms uses ACPI tables to provide the hardware information to the operating system. Currently the ACPI tables are aligned with ACPI v6.2 and v6.3 specification. This patch series update the tables to ACPI v6.4 specificaion. The first patch in this series update the generic tables which are common for all platfoms. The subsequent patches in this series update platform specific ACPI tables to v6.4 for the respective platform. This patch series also update the headers in the .aslc files to match the coding style as per edk2 coding guidelines. This patch series should be applied on top of the patch series https://edk2.groups.io/g/devel/message/88996 Link to github branch with the patches in this series - https://github.com/Pranav-Madhu/edk2- platforms/tree/topics/acpi64_for_ rd_platforms Pranav Madhu (9): Platform/Sgi: Update ACPI version to v6.4 Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 platform For this series - Reviewed-by: Thomas Abraham IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#90654): https://edk2.groups.io/g/devel/message/90654 Mute This Topic: https://groups.io/mt/90537675/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
Hi All, Could you please let me know if any updates are required for this series of patch? Regards, Pranav > -Original Message- > From: Thomas Abraham > Sent: Tuesday, April 19, 2022 1:40 AM > To: devel@edk2.groups.io; Pranav Madhu > Cc: Ard Biesheuvel ; Sami Mujawar > > Subject: Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI > tables for RD platforms > > > On 4/18/22 12:54, Pranav Madhu via groups.io wrote: > > Changes since V1: > > - Rebased on top of latest master branch. > > - Rebased on top of patch to remove SLC cache entries from PPTT (link > >for the same in edk2.groups.io is provided below) > > > > Arm infrastructure reference design platforms uses ACPI tables to > > provide the hardware information to the operating system. Currently > > the ACPI tables are aligned with ACPI v6.2 and v6.3 specification. > > This patch series update the tables to ACPI v6.4 specificaion. > > > > The first patch in this series update the generic tables which are > > common for all platfoms. The subsequent patches in this series update > > platform specific ACPI tables to v6.4 for the respective platform. > > > > This patch series also update the headers in the .aslc files to match > > the coding style as per edk2 coding guidelines. > > > > This patch series should be applied on top of the patch series > > https://edk2.groups.io/g/devel/message/88996 > > > > Link to github branch with the patches in this series - > > https://github.com/Pranav-Madhu/edk2- > platforms/tree/topics/acpi64_for_ > > rd_platforms > > > > > > Pranav Madhu (9): > >Platform/Sgi: Update ACPI version to v6.4 > >Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform > >Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 platform > > For this series - > Reviewed-by: Thomas Abraham IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#90632): https://edk2.groups.io/g/devel/message/90632 Mute This Topic: https://groups.io/mt/90537675/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
Hi Ard, Thanks for your time in reviewing this patch series. Please find my response inline. > -Original Message- > From: Ard Biesheuvel > Sent: Wednesday, April 20, 2022 1:11 PM > To: Pranav Madhu > Cc: edk2-devel-groups-io ; Ard Biesheuvel > ; Sami Mujawar > Subject: Re: [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD > platforms > > On Mon, 18 Apr 2022 at 13:54, Pranav Madhu > wrote: > > > > Changes since V1: > > - Rebased on top of latest master branch. > > - Rebased on top of patch to remove SLC cache entries from PPTT (link > > for the same in edk2.groups.io is provided below) > > > > Arm infrastructure reference design platforms uses ACPI tables to > > provide the hardware information to the operating system. Currently > > the ACPI tables are aligned with ACPI v6.2 and v6.3 specification. > > This patch series update the tables to ACPI v6.4 specificaion. > > > > Why is this necessary? By doing this, you will lose the ability to boot an OS > that > implements ACPI 6.2 but not ACPI 6.4, and so this should only be done if you > are > actually relying on a feature that was not available in ACPI 6.2. Otherwise, > this > is just pointless busywork. > As per SystemReady SBBR specification, all the ACPI tables should adhere to a specific ACPI revision. At present the FADT, GTDT, MADT... tables are as per ACPI specification 6.2 and PPTT, HMAT... tables are as per specification 6.3. To achieve compliance, we need to either downgrade some tables to 6.2 or else upgrade the rest of the tables to 6.3. As a solution for this, upgrade all tables to 6.4 as this is the most recent. The effort required to bring all tables to ACPI 6.3 and 6.4 are almost same. Also, with ACPI 6.4 we successfully validated WinPE and other Linux distributions. > > > The first patch in this series update the generic tables which are > > common for all platfoms. The subsequent patches in this series update > > platform specific ACPI tables to v6.4 for the respective platform. > > > > This patch series also update the headers in the .aslc files to match > > the coding style as per edk2 coding guidelines. > > > > This patch series should be applied on top of the patch series > > https://edk2.groups.io/g/devel/message/88996 > > > > Link to github branch with the patches in this series - > > https://github.com/Pranav-Madhu/edk2-platforms/tree/topics/acpi64_for_ > > rd_platforms > > > > > > Pranav Madhu (9): > > Platform/Sgi: Update ACPI version to v6.4 > > Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform > > Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 platform > > > > Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h | 219 > > Platform/ARM/SgiPkg/AcpiTables/Dbg2.aslc | 30 ++- > > Platform/ARM/SgiPkg/AcpiTables/Fadt.aslc | 38 +-- > > Platform/ARM/SgiPkg/AcpiTables/Gtdt.aslc | 69 ++--- > > Platform/ARM/SgiPkg/AcpiTables/Mcfg.aslc | 26 +- > > .../ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc | 247 > > +- .../ARM/SgiPkg/AcpiTables/RdE1Edge/Pptt.aslc | > > 106 .../ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc | 105 > > .../ARM/SgiPkg/AcpiTables/RdN1Edge/Pptt.aslc | 88 --- > > .../SgiPkg/AcpiTables/RdN1EdgeX2/Hmat.aslc| 52 ++-- > > .../SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc| 151 ++- > > .../SgiPkg/AcpiTables/RdN1EdgeX2/Pptt.aslc| 92 +++ > > .../SgiPkg/AcpiTables/RdN1EdgeX2/Srat.aslc| 74 +++--- > > Platform/ARM/SgiPkg/AcpiTables/RdN2/Madt.aslc | 163 ++-- > > Platform/ARM/SgiPkg/AcpiTables/RdN2/Pptt.aslc | 77 +++--- > > .../ARM/SgiPkg/AcpiTables/RdN2Cfg1/Madt.aslc | 112 > > .../ARM/SgiPkg/AcpiTables/RdN2Cfg1/Pptt.aslc | 73 +++--- > > Platform/ARM/SgiPkg/AcpiTables/RdV1/Madt.aslc | 159 +-- > > Platform/ARM/SgiPkg/AcpiTables/RdV1/Pptt.aslc | 77 +++--- > > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Hmat.aslc| 64 +++-- > > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Madt.aslc| 165 ++-- > > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Pptt.aslc| 81 +++--- > > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Srat.aslc| 82 +++--- > > .../ARM/SgiPkg/AcpiTables/Sgi575/Madt.aslc| 105 > > .../ARM/SgiPkg/AcpiTables/Sgi575/Pptt.aslc| 86 +++--- > > Platform/ARM/SgiPkg/AcpiTables/Spcr.aslc | 29 +- > > 26 files changed, 1343 insertions(+), 1227 deletions(-) > > > > -- > > 2.17.1 > > Regards, Pranav -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#89122):
Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
On Mon, 18 Apr 2022 at 13:54, Pranav Madhu wrote: > > Changes since V1: > - Rebased on top of latest master branch. > - Rebased on top of patch to remove SLC cache entries from PPTT (link > for the same in edk2.groups.io is provided below) > > Arm infrastructure reference design platforms uses ACPI tables to > provide the hardware information to the operating system. Currently the > ACPI tables are aligned with ACPI v6.2 and v6.3 specification. This > patch series update the tables to ACPI v6.4 specificaion. > Why is this necessary? By doing this, you will lose the ability to boot an OS that implements ACPI 6.2 but not ACPI 6.4, and so this should only be done if you are actually relying on a feature that was not available in ACPI 6.2. Otherwise, this is just pointless busywork. > The first patch in this series update the generic tables which are > common for all platfoms. The subsequent patches in this series update > platform specific ACPI tables to v6.4 for the respective platform. > > This patch series also update the headers in the .aslc files to match > the coding style as per edk2 coding guidelines. > > This patch series should be applied on top of the patch series > https://edk2.groups.io/g/devel/message/88996 > > Link to github branch with the patches in this series - > https://github.com/Pranav-Madhu/edk2-platforms/tree/topics/acpi64_for_rd_platforms > > > Pranav Madhu (9): > Platform/Sgi: Update ACPI version to v6.4 > Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform > Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 platform > > Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h | 219 > Platform/ARM/SgiPkg/AcpiTables/Dbg2.aslc | 30 ++- > Platform/ARM/SgiPkg/AcpiTables/Fadt.aslc | 38 +-- > Platform/ARM/SgiPkg/AcpiTables/Gtdt.aslc | 69 ++--- > Platform/ARM/SgiPkg/AcpiTables/Mcfg.aslc | 26 +- > .../ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc | 247 +- > .../ARM/SgiPkg/AcpiTables/RdE1Edge/Pptt.aslc | 106 > .../ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc | 105 > .../ARM/SgiPkg/AcpiTables/RdN1Edge/Pptt.aslc | 88 --- > .../SgiPkg/AcpiTables/RdN1EdgeX2/Hmat.aslc| 52 ++-- > .../SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc| 151 ++- > .../SgiPkg/AcpiTables/RdN1EdgeX2/Pptt.aslc| 92 +++ > .../SgiPkg/AcpiTables/RdN1EdgeX2/Srat.aslc| 74 +++--- > Platform/ARM/SgiPkg/AcpiTables/RdN2/Madt.aslc | 163 ++-- > Platform/ARM/SgiPkg/AcpiTables/RdN2/Pptt.aslc | 77 +++--- > .../ARM/SgiPkg/AcpiTables/RdN2Cfg1/Madt.aslc | 112 > .../ARM/SgiPkg/AcpiTables/RdN2Cfg1/Pptt.aslc | 73 +++--- > Platform/ARM/SgiPkg/AcpiTables/RdV1/Madt.aslc | 159 +-- > Platform/ARM/SgiPkg/AcpiTables/RdV1/Pptt.aslc | 77 +++--- > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Hmat.aslc| 64 +++-- > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Madt.aslc| 165 ++-- > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Pptt.aslc| 81 +++--- > .../ARM/SgiPkg/AcpiTables/RdV1Mc/Srat.aslc| 82 +++--- > .../ARM/SgiPkg/AcpiTables/Sgi575/Madt.aslc| 105 > .../ARM/SgiPkg/AcpiTables/Sgi575/Pptt.aslc| 86 +++--- > Platform/ARM/SgiPkg/AcpiTables/Spcr.aslc | 29 +- > 26 files changed, 1343 insertions(+), 1227 deletions(-) > > -- > 2.17.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#89119): https://edk2.groups.io/g/devel/message/89119 Mute This Topic: https://groups.io/mt/90537675/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH V2 0/9] Upadate the ACPI tables for RD platforms
On 4/18/22 12:54, Pranav Madhu via groups.io wrote: Changes since V1: - Rebased on top of latest master branch. - Rebased on top of patch to remove SLC cache entries from PPTT (link for the same in edk2.groups.io is provided below) Arm infrastructure reference design platforms uses ACPI tables to provide the hardware information to the operating system. Currently the ACPI tables are aligned with ACPI v6.2 and v6.3 specification. This patch series update the tables to ACPI v6.4 specificaion. The first patch in this series update the generic tables which are common for all platfoms. The subsequent patches in this series update platform specific ACPI tables to v6.4 for the respective platform. This patch series also update the headers in the .aslc files to match the coding style as per edk2 coding guidelines. This patch series should be applied on top of the patch series https://edk2.groups.io/g/devel/message/88996 Link to github branch with the patches in this series - https://github.com/Pranav-Madhu/edk2-platforms/tree/topics/acpi64_for_rd_platforms Pranav Madhu (9): Platform/Sgi: Update ACPI version to v6.4 Platform/Sgi: Update ACPI version to v6.4 for SGI-575 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N1-Edge-X2 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-E1-Edge platform Platform/Sgi: Update ACPI version to v6.4 for Rd-V1 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-V1-MC platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N2 platform Platform/Sgi: Update ACPI version to v6.4 for Rd-N2-Cfg1 platform For this series - Reviewed-by: Thomas Abraham IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#89024): https://edk2.groups.io/g/devel/message/89024 Mute This Topic: https://groups.io/mt/90537675/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
