kselftest/arm64: Don't repeat termination handler for fp-stress
authorMark Brown <broonie@kernel.org>
Wed, 21 Sep 2022 18:13:43 +0000 (19:13 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Thu, 29 Sep 2022 17:12:37 +0000 (18:12 +0100)
When fp-stress gets a termination signal it sets a flag telling itself to
exit and sends a termination signal to all the children. If the flag is set
then don't bother repeating this process, it isn't going to accomplish
anything other than consume CPU time which can be an issue when running in
emulation.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20220921181345.618085-2-broonie@kernel.org
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
tools/testing/selftests/arm64/fp/fp-stress.c

index a5c0ebef241936407a46b88e59aeee0681f6008d..4387c3cacaa7f93405e63f623d9dda09af329d7e 100644 (file)
@@ -255,6 +255,10 @@ static void handle_exit_signal(int sig, siginfo_t *info, void *context)
 {
        int i;
 
+       /* If we're already exiting then don't signal again */
+       if (terminate)
+               return;
+
        ksft_print_msg("Got signal, exiting...\n");
 
        terminate = true;