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
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
|
From ac56795420156559b5381fbf4ad1f607baca2935 Mon Sep 17 00:00:00 2001
From: Eric Anholt <eric@anholt.net>
Date: Mon, 19 Oct 2015 08:23:18 -0700
Subject: [PATCH 092/423] drm/vc4: bo cache locking fixes.
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/gpu/drm/vc4/vc4_bo.c | 32 ++++++++++++++++++--------------
drivers/gpu/drm/vc4/vc4_drv.h | 2 +-
2 files changed, 19 insertions(+), 15 deletions(-)
--- a/drivers/gpu/drm/vc4/vc4_bo.c
+++ b/drivers/gpu/drm/vc4/vc4_bo.c
@@ -112,14 +112,14 @@ void vc4_bo_cache_purge(struct drm_devic
{
struct vc4_dev *vc4 = to_vc4_dev(dev);
- spin_lock(&vc4->bo_lock);
+ mutex_lock(&vc4->bo_lock);
while (!list_empty(&vc4->bo_cache.time_list)) {
struct vc4_bo *bo = list_last_entry(&vc4->bo_cache.time_list,
struct vc4_bo, unref_head);
vc4_bo_remove_from_cache(bo);
vc4_bo_destroy(bo);
}
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
}
struct vc4_bo *vc4_bo_create(struct drm_device *dev, size_t unaligned_size)
@@ -134,18 +134,18 @@ struct vc4_bo *vc4_bo_create(struct drm_
return NULL;
/* First, try to get a vc4_bo from the kernel BO cache. */
- spin_lock(&vc4->bo_lock);
+ mutex_lock(&vc4->bo_lock);
if (page_index < vc4->bo_cache.size_list_size &&
!list_empty(&vc4->bo_cache.size_list[page_index])) {
struct vc4_bo *bo =
list_first_entry(&vc4->bo_cache.size_list[page_index],
struct vc4_bo, size_head);
vc4_bo_remove_from_cache(bo);
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
kref_init(&bo->base.base.refcount);
return bo;
}
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
/* Otherwise, make a new BO. */
for (pass = 0; ; pass++) {
@@ -215,7 +215,7 @@ vc4_bo_cache_free_old(struct drm_device
struct vc4_dev *vc4 = to_vc4_dev(dev);
unsigned long expire_time = jiffies - msecs_to_jiffies(1000);
- spin_lock(&vc4->bo_lock);
+ mutex_lock(&vc4->bo_lock);
while (!list_empty(&vc4->bo_cache.time_list)) {
struct vc4_bo *bo = list_last_entry(&vc4->bo_cache.time_list,
struct vc4_bo, unref_head);
@@ -223,14 +223,14 @@ vc4_bo_cache_free_old(struct drm_device
mod_timer(&vc4->bo_cache.time_timer,
round_jiffies_up(jiffies +
msecs_to_jiffies(1000)));
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
return;
}
vc4_bo_remove_from_cache(bo);
vc4_bo_destroy(bo);
}
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
}
/* Called on the last userspace/kernel unreference of the BO. Returns
@@ -248,21 +248,25 @@ void vc4_free_object(struct drm_gem_obje
/* If the object references someone else's memory, we can't cache it.
*/
if (gem_bo->import_attach) {
+ mutex_lock(&vc4->bo_lock);
vc4_bo_destroy(bo);
+ mutex_unlock(&vc4->bo_lock);
return;
}
/* Don't cache if it was publicly named. */
if (gem_bo->name) {
+ mutex_lock(&vc4->bo_lock);
vc4_bo_destroy(bo);
+ mutex_unlock(&vc4->bo_lock);
return;
}
- spin_lock(&vc4->bo_lock);
+ mutex_lock(&vc4->bo_lock);
cache_list = vc4_get_cache_list_for_size(dev, gem_bo->size);
if (!cache_list) {
vc4_bo_destroy(bo);
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
return;
}
@@ -278,7 +282,7 @@ void vc4_free_object(struct drm_gem_obje
vc4->bo_stats.num_cached++;
vc4->bo_stats.size_cached += gem_bo->size;
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
vc4_bo_cache_free_old(dev);
}
@@ -465,7 +469,7 @@ void vc4_bo_cache_init(struct drm_device
{
struct vc4_dev *vc4 = to_vc4_dev(dev);
- spin_lock_init(&vc4->bo_lock);
+ mutex_init(&vc4->bo_lock);
INIT_LIST_HEAD(&vc4->bo_cache.time_list);
@@ -498,9 +502,9 @@ int vc4_bo_stats_debugfs(struct seq_file
struct vc4_dev *vc4 = to_vc4_dev(dev);
struct vc4_bo_stats stats;
- spin_lock(&vc4->bo_lock);
+ mutex_lock(&vc4->bo_lock);
stats = vc4->bo_stats;
- spin_unlock(&vc4->bo_lock);
+ mutex_unlock(&vc4->bo_lock);
seq_printf(m, "num bos allocated: %d\n", stats.num_allocated);
seq_printf(m, "size bos allocated: %dkb\n", stats.size_allocated / 1024);
--- a/drivers/gpu/drm/vc4/vc4_drv.h
+++ b/drivers/gpu/drm/vc4/vc4_drv.h
@@ -49,7 +49,7 @@ struct vc4_dev {
} bo_stats;
/* Protects bo_cache and the BO stats. */
- spinlock_t bo_lock;
+ struct mutex bo_lock;
/* Sequence number for the last job queued in job_list.
* Starts at 0 (no jobs emitted).
|