1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
|
From 1e5c722d10011ef7d63469e1d3b6c904e6eb6502 Mon Sep 17 00:00:00 2001
From: Andrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Date: Wed, 21 Jun 2017 15:44:23 -0400
Subject: [PATCH 0538/4131] drm/amd/display: Refine globallock.
Switch to wait_for_completion_interruptible_timeout wait
since the lock is called from IOCTL context and can be
interrupted by a signal.
Global lock function might return EDEADLK or EINTR which
is not an error and just singals to user mode to restart
the call.
Signed-off-by: Andrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Reviewed-by: Harry Wentland <Harry.Wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
.../drm/amd/display/amdgpu_dm/amdgpu_dm_types.c | 35 +++++++++++++++-------
1 file changed, 24 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
index 5998830..2771258 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
@@ -2992,7 +2992,7 @@ static uint32_t remove_from_val_sets(
* Grabs all modesetting locks to serialize against any blocking commits,
* Waits for completion of all non blocking commits.
*/
-static void do_aquire_global_lock(
+static int do_aquire_global_lock(
struct drm_device *dev,
struct drm_atomic_state *state)
{
@@ -3004,7 +3004,9 @@ static void do_aquire_global_lock(
* ensure that when the framework release it the
* extra locks we are locking here will get released to
*/
- drm_modeset_lock_all_ctx(dev, state->acquire_ctx);
+ ret = drm_modeset_lock_all_ctx(dev, state->acquire_ctx);
+ if (ret)
+ return ret;
list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
spin_lock(&crtc->commit_lock);
@@ -3020,15 +3022,20 @@ static void do_aquire_global_lock(
/* Make sure all pending HW programming completed and
* page flips done
*/
- ret = wait_for_completion_timeout(&commit->hw_done,
- 10*HZ);
- ret = wait_for_completion_timeout(&commit->flip_done,
- 10*HZ);
+ ret = wait_for_completion_interruptible_timeout(&commit->hw_done, 10*HZ);
+
+ if (ret > 0)
+ ret = wait_for_completion_interruptible_timeout(
+ &commit->flip_done, 10*HZ);
+
if (ret == 0)
- DRM_ERROR("[CRTC:%d:%s] hw_done timed out\n",
- crtc->base.id, crtc->name);
+ DRM_ERROR("[CRTC:%d:%s] cleanup_done or flip_done "
+ "timed out\n", crtc->base.id, crtc->name);
+
drm_crtc_commit_put(commit);
}
+
+ return ret < 0 ? ret : 0;
}
int amdgpu_dm_atomic_check(struct drm_device *dev,
@@ -3295,7 +3302,7 @@ int amdgpu_dm_atomic_check(struct drm_device *dev,
* synchronization events.
*/
if (aquire_global_lock)
- do_aquire_global_lock(dev, state);
+ ret = do_aquire_global_lock(dev, state);
}
@@ -3311,8 +3318,14 @@ int amdgpu_dm_atomic_check(struct drm_device *dev,
for (i = 0; i < new_stream_count; i++)
dc_stream_release(new_streams[i]);
- if (ret != 0)
- DRM_ERROR("Atomic check failed.\n");
+ if (ret != 0) {
+ if (ret == -EDEADLK)
+ DRM_DEBUG_KMS("Atomic check stopped due to to deadlock, retrying.\n");
+ else if (ret == -EINTR || ret == -EAGAIN || ret == -ERESTARTSYS)
+ DRM_DEBUG_KMS("Atomic check stopped due to to signal, retrying.\n");
+ else
+ DRM_ERROR("Atomic check failed.\n");
+ }
return ret;
}
--
2.7.4
|