<feed xmlns='http://www.w3.org/2005/Atom'>
<title>Tianocore/edk2.git/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Srat, branch master</title>
<subtitle>EDK II (mirror)</subtitle>
<id>https://git.radix-linux.su/Tianocore/edk2.git/atom?h=master</id>
<link rel='self' href='https://git.radix-linux.su/Tianocore/edk2.git/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/'/>
<updated>2026-09-15T10:12:13+00:00</updated>
<entry>
<title>ShellPkg: AcpiView: add support for GICv5 SRAT table</title>
<updated>2026-09-15T10:12:13+00:00</updated>
<author>
<name>Sarah Walker</name>
<email>Sarah.Walker2@arm.com</email>
</author>
<published>2026-06-29T12:47:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=36927430c39dc9236f121f53ede249a33d7342ba'/>
<id>urn:sha1:36927430c39dc9236f121f53ede249a33d7342ba</id>
<content type='text'>
Add support for printing GICv5-related fields with acpiview
in the SRAT table.

Signed-off-by: Sarah Walker &lt;Sarah.Walker2@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg/AcpiView: Update print-formatter prototype</title>
<updated>2024-08-01T17:20:10+00:00</updated>
<author>
<name>Rohit Mathew</name>
<email>Rohit.Mathew@arm.com</email>
</author>
<published>2023-08-22T13:32:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=9e865f9579004b772e36f85ae4e76484bb93b484'/>
<id>urn:sha1:9e865f9579004b772e36f85ae4e76484bb93b484</id>
<content type='text'>
As of now, the print-formatter implemented by the FNPTR_PRINT_FORMATTER
function pointer takes two parameters, the format string and the pointer
to the field. For cases where the print-formatter has to have access to
the length of the field, there is no clean way to currently do it. In
order to resolve this, update the print-formatter's prototype to take
the length of the field as a third parameter. This change should improve
the overall robustness and flexibility of AcpiView.

Signed-off-by: Rohit Mathew &lt;Rohit.Mathew@arm.com&gt;
Cc: James Morse &lt;james.Morse@arm.com&gt;
Cc: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
Cc: Thomas Abraham &lt;thomas.abraham@arm.com&gt;
Cc: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg/AcpiView: Update field-validator prototype</title>
<updated>2024-08-01T17:20:10+00:00</updated>
<author>
<name>Rohit Mathew</name>
<email>Rohit.Mathew@arm.com</email>
</author>
<published>2023-08-22T11:20:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=107d0c380009ad3b17a1730fce264eefdd027616'/>
<id>urn:sha1:107d0c380009ad3b17a1730fce264eefdd027616</id>
<content type='text'>
As of now, the field-validator implemented by FNPTR_FIELD_VALIDATOR
function pointer takes two parameters, the pointer to the field and a
context pointer. For cases where the validator has to have access to the
length of the field, there is no clean way to currently do it. In order
to resolve this, this commit updates the field-validator's prototype to
take the length of the field as an additional parameter.

This enhancement allows field validators to perform more comprehensive
validation, especially when the length of the field is critical to the
validation logic. This change should improve the overall robustness and
flexibility of AcpiView.

Signed-off-by: Rohit Mathew &lt;Rohit.Mathew@arm.com&gt;
Cc: James Morse &lt;james.Morse@arm.com&gt;
Cc: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
Cc: Thomas Abraham &lt;thomas.abraham@arm.com&gt;
Cc: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: Apply uncrustify changes</title>
<updated>2021-12-07T17:24:28+00:00</updated>
<author>
<name>Michael Kubacki</name>
<email>michael.kubacki@microsoft.com</email>
</author>
<published>2021-12-05T22:54:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=47d20b54f9a65b08aa602a1866c1b59a69088dfc'/>
<id>urn:sha1:47d20b54f9a65b08aa602a1866c1b59a69088dfc</id>
<content type='text'>
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3737

Apply uncrustify changes to .c/.h files in the ShellPkg package

Cc: Andrew Fish &lt;afish@apple.com&gt;
Cc: Leif Lindholm &lt;leif@nuviainc.com&gt;
Cc: Michael D Kinney &lt;michael.d.kinney@intel.com&gt;
Signed-off-by: Michael Kubacki &lt;michael.kubacki@microsoft.com&gt;
Reviewed-by: Ray Ni &lt;ray.ni@intel.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg/AcpiView: Refactor configuration</title>
<updated>2020-06-30T01:39:50+00:00</updated>
<author>
<name>Tomas Pilar</name>
<email>Tomas.Pilar@arm.com</email>
</author>
<published>2020-06-19T11:59:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=e18ac66d84e68387dbbe4fd8761b8f3a889534a9'/>
<id>urn:sha1:e18ac66d84e68387dbbe4fd8761b8f3a889534a9</id>
<content type='text'>
A new file and header (AcpiViewConfig.[ch]) is created
that houses the user configuration. This declutters the
core code and improves modularity of the design.

The module level symbols for verbosity, table selection, and
highlighting are refactored into the new file.

