display->n_pipes is zero-indexed, so N returned in igt_display_get_n_pipes is already not a valid pipe. This patch prevents kms_ccs from going nuts when testing the first unxesting pipe. Signed-off-by: Gabriel Krisman Bertazi <krisman@xxxxxxxxxxxxxxx> --- lib/igt_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 14e2701c3afd..ce07fcc1fc73 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -1864,7 +1864,7 @@ void igt_display_require_output_on_pipe(igt_display_t *display, enum pipe pipe) { igt_output_t *output; - igt_skip_on_f(igt_display_get_n_pipes(display) < pipe, + igt_skip_on_f(igt_display_get_n_pipes(display) <= pipe, "Pipe %s does not exist.\n", kmstest_pipe_name(pipe)); for_each_valid_output_on_pipe(display, pipe, output) -- 2.11.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx