From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5C8DAC5DF70 for ; Sat, 21 Feb 2026 03:20:23 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 0F88510E109; Sat, 21 Feb 2026 03:20:23 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="k8ISk5AT"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.13]) by gabe.freedesktop.org (Postfix) with ESMTPS id EBAAF10E109 for ; Sat, 21 Feb 2026 03:20:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1771644022; x=1803180022; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=ylM7sJ6u472v46OWodCoXmRERhx9S2aH64KcRWcBQB8=; b=k8ISk5ATVRjYs5snsMT8K9kFzULWvVUfh0uYqfOnSASuSut9KqVbDBIu WHce8EWqT9oXDbzBNJl1q3oxmHMH9yd/zeLO+2MtyMzqtsi6NyYI9GGli EWZxQg65PYSzZFWhuPysPBfRY35DOlOhYAJwfPN8oKsy8HygGaiQ9zb1J lTsbJIVzk4zU9jLTRrZKcqFliSODjdCJGuRwoIyuRYXyTBxX1FduNvofy 3YqZeKGn3LyWphR5dSV/hADuxKxYQ/iK8lpTSGeixvqvSnDE5IzJhWdra xHVRINEV7ANu4jwmU88OZv7hP5WmWDWybdj4HZB24sbM+q7K5Q2mX/+XX g==; X-CSE-ConnectionGUID: 2qfa9DuJR+ujn839PSD3wA== X-CSE-MsgGUID: EoBjqOtIQ9CeJV+MdkrW2w== X-IronPort-AV: E=McAfee;i="6800,10657,11707"; a="75345753" X-IronPort-AV: E=Sophos;i="6.21,302,1763452800"; d="scan'208";a="75345753" Received: from orviesa001.jf.intel.com ([10.64.159.141]) by fmvoesa107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Feb 2026 19:20:22 -0800 X-CSE-ConnectionGUID: 0gY/TE/jRu2IGwwFb21cPg== X-CSE-MsgGUID: w8noIqPSR2G0CXbeo3aOIQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.21,302,1763452800"; d="scan'208";a="252720231" Received: from vpanait-mobl.ger.corp.intel.com (HELO localhost) ([10.245.245.139]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Feb 2026 19:20:20 -0800 From: Ville Syrjala To: igt-dev@lists.freedesktop.org Cc: Swati Sharma , Karthik B S Subject: [PATCH i-g-t v2 04/23] tests/kms_concurrent: Actually run the test over all connected crtcs Date: Sat, 21 Feb 2026 05:19:43 +0200 Message-ID: <20260221032003.30936-5-ville.syrjala@linux.intel.com> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260221032003.30936-1-ville.syrjala@linux.intel.com> References: <20260221032003.30936-1-ville.syrjala@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo, Finland Content-Transfer-Encoding: 8bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" From: Ville Syrjälä Currently kms_concurrent uses for_each_valid_output_on_pipe() with an uninitialized pipe (meaning it probably contains stack garbage). The actual intention seems to have been to use for_each_pipe_with_valid_output(). It's even mentioned in the offending commit message, but the actual code was never changed to match. We no longer have for_each_pipe_with_valid_output() so we have to switch over to for_each_crtc_with_valid_output(), which also means we end up doing the pipe->crtc conversion in one go. #include "scripts/iterators.cocci" @@ iterator name for_each_valid_output_on_pipe; iterator name for_each_crtc_with_valid_output; expression DISPLAY, OUTPUT; identifier PIPE; type T; @@ - T PIPE; + igt_crtc_t *crtc; <+... when != PIPE = ... - for_each_valid_output_on_pipe(DISPLAY, PIPE, OUTPUT) + for_each_crtc_with_valid_output(DISPLAY, crtc, OUTPUT) { <... ( - igt_crtc_for_pipe(..., PIPE) + crtc | - kmstest_pipe_name(PIPE) + igt_crtc_name(crtc) | - PIPE + crtc->pipe ) ...> } ...+> @@ identifier DISPLAY; @@ - igt_display_t *DISPLAY = ...; ... when != DISPLAY Cc: Swati Sharma Cc: Karthik B S Fixes: 6958319123d9 ("tests/kms_concurrent: remove pipe dependency in subtest name") Signed-off-by: Ville Syrjälä --- tests/kms_concurrent.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/kms_concurrent.c b/tests/kms_concurrent.c index bbc988aa2000..f31992a514a0 100644 --- a/tests/kms_concurrent.c +++ b/tests/kms_concurrent.c @@ -344,24 +344,24 @@ run_test(data_t *data, enum pipe pipe, igt_output_t *output) static void run_tests_for_pipe(data_t *data) { - igt_display_t *display = &data->display; igt_output_t *output; - enum pipe pipe; + igt_crtc_t *crtc; igt_describe("Test atomic mode setting concurrently with multiple planes and screen " "resolution."); igt_subtest_with_dynamic("multi-plane-atomic-lowres") { - for_each_valid_output_on_pipe(&data->display, pipe, output) { + for_each_crtc_with_valid_output(&data->display, crtc, output) { igt_display_reset(&data->display); igt_output_set_crtc(output, - igt_crtc_for_pipe(display, pipe)); + crtc); if (!intel_pipe_output_combo_valid(&data->display)) continue; - igt_require(igt_crtc_for_pipe(display, pipe)->n_planes > 0); - igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) - run_test(data, pipe, output); + igt_require(crtc->n_planes > 0); + igt_dynamic_f("pipe-%s-%s", igt_crtc_name(crtc), + igt_output_name(output)) + run_test(data, crtc->pipe, output); } } } -- 2.52.0