From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Egorov Subject: Re: [PATCH] rt-migrate-test: exit early if nr_runs is non-positive Date: Tue, 17 Feb 2015 20:30:36 +0600 Message-ID: <54E3508C.3080704@linux.com> References: <5476FAF2.1060702@linux.com> <20150217140838.GP26177@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: 716237@bugs.debian.org, linux-rt-users@vger.kernel.org To: Sebastian Andrzej Siewior , Clark Williams Return-path: Received: from mail-la0-f54.google.com ([209.85.215.54]:45982 "EHLO mail-la0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932920AbbBQOan (ORCPT ); Tue, 17 Feb 2015 09:30:43 -0500 Received: by labge10 with SMTP id ge10so35919787lab.12 for ; Tue, 17 Feb 2015 06:30:41 -0800 (PST) In-Reply-To: <20150217140838.GP26177@linutronix.de> Sender: linux-rt-users-owner@vger.kernel.org List-ID: Looks like this patch is not applied yet. I can move value check to parse_options and send another version of patch. As for macros - yes, I already mentioned them in debian bug report thread. There are three of them (INTERVAL, RUN_INTERVAL, PROGRESS_CHARS) which are improperly (IMO) used instead of corresponding lowercase variables. I thought these changes should go to another patch. On 02/17/2015 08:08 PM, Sebastian Andrzej Siewior wrote: > * Boris Egorov | 2014-11-27 16:20:34 [+0600]: > >> Program will crash if nr_runs is 0 due to dividing by it in >> print_results(). Let's exit early instead. >> >> Fixes: http://bugs.debian.org/716237 > > Has this been dealt with? I can't see this applied. I would prefer > doing the value check in parse_options() itself. And looking at it, it > seems that run_interval is interval ignored. Or is there macro magic > invovled? > >> Signed-off-by: Boris Egorov >> --- >> src/rt-migrate-test/rt-migrate-test.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/src/rt-migrate-test/rt-migrate-test.c >> b/src/rt-migrate-test/rt-migrate-test.c >> index e3c7a09..876a122 100644 >> --- a/src/rt-migrate-test/rt-migrate-test.c >> +++ b/src/rt-migrate-test/rt-migrate-test.c >> @@ -465,6 +465,11 @@ int main (int argc, char **argv) >> parse_options(argc, argv); >> + if (nr_runs <= 0) { >> + fprintf(stderr, "Warning, --loops argument is non-positive. Exiting.\n"); >> + exit(-1); >> + } >> + >> signal(SIGINT, stop_log); >> if (argc >= (optind + 1)) >> -- >> 2.1.3 > > Sebastian > -- Best regards, Boris Egorov