Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH] [i-g-t] tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs
@ 2023-06-13 12:46 Hersen Wu
  2023-06-13 13:36 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
                   ` (15 more replies)
  0 siblings, 16 replies; 22+ messages in thread
From: Hersen Wu @ 2023-06-13 12:46 UTC (permalink / raw)
  To: igt-dev, rodrigo.siqueira, aurabindo.pillai, alex.hung,
	stylon.wang, hamza.mahfooz
  Cc: Hersen Wu

Fix test crash by looping outputs using
for_each_connected_output(display, output).

When n_pipes > n_outputs, i >= n_outputs,
display->outputs[i]->config.connector within
for_each_pipe(display, i) are invalid.
Invalid connectors cause test crash.

Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
---
 tests/amdgpu/amd_vrr_range.c | 20 ++++++--------------
 1 file changed, 6 insertions(+), 14 deletions(-)

diff --git a/tests/amdgpu/amd_vrr_range.c b/tests/amdgpu/amd_vrr_range.c
index 3f8f23e9a..611f41f95 100644
--- a/tests/amdgpu/amd_vrr_range.c
+++ b/tests/amdgpu/amd_vrr_range.c
@@ -258,18 +258,14 @@ static void test_freesync_parsing_base(data_t *data, uint32_t test_flags)
 	const struct edid *edid;
 	range_t range, expected_range;
 	igt_output_t *output;
-	int i, j, test_conn_cnt = 0;
+	int j, test_conn_cnt = 0;
+	igt_display_t *display = &data->display;
 
 	test_init(data);
 
 	igt_amd_require_hpd(&data->display, data->fd);
 
-	for_each_pipe(&data->display, i) {
-		/* setup the output */
-		output = data->output[i];
-		if (!output || !igt_output_is_connected(output))
-			continue;
-
+	for_each_connected_output(display, output) {
 		/* find a test EDID */
 		j = find_test_edid_index(output->config.connector->connector_type);
 
@@ -335,18 +331,14 @@ static void test_freesync_range_base(data_t *data, uint32_t test_flags)
 {
 	range_t range;
 	igt_output_t *output;
-	int i, test_conn_cnt = 0;
+	int test_conn_cnt = 0;
+	igt_display_t *display = &data->display;
 
 	test_init(data);
 
 	igt_amd_require_hpd(&data->display, data->fd);
 
-	for_each_pipe(&data->display, i) {
-		/* setup the output */
-		output = data->output[i];
-		if (!output || !igt_output_is_connected(output))
-			continue;
-
+	for_each_connected_output(display, output) {
 		igt_debug_on_f(!has_vrr(output), "Requires output supports VRR\n");
 
 		if (!has_vrr(output)) {
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2023-06-17  7:05 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-13 12:46 [igt-dev] [PATCH] [i-g-t] tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs Hersen Wu
2023-06-13 13:36 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2023-06-13 14:08 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2023-06-13 16:27   ` Kamil Konieczny
2023-06-15 13:25     ` Yedireswarapu, SaiX Nandan
2023-06-13 17:22 ` [igt-dev] [PATCH] [i-g-t] " Alex Hung
2023-06-13 17:53   ` Wu, Hersen
2023-06-13 23:28 ` Alex Hung
2023-06-14 19:21 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs (rev2) Patchwork
2023-06-14 21:02 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2023-06-15 13:01 ` Patchwork
2023-06-15 13:18 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork
2023-06-15 15:11 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs (rev3) Patchwork
2023-06-15 18:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2023-06-15 19:06 ` [igt-dev] ✗ GitLab.Pipeline: warning for tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs (rev4) Patchwork
2023-06-15 19:39 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2023-06-16  8:52   ` Kamil Konieczny
2023-06-17  7:05     ` Yedireswarapu, SaiX Nandan
2023-06-16 12:41 ` Patchwork
2023-06-16 13:06 ` Patchwork
2023-06-16 13:40 ` [igt-dev] ✓ Fi.CI.IGT: success for tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs (rev3) Patchwork
2023-06-16 13:47 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/amdgpu/amd_vrr_range: fix test crash when n_pipes > n_outputs (rev4) Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox