mirror of
https://github.com/torvalds/linux.git
synced 2024-11-26 22:21:42 +00:00
4f1d038b5e
This code is broken since day one. ppc4xx_setup_msi_irqs() has the
following gems:
1) The handling of the result of msi_bitmap_alloc_hwirqs() is completely
broken:
When the result is greater than or equal 0 (bitmap allocation
successful) then the loop terminates and the function returns 0
(success) despite not having installed an interrupt.
When the result is less than 0 (bitmap allocation fails), it prints an
error message and continues to "work" with that error code which would
eventually end up in the MSI message data.
2) On every invocation the file global pp4xx_msi::msi_virqs bitmap is
allocated thereby leaking the previous one.
IOW, this has never worked and for more than 10 years nobody cared. Remove
the gunk.
Fixes: 3fb7933850 ("powerpc/4xx: Adding PCIe MSI support")
Fixes: 247540b03b ("powerpc/44x: Fix PCI MSI support for Maui APM821xx SoC and Bluestone board")
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Link: https://lore.kernel.org/r/20211206210223.872249537@linutronix.de
32 lines
609 B
Plaintext
32 lines
609 B
Plaintext
# SPDX-License-Identifier: GPL-2.0
|
|
# For a description of the syntax of this configuration file,
|
|
# see Documentation/kbuild/kconfig-language.rst.
|
|
#
|
|
|
|
config PPC4xx_PCI_EXPRESS
|
|
bool
|
|
depends on PCI && 4xx
|
|
|
|
config PPC4xx_HSTA_MSI
|
|
bool
|
|
depends on PCI_MSI
|
|
depends on PCI && 4xx
|
|
|
|
config PPC_MSI_BITMAP
|
|
bool
|
|
depends on PCI_MSI
|
|
default y if MPIC
|
|
default y if FSL_PCI
|
|
default y if PPC_POWERNV
|
|
|
|
source "arch/powerpc/sysdev/xics/Kconfig"
|
|
source "arch/powerpc/sysdev/xive/Kconfig"
|
|
|
|
config GE_FPGA
|
|
bool
|
|
|
|
config FSL_CORENET_RCPM
|
|
bool
|
|
help
|
|
This option enables support for RCPM (Run Control/Power Management).
|