Thread (8 messages) flat view 8 messages, 3 authors, 5d ago

Re: [PATCH 0/2] powerpc/sstep: Fix compilation on 32-bit PowerPC

From: "Christophe Leroy (CS GROUP)" <chleroy@kernel.org>
Date: 2026-07-30 05:51:49
Also in: lkml


Le 29/07/2026 à 22:49, Link Mauve a écrit :
Two local variables were only set but never read on 32-bit PowerPC,
which causes an error during compilation.
Are those errors new ? This series and the one from yesterday on ppc64, 
I have the feeling that the code is not new. Has anything changed in the 
building flags or does this pops up with recent versions of GCC ? If it 
is the case it is worth saying it in the commit messages in order to 
clarify the reason for the fix.
This series first makes the 64-bit guards consistent by converting all
uses of CONFIG_PPC64 to use __powerpc64__ instead, and then guard the
two variables rc and suffix behind __powerpc64__ checks.

Link Mauve (2):
   powerpc/sstep: Consistently use one define to check arch bit width
   powerpc/sstep: Don’t define variables we won’t be using

  arch/powerpc/lib/sstep.c | 26 ++++++++++++++++----------
  1 file changed, 16 insertions(+), 10 deletions(-)


base-commit: 78bc8af4affb9a732504eb22eeac7d1e50883853
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help