powerpc/xmon: Fix disassembly CPU feature checks
authorMichael Ellerman <mpe@ellerman.id.au>
Thu, 9 May 2024 12:12:47 +0000 (22:12 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 11 Jul 2024 07:31:54 +0000 (17:31 +1000)
commit14196e47c5ffe32af7ed5a51c9e421c5ea5bccce
tree0c644a954c977e80d02714b21c8309e372c74602
parent489116d784bebec7e441f400715fbfe6edbce66c
powerpc/xmon: Fix disassembly CPU feature checks

In the xmon disassembly code there are several CPU feature checks to
determine what dialects should be passed to the disassembler. The
dialect controls which instructions the disassembler will recognise.

Unfortunately the checks are incorrect, because instead of passing a
single CPU feature they are passing a mask of feature bits.

For example the code:

  if (cpu_has_feature(CPU_FTRS_POWER5))
      dialect |= PPC_OPCODE_POWER5;

Is trying to check if the system is running on a Power5 CPU. But
CPU_FTRS_POWER5 is a mask of *all* the feature bits that are enabled on
a Power5.

In practice the test will always return true for any 64-bit CPU, because
at least one bit in the mask will be present in the CPU_FTRS_ALWAYS
mask.

Similarly for all the other checks against CPU_FTRS_xx masks.

Rather than trying to match the disassembly behaviour exactly to the
current CPU, just differentiate between 32-bit and 64-bit, and Altivec,
VSX and HTM.

That will cause some instructions to be shown in disassembly even
on a CPU that doesn't support them, but that's OK, objdump -d output
has the same behaviour, and if anything it's less confusing than some
instructions not being disassembled.

Fixes: 897f112bb42e ("[POWERPC] Import updated version of ppc disassembly code for xmon")
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20240509121248.270878-2-mpe@ellerman.id.au
arch/powerpc/xmon/ppc-dis.c