diff options
author | Nicholas Piggin <npiggin@gmail.com> | 2024-08-06 23:13:12 +1000 |
---|---|---|
committer | Nicholas Piggin <npiggin@gmail.com> | 2024-11-04 09:08:24 +1000 |
commit | 84416e262ea1218026a8567ed9ea31c16d77edea (patch) | |
tree | 497cb6af801d1bb47eafe16e9ac7affe001186e7 /module-common.c | |
parent | 899e488650bb8bd52e1b2b44ceaae17df2e20b7f (diff) |
ppc/pnv: Fix LPC POWER8 register sanity check
POWER8 does not have the ISA IRQ -> SERIRQ routing system of later
CPUs, instead all ISA IRQs are sent to the CPU via a single PSI
interrupt. There is a sanity check in the POWER8 case to ensure the
routing bits have not been set, because that would indicate a
programming error.
Those bits were incorrectly specified because of ppc bit numbering
fun. Coverity detected this as an always-zero expression.
Cc: qemu-stable@nongnu.org
Reported-by: Cédric Le Goater <clg@redhat.com>
Resolves: Coverity CID 1558829 (partially)
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Diffstat (limited to 'module-common.c')
0 files changed, 0 insertions, 0 deletions