From 3290f8412a4562adbc05b38d8e5212090e41587e Mon Sep 17 00:00:00 2001 From: Luiz Capitulino Date: Thu, 17 Mar 2016 14:29:04 -0400 Subject: don't use exit(-1) for failures The kernel uses only 8 bits of the status as a return code, so this actually becomes 255 in the shell. In any case, the most widely convension is exit(1) for failures, so let's be consistent. Signed-off-by: Luiz Capitulino Signed-off-by: John Kacur --- src/pi_tests/pi_stress.c | 2 +- src/pi_tests/pip_stress.c | 6 +++--- src/rt-migrate-test/rt-migrate-test.c | 5 ++--- src/signaltest/signaltest.c | 2 +- 4 files changed, 7 insertions(+), 8 deletions(-) diff --git a/src/pi_tests/pi_stress.c b/src/pi_tests/pi_stress.c index a02f706..36b64ca 100644 --- a/src/pi_tests/pi_stress.c +++ b/src/pi_tests/pi_stress.c @@ -634,7 +634,7 @@ int verify_cpu(int cpu) if (status == -1) { err = errno; fprintf(stderr, "sched_getaffinity %s\n", strerror(err)); - exit(-1); + exit(1); } if (CPU_ISSET(cpu, &mask)) diff --git a/src/pi_tests/pip_stress.c b/src/pi_tests/pip_stress.c index a0477cc..e6cf0ce 100644 --- a/src/pi_tests/pip_stress.c +++ b/src/pi_tests/pip_stress.c @@ -95,7 +95,7 @@ int main(void) *minimum_priority = sched_get_priority_min(policy); if (check_privs()) - exit(-1); + exit(1); mptr = mmap_page(); /* Get a page of shared memory */ resource = (pthread_mutex_t*)mptr; /* point our lock to it */ @@ -138,7 +138,7 @@ int main(void) pid2 = fork(); /* parent code */ if (pid2 == -1) { perror("fork: "); - exit(-1); + exit(1); } else if (pid2 != 0) { /* parent code */ high(pid2); } else { /* child code */ @@ -231,7 +231,7 @@ void *mmap_page(void) mptr = mmap(NULL, pgsize, PROTRW, MMAP_FLAGS, 0, 0); if (mptr == MAP_FAILED) { perror("In function mmap_page - mmap"); - exit(-1); + exit(1); } return mptr; diff --git a/src/rt-migrate-test/rt-migrate-test.c b/src/rt-migrate-test/rt-migrate-test.c index d7b68dd..e18beca 100644 --- a/src/rt-migrate-test/rt-migrate-test.c +++ b/src/rt-migrate-test/rt-migrate-test.c @@ -138,7 +138,7 @@ static void perr(char *fmt, ...) perror(buffer); fflush(stderr); - exit(-1); + exit(1); } static void print_progress_bar(int percent) @@ -226,7 +226,6 @@ static void parse_options (int argc, char *argv[]) break; } } - } static unsigned long long get_time(void) @@ -604,7 +603,7 @@ int main (int argc, char **argv) exit(1); } if (check < 0) - exit(-1); + exit(1); else exit(0); diff --git a/src/signaltest/signaltest.c b/src/signaltest/signaltest.c index b80969b..59f979e 100644 --- a/src/signaltest/signaltest.c +++ b/src/signaltest/signaltest.c @@ -323,7 +323,7 @@ int main(int argc, char **argv) process_options(argc, argv); if (check_privs()) - exit(-1); + exit(1); /* lock all memory (prevent paging) */ if (lockall) -- cgit 1.2.3-korg