From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794326601; b=VqY9L1o/2cXWk+8iGrcP2mJHyzDc12EKDLQLxK5faKQyrSlIp0Rf4UMU2xS5qk37qwe7 +uXppHYe1KxTHEPOhuLD1AMp/4jkIdj8DlFJ93VXrP8IVN4b+qiOYO5ag+vDMC2h5hKSo b/SVcSHpQq025L/X05dUMqoEdNw1l6oPkze5cg5urI65YWDe0sy/tem++o1hY02WcnLFH EpEIp9ooQPqD4mmvZQJNIrI4lp5LS1Bs3G5IVmeq8z4yCSsG0S7otgx8drCnX58kxK6r0 xXtYw5f7OheMLaSfYOCNsAgAvHeYrbZby0I12R/hrpWwe8y9/5eo+oiE2FiYzVJ0/ng== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794326601; h=Received:DKIM-Signature:From:Date:Subject:MIME-Version:Content-Type: Content-Transfer-Encoding:Message-Id:References:In-Reply-To:To:Cc: X-Mailer:X-Developer-Signature:X-Developer-Key; bh=bBg0p6MY69gwWMSsuXkmi1O0AL9b/cHh43WEUCk3kj4=; b=nwDGvkNlvEM9GnNQpC0MoY/PgNDlBXGR8YI8JNmnC6y9YKpmd3bS2/Y+E5pxxIxuOVY7 rMZ37YqPEyA7puRBWzwEqwLi4RTlr6QWHQk4fZivmyJzs7v/j7Hj/hplWRacuX1q7N9Po xd/4LV1+2YYmeyPgeahj3lpGURXe2BSpgj0oM4azeLe1PqGV8CpgiZPkJ/em/INCqL7Vu j673Z8RKswIbuEJ/aw1k460jp2WY2mlscyGmKhTqzEcRxwKGejOWs5YSQDnsCZ4zB5tjA v1f/2qNHz/1Sy9wpLnwJxsdeXTqkCREKAOb8/hhDjbOaZiNavqVIsRmRBdUZRedG+xg== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=R2q79cmA; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1DE9960A380E for ; Tue, 10 Nov 2026 16:03:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=R2q79cmA Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6ED721F008A4; Tue, 10 Nov 2026 16:03:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794326601; bh=bBg0p6MY69gwWMSsuXkmi1O0AL9b/cHh43WEUCk3kj4=; h=From:Date:Subject:References:In-Reply-To:To:Cc; b=R2q79cmASilHES/8mGprgYZRQMw/32tsW6m1vI0J9Ff1pkUlxqHAdB9J6wOLj6mEE FzKh2lR0wQeSVMnyckMgSDXcKDl14aXBhCdi0cyD08+lJ7J16kQwVDz5ydIBmCX6xA QkdKCfCOOU7pTiBXpSt8biBuuZA3QYnp42cbLLCrlprtdL9HIN6xkx2MzlplwKMTfq Xq+AAYTZhQa+L3QbcVslya0fWzN4hXS5LOoMlHZGbXeBaJb/GtFxSuqiiuN6iwR5hC +rmP16JsY+DDdl+iwqlPoWa4aajdXOyj4JtGvVNbr895HYP7WtJ1eYs+Tuq6mNgge7 W1k07Le/uDLPQ== From: Maxime Ripard Date: Tue, 10 Nov 2026 17:02:54 +0100 Subject: [PATCH v9 07/10] drm/vc4: Carry the HVS channel over to created CRTC states X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Message-Id: <20261110-drm-reset-state-flag-v9-7-ddb0e26324a0@kernel.org> References: <20261110-drm-reset-state-flag-v9-0-ddb0e26324a0@kernel.org> In-Reply-To: <20261110-drm-reset-state-flag-v9-0-ddb0e26324a0@kernel.org> To: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Louis Chauvet , Haneen Mohammed , Melissa Wen Cc: Daniel Stone , Harry Wentland , Jocelyn Falempe , =?utf-8?q?Jonas_=C3=85dahl?= , =?utf-8?q?Michel_D=C3=A4nzer?= , Pekka Paalanen , Sebastian Wick , Simon Ser , Victoria Brekenfeld , Xaver Hugl , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Maxime Ripard , Dave Stevenson , =?utf-8?q?Ma=C3=ADra_Canal?= , Raspberry Pi Kernel Maintenance X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=6786; i=mripard@kernel.org; h=from:subject:message-id; bh=MdgUI9oZhoGM9H2Ux74IeNHR8pguaAageXhvGfFYnu0=; b=owGbwMvMwCmsHn9OcpHtvjLG02pJDFmfHfSPr22Y83iRs+X3gMOWzp7sQlpP+PYxdhW0/JBpu GT3cXlEx1QWBmFOBlkxRZYnMmGnl7cvrnKwX/kDZg4rE8gQBi5OAZiI6h/GWpm/rELeC7XVSs02 mFguYT6guS+/QTSkr+/bJb/FIfUsylmtZbkbdp+5YuCzR3DlZVc7xirmjulzn5ywfum2ukxswzF ms8j2p/8FV5/+9/P2ngxuK1mvice/HH7s+v+YSl+ypUp582EA X-Developer-Key: i=mripard@kernel.org; a=openpgp; fpr=BE5675C37E818C8B5764241C254BCFC56BF6CE8D drm_atomic_commit_fill_with_defaults() builds its commits out of states created through atomic_create_state() instead of duplicating the current state. A created CRTC state comes with its assigned_channel left to VC4_HVS_CHANNEL_DISABLED, while a duplicated state carries the channel currently in use. vc4_pv_muxing_atomic_check() skips the CRTCs whose enablement doesn't change, so a CRTC that stays enabled with a created state keeps its channel unset, and vc4_core_clock_atomic_check() then indexes hvs_new_state->fifo_state[] with VC4_HVS_CHANNEL_DISABLED (-1U), far outside of the array. Copy the channel of the current state over in the skip branch, like a duplicated state would have carried it. This is a no-op for duplicated states, which already have the channel set. Add a KUnit test that enables an output, fills a state with defaults and re-applies the same configuration, and checks that the CRTC kept its channel. Signed-off-by: Maxime Ripard --- To: Dave Stevenson To: Maíra Canal Cc: Raspberry Pi Kernel Maintenance --- drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c | 105 +++++++++++++++++++++++++ drivers/gpu/drm/vc4/vc4_kms.c | 16 +++- 2 files changed, 119 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c b/drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c index 7a54f46acff9..c868d2cd1b3d 100644 --- a/drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c +++ b/drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c @@ -1119,14 +1119,119 @@ drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable_too_many_crtc_state(struct ku drm_modeset_drop_locks(&ctx); drm_modeset_acquire_fini(&ctx); } +/* + * Test that a CRTC that stays enabled with a state created through + * atomic_create_state(), like the ones drm_atomic_commit_fill_with_defaults() + * builds for a reset, keeps the HVS channel it was assigned: + * vc4_pv_muxing_atomic_check() skips the CRTCs whose enablement doesn't + * change, so their new state has to carry the channel over from the + * current state, the way a duplicated state would, for + * vc4_core_clock_atomic_check() to see a valid channel. + */ +static void +drm_test_vc5_pv_muxing_bugs_created_state_keeps_channel(struct kunit *test) +{ + struct drm_modeset_acquire_ctx ctx; + struct drm_atomic_commit *state; + struct vc4_crtc_state *new_vc4_crtc_state; + struct drm_device *drm; + struct vc4_dev *vc4; + unsigned int old_channel; + int ret; + + vc4 = vc5_mock_device(test); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, vc4); + + drm_modeset_acquire_init(&ctx, 0); + + drm = &vc4->base; +retry_first: + state = drm_kunit_helper_atomic_state_alloc(test, drm, &ctx); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state); + + ret = vc4_mock_atomic_add_output(test, state, VC4_ENCODER_TYPE_HDMI0); + if (ret == -EDEADLK) { + drm_atomic_commit_clear(state); + ret = drm_modeset_backoff(&ctx); + if (!ret) + goto retry_first; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + ret = drm_atomic_check_only(state); + if (ret == -EDEADLK) { + drm_atomic_commit_clear(state); + ret = drm_modeset_backoff(&ctx); + if (!ret) + goto retry_first; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + new_vc4_crtc_state = get_vc4_crtc_state_for_encoder(test, state, + VC4_ENCODER_TYPE_HDMI0); + KUNIT_ASSERT_NOT_NULL(test, new_vc4_crtc_state); + + old_channel = new_vc4_crtc_state->assigned_channel; + KUNIT_ASSERT_NE(test, old_channel, VC4_HVS_CHANNEL_DISABLED); + + /* Make the enabled state the current one */ + ret = drm_atomic_helper_swap_state(state, false); + KUNIT_ASSERT_EQ(test, ret, 0); + +retry_second: + state = drm_kunit_helper_atomic_state_alloc(test, drm, &ctx); + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, state); + + /* + * Fill the state with defaults, like a DRM_MODE_ATOMIC_RESET + * commit would, then re-apply the exact same configuration. + */ + ret = drm_atomic_commit_fill_with_defaults(NULL, state); + if (ret == -EDEADLK) { + drm_atomic_commit_clear(state); + ret = drm_modeset_backoff(&ctx); + if (!ret) + goto retry_second; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + ret = vc4_mock_atomic_add_output(test, state, VC4_ENCODER_TYPE_HDMI0); + if (ret == -EDEADLK) { + drm_atomic_commit_clear(state); + ret = drm_modeset_backoff(&ctx); + if (!ret) + goto retry_second; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + ret = drm_atomic_check_only(state); + if (ret == -EDEADLK) { + drm_atomic_commit_clear(state); + ret = drm_modeset_backoff(&ctx); + if (!ret) + goto retry_second; + } + KUNIT_ASSERT_EQ(test, ret, 0); + + new_vc4_crtc_state = get_vc4_crtc_state_for_encoder(test, state, + VC4_ENCODER_TYPE_HDMI0); + KUNIT_ASSERT_NOT_NULL(test, new_vc4_crtc_state); + + KUNIT_EXPECT_EQ(test, new_vc4_crtc_state->assigned_channel, old_channel); + + drm_modeset_drop_locks(&ctx); + drm_modeset_acquire_fini(&ctx); +} + static struct kunit_case vc5_pv_muxing_bugs_tests[] = { KUNIT_CASE(drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable), KUNIT_CASE(drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable_too_many_crtc_state), KUNIT_CASE(drm_test_vc5_pv_muxing_bugs_stable_fifo), + KUNIT_CASE(drm_test_vc5_pv_muxing_bugs_created_state_keeps_channel), {} }; static struct kunit_suite vc5_pv_muxing_bugs_test_suite = { .name = "vc5-pv-muxing-bugs", diff --git a/drivers/gpu/drm/vc4/vc4_kms.c b/drivers/gpu/drm/vc4/vc4_kms.c index ab551eb2da50..562f2d1f45a4 100644 --- a/drivers/gpu/drm/vc4/vc4_kms.c +++ b/drivers/gpu/drm/vc4/vc4_kms.c @@ -970,15 +970,27 @@ static int vc4_pv_muxing_atomic_check(struct drm_device *dev, drm_dbg(dev, "%s: Trying to find a channel.\n", crtc->name); /* Nothing to do here, let's skip it */ if (old_crtc_state->enable == new_crtc_state->enable) { - if (new_crtc_state->enable) + /* + * A CRTC that stays enabled keeps its channel. A + * duplicated state carries it over, but a state + * created through atomic_create_state() comes with + * the channel unset, so copy it from the current + * state to make sure the core clock check and the + * commit see the channel in use. + */ + if (new_crtc_state->enable) { + new_vc4_crtc_state->assigned_channel = + old_vc4_crtc_state->assigned_channel; + drm_dbg(dev, "%s: Already enabled, reusing channel %d.\n", crtc->name, new_vc4_crtc_state->assigned_channel); - else + } else { drm_dbg(dev, "%s: Disabled, ignoring.\n", crtc->name); + } continue; } /* Muxing will need to be modified, mark it as such */ -- 2.56.0