From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [PATCH 3/3] pm_rps: Fix verbose option and
 streamline its use
Date: Sun, 19 Jan 2014 22:33:27 +0100
Message-ID: <20140119213327.GC15089@phenom.ffwll.local>
References: <1389999366-19715-1-git-send-email-jeff.mcgee@intel.com>
	<1389999366-19715-4-git-send-email-jeff.mcgee@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mail-ea0-f180.google.com (mail-ea0-f180.google.com
	[209.85.215.180])
	by gabe.freedesktop.org (Postfix) with ESMTP id 43620FA74D
	for <intel-gfx@lists.freedesktop.org>;
	Sun, 19 Jan 2014 13:33:33 -0800 (PST)
Received: by mail-ea0-f180.google.com with SMTP id f15so2741104eak.11
	for <intel-gfx@lists.freedesktop.org>;
	Sun, 19 Jan 2014 13:33:31 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <1389999366-19715-4-git-send-email-jeff.mcgee@intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: jeff.mcgee@intel.com
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Fri, Jan 17, 2014 at 04:56:06PM -0600, jeff.mcgee@intel.com wrote:
> From: Jeff McGee <jeff.mcgee@intel.com>
> 
> Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>
> ---
>  tests/pm_rps.c | 52 ++++++++++++++++++++++++++++++++++++++++++----------
>  1 file changed, 42 insertions(+), 10 deletions(-)
> 
> diff --git a/tests/pm_rps.c b/tests/pm_rps.c
> index 4d64f0a..192dca7 100644
> --- a/tests/pm_rps.c
> +++ b/tests/pm_rps.c
> @@ -31,6 +31,7 @@
>  #include <stdlib.h>
>  #include <string.h>
>  #include <unistd.h>
> +#include <getopt.h>
>  #include "drmtest.h"
>  
>  static bool verbose = false;
> @@ -130,6 +131,8 @@ static void dumpit(void)
>  
>  	printf("\n");
>  }
> +#define dump() if (verbose) dumpit()
> +#define log(...) if (verbose) printf(__VA_ARGS__)
>  
>  static void pm_rps_exit_handler(int sig)
>  {
> @@ -142,8 +145,40 @@ static void pm_rps_exit_handler(int sig)
>  	}
>  }
>  
> -igt_main
> +static int opt_handler(int opt, int opt_index)
>  {
> +	switch (opt) {
> +	case 'v':
> +		verbose = true;
> +		break;
> +	default:
> +		assert(0);
> +	}
> +
> +	return 0;
> +}
> +
> +/* Mod of igt_subtest_init that adds our extra options */
> +void subtest_init(int argc, char **argv)

Missing static here. I've added it back in.
-Daniel

> +{
> +	struct option long_opts[] = {
> +		{"verbose", 0, 0, 'v'}
> +	};
> +	const char *help_str = "  -v, --verbose";
> +	int ret;
> +
> +	ret = igt_subtest_init_parse_opts(argc, argv, "v", long_opts,
> +					  help_str, opt_handler);
> +
> +	if (ret < 0)
> +		/* exit with no error for -h/--help */
> +		exit(ret == -1 ? 0 : ret);
> +}
> +
> +int main(int argc, char **argv)
> +{
> +	subtest_init(argc, argv);
> +
>  	igt_skip_on_simulation();
>  
>  	igt_fixture {
> @@ -176,21 +211,16 @@ igt_main
>  	}
>  
>  	igt_subtest("min-max-config-at-idle") {
> -		if (verbose)
> -			printf("Original min = %d\nOriginal max = %d\n",
> -			        origmin, origmax);
> +		log("Original min = %d\nOriginal max = %d\n", origmin, origmax);
>  
> -		if (verbose)
> -			dumpit();
> +		dump();
>  
>  		checkit();
>  		setfreq(origmin);
> -		if (verbose)
> -			dumpit();
> +		dump();
>  		igt_assert(fcur == fmin);
>  		setfreq(origmax);
> -		if (verbose)
> -			dumpit();
> +		dump();
>  		igt_assert(fcur == fmax);
>  		checkit();
>  
> @@ -209,4 +239,6 @@ igt_main
>  		writeval(stuff[MIN].filp, origmin);
>  		writeval(stuff[MAX].filp, origmax);
>  	}
> +
> +	igt_exit();
>  }
> -- 
> 1.8.5.2
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch