| Message ID | 20210421164317.1718831-1-maz@kernel.org |
|---|---|
| Headers | show |
| Series | arm64: ACPI GTDT watchdog fixes | expand |
On Wed, Apr 21, 2021 at 05:43:15PM +0100, Marc Zyngier wrote: > Dann recently reported that his ThunderX machine failed to boot since > 64b499d8df40 ("irqchip/gic-v3: Configure SGIs as standard > interrupts"), with a not so pretty crash while trying to send an IPI. > > It turned out to be caused by a mix of broken firmware and a buggy > GTDT watchdog driver. Both have forever been buggy, but the above > commit revealed that the error handling path of the driver was > probably the worse part of it all. > > Anyway, this short series has two goals: > - handle broken firmware in a less broken way > - make sure that the route cause of the problem can be identified > quickly > > Thanks, > > M. > > Marc Zyngier (2): > ACPI: GTDT: Don't corrupt interrupt mappings on watchdow probe failure > ACPI: irq: Prevent unregistering of GIC SGIs > > drivers/acpi/arm64/gtdt.c | 10 ++++++---- > drivers/acpi/irq.c | 6 +++++- > 2 files changed, 11 insertions(+), 5 deletions(-) Patch(2) needs an ACK from Rafael - usually these patches go via the ARM64 tree but I don't think it is compulsory for this series. Thank you ! Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
On Thu, Apr 22, 2021 at 03:23:42PM +0100, Lorenzo Pieralisi wrote: > On Wed, Apr 21, 2021 at 05:43:15PM +0100, Marc Zyngier wrote: > > Dann recently reported that his ThunderX machine failed to boot since > > 64b499d8df40 ("irqchip/gic-v3: Configure SGIs as standard > > interrupts"), with a not so pretty crash while trying to send an IPI. > > > > It turned out to be caused by a mix of broken firmware and a buggy > > GTDT watchdog driver. Both have forever been buggy, but the above > > commit revealed that the error handling path of the driver was > > probably the worse part of it all. > > > > Anyway, this short series has two goals: > > - handle broken firmware in a less broken way > > - make sure that the route cause of the problem can be identified > > quickly > > > > Thanks, > > > > M. > > > > Marc Zyngier (2): > > ACPI: GTDT: Don't corrupt interrupt mappings on watchdow probe failure > > ACPI: irq: Prevent unregistering of GIC SGIs > > > > drivers/acpi/arm64/gtdt.c | 10 ++++++---- > > drivers/acpi/irq.c | 6 +++++- > > 2 files changed, 11 insertions(+), 5 deletions(-) > > Patch(2) needs an ACK from Rafael - usually these patches go via > the ARM64 tree but I don't think it is compulsory for this series. > > Thank you ! > > Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Thanks Lorenzo. Rafael, if there are no objections, I'll take these two patches in the arm64 tree. -- Catalin
On Wed, 21 Apr 2021 17:43:15 +0100, Marc Zyngier wrote: > Dann recently reported that his ThunderX machine failed to boot since > 64b499d8df40 ("irqchip/gic-v3: Configure SGIs as standard > interrupts"), with a not so pretty crash while trying to send an IPI. > > It turned out to be caused by a mix of broken firmware and a buggy > GTDT watchdog driver. Both have forever been buggy, but the above > commit revealed that the error handling path of the driver was > probably the worse part of it all. > > [...] Applied to arm64 (for-next/core), thanks! [1/2] ACPI: GTDT: Don't corrupt interrupt mappings on watchdow probe failure https://git.kernel.org/arm64/c/1ecd5b129252 [2/2] ACPI: irq: Prevent unregistering of GIC SGIs https://git.kernel.org/arm64/c/2a20b08f06e7 -- Catalin
Dann recently reported that his ThunderX machine failed to boot since 64b499d8df40 ("irqchip/gic-v3: Configure SGIs as standard interrupts"), with a not so pretty crash while trying to send an IPI. It turned out to be caused by a mix of broken firmware and a buggy GTDT watchdog driver. Both have forever been buggy, but the above commit revealed that the error handling path of the driver was probably the worse part of it all. Anyway, this short series has two goals: - handle broken firmware in a less broken way - make sure that the route cause of the problem can be identified quickly Thanks, M. Marc Zyngier (2): ACPI: GTDT: Don't corrupt interrupt mappings on watchdow probe failure ACPI: irq: Prevent unregistering of GIC SGIs drivers/acpi/arm64/gtdt.c | 10 ++++++---- drivers/acpi/irq.c | 6 +++++- 2 files changed, 11 insertions(+), 5 deletions(-)