On Thu, 24 Mar 2016 15:28:24 +0100 (CET) John Kacur <jkacur@xxxxxxxxxx> wrote: > From 607dc3627ab7ff28a116c5bb09058aabddf10f27 Mon Sep 17 00:00:00 2001 > From: John Kacur <jkacur@xxxxxxxxxx> > Date: Thu, 24 Mar 2016 15:21:54 +0100 > Subject: [PATCH] Revert changes to rt-migrate-test for exit(1) > > Commit 3290f8412a4562adbc05b38d8e5212090e41587e changed > exit(-1) to exit(1) for many programs in rt-tests for consistency. > > rt-migrate-test needs tri-state exit statuses, because of the way > some existing bash scripts consume it's results. > > This could no-doubt be improved in the future, and there are some > inconsistencies in the exit statuses in this program alone, but as they > are not urgent to fix, and the current fix makes things worse, reverting > this for now. > > Signed-off-by: John Kacur <jkacur@xxxxxxxxxx> > --- > src/rt-migrate-test/rt-migrate-test.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/src/rt-migrate-test/rt-migrate-test.c b/src/rt-migrate-test/rt-migrate-test.c > index e18beca816aa..d7b68ddac3e5 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,6 +226,7 @@ static void parse_options (int argc, char *argv[]) > break; > } > } > + > } > > static unsigned long long get_time(void) > @@ -603,7 +604,7 @@ int main (int argc, char **argv) > exit(1); > } > if (check < 0) > - exit(1); > + exit(-1); > else > exit(0); > Reviewed-by: Luiz Capitulino <lcapitulino@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
