On Thu, Aug 03, 2017 at 01:43:39PM +0200, Daniel Vetter wrote:
> pm_rpm is one of the main offenders for long runtime in our full igt
> run. And hey, there's a quick option already, so make it the default.
> 
> This means there's yet another way to enable stress tests, we really
> need to standardize this somehow. Or shred the stress-tests we really
> don't want/need anymore.
> 
> Cc: Paulo Zanoni <paulo.r.zan...@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hi...@intel.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
> ---
>  tests/pm_rpm.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c
> index 1873c9017b6e..8962bc3e56d8 100644
> --- a/tests/pm_rpm.c
> +++ b/tests/pm_rpm.c
> @@ -1852,14 +1852,14 @@ static void fences_subtest(bool dpms)
>       igt_assert(munmap(buf_ptr, params.fb.size) == 0);
>  }
>  
> -int rounds = 40;
> +int rounds = 10;
>  bool stay = false;
>  
>  static int opt_handler(int opt, int opt_index, void *data)
>  {
>       switch (opt) {
>       case 'q':
> -             rounds = 10;
> +             rounds = 50;
>               break;
>       case 's':
>               stay = true;
> @@ -1874,10 +1874,10 @@ static int opt_handler(int opt, int opt_index, void 
> *data)
>  int main(int argc, char *argv[])
>  {
>       const char *help_str =
> -            "  --quick\t\tMake the stress-tests not stressful, for quick 
> regression testing.\n"
> +            "  --stress\t\tMake the stress-tests more stressful, for quick 
> regression testing.\n"

wouldn't adding more rounds make it *less* quick?

>              "  --stay\t\tDisable all screen and try to go into runtime pm. 
> Useful for debugging.";
>       static struct option long_options[] = {
> -             {"quick", 0, 0, 'q'},
> +             {"stress", 0, 0, 'q'},
>               {"stay", 0, 0, 's'},
>               { 0, 0, 0, 0 }
>       };
> -- 
> 2.13.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to