diff options
author | Luiz Capitulino <lcapitulino@redhat.com> | 2016-03-17 14:29:04 -0400 |
---|---|---|
committer | John Kacur <jkacur@redhat.com> | 2016-03-22 15:13:03 +0100 |
commit | 3290f8412a4562adbc05b38d8e5212090e41587e (patch) | |
tree | abe270b94ee4b08882f711a82f013ed1dc18c7a4 | |
parent | 1a1be10bbd5b58016df4cfe357a2e9b3d7cafd46 (diff) | |
download | rt-tests-3290f8412a4562adbc05b38d8e5212090e41587e.tar.gz |
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 <lcapitulino@redhat.com>
Signed-off-by: John Kacur <jkacur@redhat.com>
-rw-r--r-- | src/pi_tests/pi_stress.c | 2 | ||||
-rw-r--r-- | src/pi_tests/pip_stress.c | 6 | ||||
-rw-r--r-- | src/rt-migrate-test/rt-migrate-test.c | 5 | ||||
-rw-r--r-- | 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) |