Cc: Ray Ni &lt;ray.ni@intel.com&gt;
Cc: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
Reviewed-by: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
Signed-off-by: Tomas Pilar &lt;tomas.pilar@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: acpiview: Prevent infinite loop if structure length is 0</title>
<updated>2020-02-20T09:27:00+00:00</updated>
<author>
<name>Krzysztof Koch</name>
<email>krzysztof.koch@arm.com</email>
</author>
<published>2020-02-19T10:23:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=b85048261aa5dc0b8fd98ddc4431d8c2a6389b25'/>
<id>urn:sha1:b85048261aa5dc0b8fd98ddc4431d8c2a6389b25</id>
<content type='text'>
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2534

Extend validation of ACPI structure lengths which are read from the
ACPI table being parsed. Additionally check if the structure 'Length'
field value is positive. If not, stop parsing the faulting table.

Some ACPI tables define internal structures of variable size. The
'Length' field inside the substructure is used to update a pointer used
for table traversal. If the byte-length of the structure is equal to 0,
acpiview can enter an infinite loop. This condition can occur if, for
example, the zero-allocated ACPI table buffer is not fully populated.
This is typically a bug on the ACPI table writer side.

In short, this method helps acpiview recover gracefully from a
zero-valued ACPI structure length.

Signed-off-by: Krzysztof Koch &lt;krzysztof.koch@arm.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
Reviewed-by: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: acpiview: SRAT: Validate global pointers before use</title>
<updated>2020-02-11T02:12:45+00:00</updated>
<author>
<name>Krzysztof Koch</name>
<email>krzysztof.koch@arm.com</email>
</author>
<published>2020-01-20T11:13:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=ce7b77a71b467daa6ff68f4f3a1330dffc620322'/>
<id>urn:sha1:ce7b77a71b467daa6ff68f4f3a1330dffc620322</id>
<content type='text'>
Check if SratRAType and SratRALength pointers have been successfully
updated before they are used for further table parsing.

Signed-off-by: Krzysztof Koch &lt;krzysztof.koch@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: acpiview: Update SRAT parser to ACPI 6.3</title>
<updated>2020-01-15T05:37:53+00:00</updated>
<author>
<name>Krzysztof Koch</name>
<email>krzysztof.koch@arm.com</email>
</author>
<published>2020-01-08T17:11:44+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=710ff7490ad897383eb35d1becadabd21a733f24'/>
<id>urn:sha1:710ff7490ad897383eb35d1becadabd21a733f24</id>
<content type='text'>
Add support for revision 3 of System Resource Affinity Table (SRAT).

Decode and dump the new Generic Initiator Affinity Structure.

Validate the Device Handle Type field inside the Generic Initiator
Affinity Structure.

Reviewed-by: Alexei Fedorov &lt;Alexei.Fedorov@arm.com&gt;
Reviewed-by: Jaben Carsey &lt;jaben.carsey@intel.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
Reviewed-by: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
Tested-by: Sudipto Paul &lt;sudipto.paul@arm.com&gt;
Signed-off-by: Krzysztof Koch &lt;krzysztof.koch@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: acpiview: SRAT: Prevent buffer overruns</title>
<updated>2019-08-12T17:14:07+00:00</updated>
<author>
<name>Krzysztof Koch</name>
<email>krzysztof.koch@arm.com</email>
</author>
<published>2019-08-01T23:44:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=48d5d6d5e09e5089d1c95c2d0812449e87f4ef7e'/>
<id>urn:sha1:48d5d6d5e09e5089d1c95c2d0812449e87f4ef7e</id>
<content type='text'>
Modify the SRAT parsing logic to prevent reading past the table buffer
length provided.

Check if the Static Resource Allocation Structure's buffer fits in the
SRAT table buffer before its contents are dumped.

Prevent buffer overruns when reading the Static Resource Allocation
Structure's header.

References:
- ACPI 6.3, January 2019, Section 5.2.16

Signed-off-by: Krzysztof Koch &lt;krzysztof.koch@arm.com&gt;
Reviewed-by: Alexei Fedorov &lt;Alexei.Fedorov@arm.com&gt;
Reviewed-by: Zhichao Gao &lt;zhichao.gao@inte.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
</content>
</entry>
<entry>
<title>ShellPkg: acpiview: SRAT: Minor code style enhancements</title>
<updated>2019-07-31T16:52:53+00:00</updated>
<author>
<name>Krzysztof Koch</name>
<email>krzysztof.koch@arm.com</email>
</author>
<published>2019-07-22T22:50:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/Tianocore/edk2.git/commit/?id=748c1efd1e0d934c68b6dafd3011aee47c25ad57'/>
<id>urn:sha1:748c1efd1e0d934c68b6dafd3011aee47c25ad57</id>
<content type='text'>
Minor changes to the SRAT parser code to conform with the EDKII coding
style and to make it consistent with other ACPI table parsers.

Signed-off-by: Krzysztof Koch &lt;krzysztof.koch@arm.com&gt;
Reviewed-by: Sami Mujawar &lt;sami.mujawar@arm.com&gt;
Reviewed-by: Jaben Carsey &lt;jaben.carsey@intel.com&gt;
Reviewed-by: Zhichao Gao &lt;zhichao.gao@intel.com&gt;
</content>
</entry>
</feed>
