Re: [patch 2/3] cyclictest: Convert the offset of the alignment option to microseconds

[John Kacur <jkacur@redhat.com>]

On Tue, 26 May 2015, anna-maria@glx-um.de wrote:

> The offset is specified in microseconds according to the
> documentation, but, the microseconds to nanoseconds conversion is
> missing so the effective offset has the unit of nanoseconds.
> 
> Signed-off-by: Anna-Maria Gleixner <anna-maria@glx-um.de>
> ---
>  src/cyclictest/cyclictest.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: rt-tests/src/cyclictest/cyclictest.c
> ===================================================================
> --- rt-tests.orig/src/cyclictest/cyclictest.c
> +++ rt-tests/src/cyclictest/cyclictest.c
> @@ -1289,7 +1289,7 @@ static void process_options (int argc, c
>  		case OPT_ALIGNED:
>  			aligned=1;
>  			if (optarg != NULL)
> -				offset = atoi(optarg);
> +				offset = atoi(optarg) * 1000;
>  			else if (optind<argc && atoi(argv[optind])

Don't we need the conversion here too then?
>  				offset = atoi(argv[optind]);

>  			else
> 
> 
> --

Thanks

John