Commit 3fcfb5d1 authored by Christophe Leroy's avatar Christophe Leroy Committed by Michael Ellerman
Browse files

powerpc/signal: Remove BUG_ON() in handler_signal functions 



There is already the same BUG_ON() check in do_signal() which
is the only caller of handle_rt_signal64() handle_rt_signal32() and
handle_signal32().

Remove those three redundant BUG_ON().

Signed-off-by: default avatarChristophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/3582e10a341d523c9c3f1ac925c3aaefc9d9293d.1597770847.git.christophe.leroy@csgroup.eu
parent 7d68c891

arch/powerpc/kernel/signal_32.c

+0 −4
Original line number Diff line number Diff line
@@ -764,8 +764,6 @@ int handle_rt_signal32(struct ksignal *ksig, sigset_t *oldset, 
	unsigned long msr = regs->msr;

#endif



	BUG_ON(tsk != current);



	/* Set up Signal Frame */

	/* Put a Real Time Context onto stack */

	rt_sf = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*rt_sf), 1);
@@ -1227,8 +1225,6 @@ int handle_signal32(struct ksignal *ksig, sigset_t *oldset, 
	unsigned long msr = regs->msr;

#endif



	BUG_ON(tsk != current);



	/* Set up Signal Frame */

	frame = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*frame), 1);

	if (unlikely(frame == NULL))

arch/powerpc/kernel/signal_64.c

+0 −2
Original line number Diff line number Diff line
@@ -822,8 +822,6 @@ int handle_rt_signal64(struct ksignal *ksig, sigset_t *set, 
	unsigned long msr = regs->msr;

#endif



	BUG_ON(tsk != current);



	frame = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*frame), 0);

	if (unlikely(frame == NULL))

		goto badframe;