powerpc/traps: Avoid rate limit messages from show unhandled signals
authorMichael Ellerman <mpe@ellerman.id.au>
Thu, 16 Aug 2018 05:27:47 +0000 (15:27 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 20 Aug 2018 10:19:46 +0000 (20:19 +1000)
commit997dd26cb3c8b7c9b8765751cc1491ad33b2024f
tree7d042fbb2b005137beca297ea7cdb4e172b54dd4
parent993ff6d9df74305bc7b5bbc7a0810cf599b6394c
powerpc/traps: Avoid rate limit messages from show unhandled signals

In the recent commit to add an explicit ratelimit state when showing
unhandled signals, commit 35a52a10c3ac ("powerpc/traps: Use an
explicit ratelimit state for show_signal_msg()"), I put the check of
show_unhandled_signals and the ratelimit state before the call to
unhandled_signal() so as to avoid unnecessarily calling the latter
when show_unhandled_signals is false.

However that causes us to check the ratelimit state on every call, so
if we take a lot of *handled* signals that has the effect of making
the ratelimit code print warnings that callbacks have been suppressed
when they haven't.

So rearrange the code so that we check show_unhandled_signals first,
then call unhandled_signal() and finally check the ratelimit state.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Reviewed-by: Murilo Opsfelder Araujo <muriloo@linux.ibm.com>
arch/powerpc/kernel/traps.c