Instead printing the result high() close directly move it to main()
function to simplify the return code if the test fails.
Signed-off-by: Daniel Wagner <dwagner@xxxxxxx>
---
src/pi_tests/pip_stress.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/src/pi_tests/pip_stress.c b/src/pi_tests/pip_stress.c
index 8b91578f9ab8..2c3288984202 100644
--- a/src/pi_tests/pip_stress.c
+++ b/src/pi_tests/pip_stress.c
@@ -146,6 +146,7 @@ int main(int argc, char *argv[])
exit(1);
} else if (pid1 != 0) { /* parent code */
low(pid1);
+ exit(0);
} else { /* child code */
pid2 = fork(); /* parent code */
if (pid2 == -1) {
@@ -155,9 +156,15 @@ int main(int argc, char *argv[])
high(pid2);
} else { /* child code */
medium();
+ exit(0);
}
}
+ if (statep->inversion)
+ printf("Successfully used priority inheritance to handle an inversion\n");
+ else
+ printf("No inversion incurred\n");
+
exit(0);
}
@@ -222,14 +229,6 @@ void high(pid_t pid)
Pthread_mutex_unlock(resource);
kill(pid, SIGKILL); /* kill the medium thread */
waitpid(pid, &status, 0);
-
- Pthread_mutex_lock(statep->mutex);
-
- if (statep->inversion)
- printf("Successfully used priority inheritance to handle an inversion\n");
- else
- printf("No inversion incurred\n");
- Pthread_mutex_unlock(statep->mutex);
}
/* mmap a page of anonymous shared memory */
--
2.30.2
