Skip to contentSkip to conversation

FAILED: patch "[PATCH] drm/amd/display: should support dmub hw lock on Replay" failed to apply to 6.1-stable tree

PATCH In review [email protected] 3 participants 5 messages Mar 24, 2025 – Mar 25, 2025

Subsystems (4)
lore
Share & tools
lore

[email protected]

author maintainer
#0 lore raw

The patch below does not apply to the 6.1-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.

To reproduce the conflict and resubmit, you may use the following commands:

git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-6.1.y
git checkout FETCH_HEAD
git cherry-pick -x bfeefe6ea5f18cabb8fda55364079573804623f9
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<[email protected]>' --in-reply-to '2025032457-occultist-maximum-38b6@gregkh' --subject-prefix 'PATCH 6.1.y' HEAD^..

Possible dependencies:

thanks,

greg k-h

------------------ original commit in Linus's tree ------------------

From bfeefe6ea5f18cabb8fda55364079573804623f9 Mon Sep 17 00:00:00 2001
From: Martin Tsai <[email protected]>
Date: Fri, 2 Feb 2024 14:39:29 +0800
Subject: [PATCH] drm/amd/display: should support dmub hw lock on Replay

[Why]
Without acquiring DMCUB hw lock, a race condition is caused with Panel Replay feature, which will trigger a hang. Indicate that a lock is necessary to prevent this when replay feature is enabled.

[How]
To allow dmub hw lock on Replay.

drivers/gpu/drm/amd/display/dc/dce/dmub_hw_lock_mgr.c +4 -0
index ba1fec3016d5..bf636b28e3e1 100644
@@ -65,5 +65,9 @@ bool should_use_dmub_lock(struct dc_link *link)
 {
 	if (link->psr_settings.psr_version == DC_PSR_VERSION_SU_1)
 		return true;
+
+	if (link->replay_settings.replay_feature_enabled)
+		return true;
+
 	return false;
 }
#1 lore raw

PATCH 1/1 drm/amd/display: should support dmub hw lock on Replay

From: Martin Tsai <[email protected]>

[Why]
Without acquiring DMCUB hw lock, a race condition is caused with Panel Replay feature, which will trigger a hang. Indicate that a lock is necessary to prevent this when replay feature is enabled.

[How]
To allow dmub hw lock on Replay.

(cherry picked from commit bfeefe6ea5f18cabb8fda55364079573804623f9)
---
 drivers/gpu/drm/amd/display/dc/dce/dmub_hw_lock_mgr.c | 3 +++
 1 file changed, 3 insertions(+)
drivers/gpu/drm/amd/display/dc/dce/dmub_hw_lock_mgr.c +3 -0
index 6e2fce329d73..7407891286cd 100644
@@ -69,6 +69,9 @@ bool should_use_dmub_lock(struct dc_link *link)
 	if (link->replay_settings.replay_feature_enabled)
 		return true;
 
+	if (link->replay_settings.replay_feature_enabled)
+		return true;
+
 	/* only use HW lock for PSR1 on single eDP */
 	if (link->psr_settings.psr_version == DC_PSR_VERSION_1) {
 		struct dc_link *edp_links[MAX_NUM_EDP];
Signature · 1 line
2.49.0
author maintainer
#2 lore raw

Re: [PATCH 6.1.y] drm/amd/display: should support dmub hw lock on Replay

Quoted 16 lines · Aurabindo Pillai

From: Martin Tsai <[email protected]>

[Why]
Without acquiring DMCUB hw lock, a race condition is caused with Panel Replay feature, which will trigger a hang. Indicate that a lock is necessary to prevent this when replay feature is enabled.

[How]
To allow dmub hw lock on Replay.

(cherry picked from commit bfeefe6ea5f18cabb8fda55364079573804623f9)
Quoted 3 lines
---
 drivers/gpu/drm/amd/display/dc/dce/dmub_hw_lock_mgr.c | 3 +++
 1 file changed, 3 insertions(+)

Does not apply to the tree at all :(

#3 lore raw

Re: [PATCH 6.1.y] drm/amd/display: should support dmub hw lock on Replay

[AMD Official Use Only - AMD Internal Distribution Only]

Quoted 1 line

Does not apply to the tree at all :(

Hi Greg,

Sorry about the error - I was on the wrong stable branch.

But please drop this patch for 6.1 stable, since this tree does not have replay support, so this bug fix isn't valid.

--
Regards,
Jay

maintainer
#4 lore raw

Re: [PATCH 6.1.y] drm/amd/display: should support dmub hw lock on Replay

In reply to Aurabindo Pillai

[ Sasha's backport helper bot ]

Hi,

Summary of potential issues:
⚠️ Found matching upstream commit but patch is missing proper reference to it

Found matching upstream commit: bfeefe6ea5f18cabb8fda55364079573804623f9

WARNING: Author mismatch between patch and found commit:
Backport author: Aurabindo Pillai<[email protected]> Commit author: Martin Tsai<[email protected]>

Status in newer kernel trees:
6.13.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Not found

Note: The patch differs from the upstream commit:
---

1:  bfeefe6ea5f18 < -:  ------------- drm/amd/display: should support dmub hw lock on Replay
-:  ------------- > 1:  344a09659766c Linux 6.1.131

---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-6.1.y        |  Success    |  Success   |