Re: [rt-tests v2 v2 06/20] cyclictest: Move verbose message into main

[John Kacur <jkacur@redhat.com>]

On Fri, 18 Dec 2020, Daniel Wagner wrote:

> By moving the verbose message down we print the message with the final
> affinity_mask. This also handles the case where the bitmask is not set.
> 
> Signed-off-by: Daniel Wagner <dwagner@suse.de>
> ---
>  src/cyclictest/cyclictest.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
> index a34b2140f940..7e84c25f7070 100644
> --- a/src/cyclictest/cyclictest.c
> +++ b/src/cyclictest/cyclictest.c
> @@ -1020,10 +1020,6 @@ static void process_options(int argc, char *argv[], int max_cpus)
>  				    argv[optind][0] == '!')) {
>  				parse_cpumask(argv[optind], max_cpus, &affinity_mask);
>  			}
> -
> -			if (verbose)
> -				printf("Using %u cpus.\n",
> -					numa_bitmask_weight(affinity_mask));
>  			break;
>  		case 'A':
>  		case OPT_ALIGNED:
> @@ -1732,6 +1728,10 @@ int main(int argc, char **argv)
>  		res = numa_sched_setaffinity(getpid(), affinity_mask);
>  		if (res != 0)
>  			warn("Couldn't setaffinity in main thread: %s\n", strerror(errno));
> +
> +		if (verbose)
> +			printf("Using %u cpus.\n",
> +				numa_bitmask_weight(affinity_mask));
>  	}
>  
>  	if (trigger) {
> -- 
> 2.29.2
> 
> 
Signed-off-by: John Kacur <jkacur@redhat.com>