Branch: refs/heads/master
Home: https://github.com/tianocore/edk2
Commit: 081763916078300473e1bc1813075a02c0d85f08
https://github.com/tianocore/edk2/commit/081763916078300473e1bc1813075a02c0d85f08
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Declare status in IMSIC RINTC parsing
ReadFdtCells64() stores its result in this local, but the declaration
was omitted. Add it so the parser builds.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 9307efd46dbfcf271de473f66e42bffd7c67783e
https://github.com/tianocore/edk2/commit/9307efd46dbfcf271de473f66e42bffd7c67783e
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtUtility.c
M DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtUtility.h
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Skip disabled interrupt controllers
Respect FDT status when enumerating PLIC, APLIC and IMSIC nodes so
unavailable controllers are not emitted into the MADT. For an MSI-mode
APLIC, also require its selected IMSIC parent to be enabled.
Expose the status check as FdtNodeIsEnabled() in FdtUtility so other
parsers can reuse it. Keep availability checks in the enumeration loops
rather than folding them into the controller-type predicates. Return
EFI_NOT_FOUND when no usable PLIC or APLIC is enumerated.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: c85e6acc959066d9387b63e6a090ff313b9253c8
https://github.com/tianocore/edk2/commit/c85e6acc959066d9387b63e6a090ff313b9253c8
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Account for IMSIC guest-file pages
Use the parsed guest-index width to advance between harts and size
each RINTC IMSIC MMIO region to include supervisor and guest-file
address slots. Bound the final region by the DT reg extent.
Reject out-of-range guest-index values before narrowing the DT cell
to the ACPI field width.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 042772ce0d873adcc8551f00b0a41632905b39b3
https://github.com/tianocore/edk2/commit/042772ce0d873adcc8551f00b0a41632905b39b3
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Initialize reserved interrupt-controller flags
PlicAplicCommonInfo.Flags is copied from an uninitialized stack field
into the PLIC/APLIC configuration-manager object and then the MADT.
Zero the common information structure before enumerating controllers
so reserved flag bits have a defined value.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 7ad39b9223b8a80cccf6e70b2433663023f59757
https://github.com/tianocore/edk2/commit/7ad39b9223b8a80cccf6e70b2433663023f59757
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Validate interrupt tuples before reading IRQs
The IMSIC and APLIC selectors read IrqProp[1] after checking only
that the property contains one cell. Require two complete cells
before reading the interrupt number. Use a null check for the APLIC
property pointer instead of comparing it with an integer.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 3278ad66117e1d0a4530ac411a644cb3624915bf
https://github.com/tianocore/edk2/commit/3278ad66117e1d0a4530ac411a644cb3624915bf
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Correct FDT cell types in RINTC parsing
Use CONST UINT32 pointers for the FDT cells passed to ReadFdtCells64,
matching its interface and avoiding pointer-sign compilation errors.
Store FdtAddressCells's result in INT32 so its negative error codes
are preserved and the existing error check can detect them.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: d97995e99cc52615feea79990fdf6f7dd50f2027
https://github.com/tianocore/edk2/commit/d97995e99cc52615feea79990fdf6f7dd50f2027
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/Acpi/RiscV/AcpiMadtLibRiscV/MadtGenerator.c
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVIntcParser.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Support IMSICs without guest interrupt files
ACPI 6.6 Table 5.57 requires NumGuestIds to be zero when no guest
interrupt files are implemented. The FDT parser currently defaults
the count to NumIds even when no guest index bits are present,
and the MADT generator asserts on a zero count.
Parse and validate the guest index width before deriving NumGuestIds.
An absent riscv,guest-index-bits property defaults to zero, so emit
zero guest identities for an absent or zero width without reading
the unused riscv,num-guest-ids property. For a nonzero width,
preserve the explicit guest identity count or its default from
riscv,num-ids. Validate explicit counts before narrowing, including
rejecting zero as required by the DT binding.
Allow zero in the MADT generator while preserving the bounds and
granularity checks for nonzero counts. The existing RINTC size
calculation already yields one supervisor page for zero guest index
bits and remains bounded by the DT reg extent.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 11944c2159165d4ff8c7371bec7a15efca5327d2
https://github.com/tianocore/edk2/commit/11944c2159165d4ff8c7371bec7a15efca5327d2
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/Acpi/Common/AcpiHmatLib/HmatGenerator.c
Log Message:
-----------
DynamicTablesPkg: Check HMAT domain lookup errors before indexing
AddMemLatBwInfo() only asserts that each proximity-domain lookup
succeeds before using its output index. Assertions can be disabled,
leaving an uninitialized index in use if either lookup fails.
Return the lookup error before writing the relation matrix. Keep the
existing assertions and successful-path behavior. This also fixes
the RISCV64 package build's -Werror=maybe-uninitialized failures.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 41540f4f6298c1fbe1488f11c1e58edc1d455ef3
https://github.com/tianocore/edk2/commit/41540f4f6298c1fbe1488f11c1e58edc1d455ef3
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M
DynamicTablesPkg/Library/Acpi/RiscV/AcpiSsdtPlicAplicLib/SsdtPlicAplicGenerator.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Handle an empty APLIC list in the SSDT
The SSDT generator selects PLIC information whenever AplicCount is
zero, but only fetches that information on EFI_NOT_FOUND. If the
Configuration Manager returns an empty APLIC list successfully, the
generator instead reads uninitialized PlicCount and PlicInfo values.
Fetch PLIC information whenever the APLIC count is zero. The object
getter also sets the count to zero on EFI_NOT_FOUND, preserving that
fallback. Initialize PlicInfo as well so GCC does not warn about the
conditionally assigned pointer in the later device-generation loop.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 275ff0725a6d6691b6bfd10e666d91f9d8e9fb89
https://github.com/tianocore/edk2/commit/275ff0725a6d6691b6bfd10e666d91f9d8e9fb89
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/DynamicTablesPkg.dsc
M DynamicTablesPkg/Library/Acpi/Common/AcpiDbg2Lib/Dbg2Generator.c
M DynamicTablesPkg/Library/Acpi/Common/AcpiPpttLib/PpttGenerator.c
M DynamicTablesPkg/Library/Acpi/RiscV/AcpiRhctLibRiscV/RhctGenerator.c
M DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepo.c
M DynamicTablesPkg/Library/Common/SsdtPcieSupportLib/SsdtPcieSupportLib.c
M DynamicTablesPkg/Library/Smbios/SmbiosType42Lib/SmbiosType42Generator.c
M DynamicTablesPkg/Library/Smbios/SmbiosType7Lib/SmbiosType7Generator.c
Log Message:
-----------
DynamicTablesPkg: Enable RISCV64 package build coverage
Add RISCV64 to the package's supported architectures and include the
FDT parser in its RISC-V components. This lets the package compiler
checks exercise the RISC-V parser and table generators instead of
silently excluding them.
Keep the parser limited to AARCH64 and RISCV64, the architectures
for which it implements a dispatcher.
Address the uninitialized-value warnings exposed by the RISCV64 GCC
build without disabling warnings:
- Return success after grouping CM objects and generating PCI slots.
- Initialize the DBG2 cleanup result when there are no SSDT tables.
- Initialize the SMBIOS Type 7 cleanup count before allocations.
- Initialize the four affected SMBIOS Type 42 CM object pointers.
- Assign RHCT section and PPTT cache offsets before their count checks.
These changes accompany the HMAT and PLIC/APLIC fixes earlier in the
series so both DEBUG and RELEASE package builds succeed with -Werror.
Signed-off-by: Djordje Nedic <[email protected]>
Commit: 27cb976bba49e08aeed9f97039f60d770f4daf5a
https://github.com/tianocore/edk2/commit/27cb976bba49e08aeed9f97039f60d770f4daf5a
Author: Djordje Nedic <[email protected]>
Date: 2026-10-07 (Wed, 07 Oct 2026)
Changed paths:
M DynamicTablesPkg/Library/FdtHwInfoParserLib/RiscV/Intc/RiscVExtIntcData.c
Log Message:
-----------
DynamicTablesPkg/RiscV: Initialize the external interrupt list once
Initialize mExtIntcList statically so GSI lookup and library cleanup
can safely traverse an empty list when no controller has been added.
Remove the list-head reset from FdtCreateExtIntcList(). Resetting the
head on every insertion discards earlier controller mappings and leaks
their entries. Preserve all mappings until the destructor frees them.
Signed-off-by: Djordje Nedic <[email protected]>
Compare: https://github.com/tianocore/edk2/compare/1f31d1c44884...27cb976bba49
To unsubscribe from these emails, change your notification settings at
https://github.com/tianocore/edk2/settings/notifications
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits
