[edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring

[Krzysztof Koch]

Hi,

Is there any way I can help get this patch merged? I have a couple of small patch sets ready to submit but they are pending on this series. 

These small sets of patches are a result of me breaking down the "[PATCH v1 00/11] Add security checks in the Acpiview table parsers" series into smaller functional units as requested in the review comments.

Kind regards,

Krzysztof

-----Original Message-----
From: Sami Mujawar <Sami.Mujawar at arm.com> 
Sent: Tuesday, July 23, 2019 10:57
To: Krzysztof Koch <Krzysztof.Koch at arm.com>; devel at edk2.groups.io
Cc: jaben.carsey at intel.com; ray.ni at intel.com; zhichao.gao at intel.com; nd <nd at arm.com>
Subject: RE: [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring

Reviewed-by: Sami Mujawar <sami.mujawar at arm.com>

-----Original Message-----
From: Krzysztof Koch <Krzysztof.Koch at arm.com> 
Sent: 22 July 2019 09:25 AM
To: devel at edk2.groups.io; Krzysztof Koch <Krzysztof.Koch at arm.com>
Cc: jaben.carsey at intel.com; ray.ni at intel.com; zhichao.gao at intel.com; Sami Mujawar <Sami.Mujawar at arm.com>; nd <nd at arm.com>
Subject: RE: [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring

Hi Everyone,

Just to let you know, only "ShellPkg: acpiview: Allow passing buffer length to DumpGasStruct()" is changed compared to v1. I had a wrong understanding of how to post revised patches to the upstream community.

Kind regards,

Krzysztof

-----Original Message-----
From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Krzysztof Koch via Groups.Io
Sent: Monday, July 22, 2019 8:50
To: devel at edk2.groups.io
Cc: jaben.carsey at intel.com; ray.ni at intel.com; zhichao.gao at intel.com; Sami Mujawar <Sami.Mujawar at arm.com>; Matteo Carlini <Matteo.Carlini at arm.com>; nd <nd at arm.com>
Subject: [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring

This set of patches consists of a number of changes which make the code structure consistent across the existing ACPI table parsers. These are all refactoring changes which do not modify the existing functionality of the acpiview UEFI shell tool.

Changes can be seen at: https://github.com/KrzysztofKoch1/edk2/tree/612_acpiview_code_style_enhance_v2

Krzysztof Koch (6):
  ShellPkg: acpiview: Allow passing buffer length to DumpGasStruct()
  ShellPkg: acpiview: XSDT: Remove redundant ParseAcpi() call
  ShellPkg: acpiview: RSDP: Make code consistent with other parsers
  ShellPkg: acpiview: SRAT: Minor code style enhancements
  ShellPkg: acpiview: MADT: Split structure length validation
  ShellPkg: acpiview: IORT: Refactor PMCG node mapping count validation

 ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c              | 26 +++++++++-------
 ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h              |  8 +++--
 ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Dbg2/Dbg2Parser.c |  2 +-  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c | 32 ++++++++++++++------  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Madt/MadtParser.c | 30 +++++++++++++-----  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Rsdp/RsdpParser.c | 11 +++++--  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Srat/SratParser.c |  3 +-  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Xsdt/XsdtParser.c | 18 +++--------
 8 files changed, 82 insertions(+), 48 deletions(-)

--
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'






-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#44463): https://edk2.groups.io/g/devel/message/44463
Mute This Topic: https://groups.io/mt/32556317/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-