diff options
| author | Ashish Mhetre <amhetre@nvidia.com> | 2026-07-03 04:56:53 +0000 |
|---|---|---|
| committer | Krzysztof Kozlowski <krzk@kernel.org> | 2026-07-08 13:14:48 +0200 |
| commit | 8111c7f3723f414b7f655a3880775cbbd0e9de8e (patch) | |
| tree | 066d1f7cc7c5a77212d2cc1924363789f81f2e3d /tools/perf/scripts/python/bin/stackcollapse-report | |
| parent | 5239692170f1815c0bbc57de432d7eb8b616f8f1 (diff) | |
memory: tegra: Guard against NULL mc_regs in IRQ handler
The per-error decode in tegra30_mc_handle_irq() dereferences
mc->soc->regs unconditionally. This 'regs' structure is optional and is
only used for decoding and logging MC error interrupts. The rest of the
MC functionality does not depend on it.
When adding support for the Tegra238 SoC, the 'regs' structure was
initially omitted because it is only used for error logging. We found
that this resulted in a NULL pointer dereference in IRQ context, causing
a crash when an MC error interrupt fired. Although existing upstream
devices will not hit this condition because 'regs' is present, guard
against it to improve the robustness of the driver.
Skip the decode and just clear the interrupt when mc_regs is NULL. This
bypasses the error interrupt logging while keeping the remaining MC
functionality intact.
Signed-off-by: Ashish Mhetre <amhetre@nvidia.com>
Link: https://patch.msgid.link/20260703045653.395498-1-amhetre@nvidia.com
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-report')
0 files changed, 0 insertions, 0 deletions
