[igt-dev] [PATCH i-g-t] Distinguish particular engines during calculating nop calibration

[Chris Wilson]

Quoting Anna Karas (2020-01-21 15:04:00)
> @@ -3242,14 +3421,13 @@ int main(int argc, char **argv)
>                 goto err;
>         }
>  
> -       if (!nop_calibration) {
> -               if (verbose > 1)
> -                       printf("Calibrating nop delay with %u%% tolerance...\n",
> +       if (!has_nop_calibration) {
> +               if (verbose > 1) {
> +                       printf("Calibrating nop delays with %u%% tolerance...\n",
>                                 tolerance_pct);
> -               nop_calibration = calibrate_nop(tolerance_pct);
> -               if (verbose)
> -                       printf("Nop calibration for %uus delay is %lu.\n",
> -                              nop_calibration_us, nop_calibration);
> +               }
> +
> +               calibrate_engines();
>  
>                 goto out;
>         }
> @@ -3309,8 +3487,7 @@ int main(int argc, char **argv)
>  
>         if (verbose > 1) {

Hmm, could this be
if (verbose > 1 || !has_nop_calibration)
so that a plain run of gem_wsim shows the calibration results?

>                 printf("Random seed is %u.\n", master_prng);
> -               printf("Using %lu nop calibration for %uus delay.\n",
> -                      nop_calibration, nop_calibration_us);
> +               print_engines_calibrations();
>                 printf("%u client%s.\n", clients, clients > 1 ? "s" : "");
>                 if (flags & SWAPVCS)
>                         printf("Swapping VCS rings between clients.\n");
> -- 
> 2.19.0
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
>