Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1 | /* |
| 2 | * Copyright © 2008-2015 Intel Corporation |
| 3 | * |
| 4 | * Permission is hereby granted, free of charge, to any person obtaining a |
| 5 | * copy of this software and associated documentation files (the "Software"), |
| 6 | * to deal in the Software without restriction, including without limitation |
| 7 | * the rights to use, copy, modify, merge, publish, distribute, sublicense, |
| 8 | * and/or sell copies of the Software, and to permit persons to whom the |
| 9 | * Software is furnished to do so, subject to the following conditions: |
| 10 | * |
| 11 | * The above copyright notice and this permission notice (including the next |
| 12 | * paragraph) shall be included in all copies or substantial portions of the |
| 13 | * Software. |
| 14 | * |
| 15 | * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR |
| 16 | * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, |
| 17 | * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL |
| 18 | * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER |
| 19 | * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING |
| 20 | * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS |
| 21 | * IN THE SOFTWARE. |
| 22 | * |
| 23 | */ |
| 24 | |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 25 | #include <linux/prefetch.h> |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 26 | #include <linux/dma-fence-array.h> |
Ingo Molnar | e601757 | 2017-02-01 16:36:40 +0100 | [diff] [blame] | 27 | #include <linux/sched.h> |
| 28 | #include <linux/sched/clock.h> |
Ingo Molnar | f361bf4 | 2017-02-03 23:47:37 +0100 | [diff] [blame] | 29 | #include <linux/sched/signal.h> |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 30 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 31 | #include "i915_drv.h" |
| 32 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 33 | static const char *i915_fence_get_driver_name(struct dma_fence *fence) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 34 | { |
| 35 | return "i915"; |
| 36 | } |
| 37 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 38 | static const char *i915_fence_get_timeline_name(struct dma_fence *fence) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 39 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 40 | /* |
| 41 | * The timeline struct (as part of the ppgtt underneath a context) |
Chris Wilson | 05506b5 | 2017-03-30 12:16:14 +0100 | [diff] [blame] | 42 | * may be freed when the request is no longer in use by the GPU. |
| 43 | * We could extend the life of a context to beyond that of all |
| 44 | * fences, possibly keeping the hw resource around indefinitely, |
| 45 | * or we just give them a false name. Since |
| 46 | * dma_fence_ops.get_timeline_name is a debug feature, the occasional |
| 47 | * lie seems justifiable. |
| 48 | */ |
| 49 | if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags)) |
| 50 | return "signaled"; |
| 51 | |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 52 | return to_request(fence)->timeline->common->name; |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 53 | } |
| 54 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 55 | static bool i915_fence_signaled(struct dma_fence *fence) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 56 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 57 | return i915_request_completed(to_request(fence)); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 58 | } |
| 59 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 60 | static bool i915_fence_enable_signaling(struct dma_fence *fence) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 61 | { |
Chris Wilson | 6f9ec41 | 2018-03-08 14:07:32 +0000 | [diff] [blame] | 62 | return intel_engine_enable_signaling(to_request(fence), true); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 63 | } |
| 64 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 65 | static signed long i915_fence_wait(struct dma_fence *fence, |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 66 | bool interruptible, |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 67 | signed long timeout) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 68 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 69 | return i915_request_wait(to_request(fence), interruptible, timeout); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 70 | } |
| 71 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 72 | static void i915_fence_release(struct dma_fence *fence) |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 73 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 74 | struct i915_request *rq = to_request(fence); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 75 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 76 | /* |
| 77 | * The request is put onto a RCU freelist (i.e. the address |
Chris Wilson | fc15840 | 2016-11-25 13:17:18 +0000 | [diff] [blame] | 78 | * is immediately reused), mark the fences as being freed now. |
| 79 | * Otherwise the debugobjects for the fences are only marked as |
| 80 | * freed when the slab cache itself is freed, and so we would get |
| 81 | * caught trying to reuse dead objects. |
| 82 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 83 | i915_sw_fence_fini(&rq->submit); |
Chris Wilson | fc15840 | 2016-11-25 13:17:18 +0000 | [diff] [blame] | 84 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 85 | kmem_cache_free(rq->i915->requests, rq); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 86 | } |
| 87 | |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 88 | const struct dma_fence_ops i915_fence_ops = { |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 89 | .get_driver_name = i915_fence_get_driver_name, |
| 90 | .get_timeline_name = i915_fence_get_timeline_name, |
| 91 | .enable_signaling = i915_fence_enable_signaling, |
| 92 | .signaled = i915_fence_signaled, |
| 93 | .wait = i915_fence_wait, |
| 94 | .release = i915_fence_release, |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 95 | }; |
| 96 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 97 | static inline void |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 98 | i915_request_remove_from_client(struct i915_request *request) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 99 | { |
Chris Wilson | c8659ef | 2017-03-02 12:25:25 +0000 | [diff] [blame] | 100 | struct drm_i915_file_private *file_priv; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 101 | |
Chris Wilson | c8659ef | 2017-03-02 12:25:25 +0000 | [diff] [blame] | 102 | file_priv = request->file_priv; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 103 | if (!file_priv) |
| 104 | return; |
| 105 | |
| 106 | spin_lock(&file_priv->mm.lock); |
Chris Wilson | c8659ef | 2017-03-02 12:25:25 +0000 | [diff] [blame] | 107 | if (request->file_priv) { |
| 108 | list_del(&request->client_link); |
| 109 | request->file_priv = NULL; |
| 110 | } |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 111 | spin_unlock(&file_priv->mm.lock); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 112 | } |
| 113 | |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 114 | static struct i915_dependency * |
| 115 | i915_dependency_alloc(struct drm_i915_private *i915) |
| 116 | { |
| 117 | return kmem_cache_alloc(i915->dependencies, GFP_KERNEL); |
| 118 | } |
| 119 | |
| 120 | static void |
| 121 | i915_dependency_free(struct drm_i915_private *i915, |
| 122 | struct i915_dependency *dep) |
| 123 | { |
| 124 | kmem_cache_free(i915->dependencies, dep); |
| 125 | } |
| 126 | |
| 127 | static void |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 128 | __i915_sched_node_add_dependency(struct i915_sched_node *node, |
| 129 | struct i915_sched_node *signal, |
| 130 | struct i915_dependency *dep, |
| 131 | unsigned long flags) |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 132 | { |
Chris Wilson | 20311bd | 2016-11-14 20:41:03 +0000 | [diff] [blame] | 133 | INIT_LIST_HEAD(&dep->dfs_link); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 134 | list_add(&dep->wait_link, &signal->waiters_list); |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 135 | list_add(&dep->signal_link, &node->signalers_list); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 136 | dep->signaler = signal; |
| 137 | dep->flags = flags; |
| 138 | } |
| 139 | |
| 140 | static int |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 141 | i915_sched_node_add_dependency(struct drm_i915_private *i915, |
| 142 | struct i915_sched_node *node, |
| 143 | struct i915_sched_node *signal) |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 144 | { |
| 145 | struct i915_dependency *dep; |
| 146 | |
| 147 | dep = i915_dependency_alloc(i915); |
| 148 | if (!dep) |
| 149 | return -ENOMEM; |
| 150 | |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 151 | __i915_sched_node_add_dependency(node, signal, dep, |
| 152 | I915_DEPENDENCY_ALLOC); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 153 | return 0; |
| 154 | } |
| 155 | |
| 156 | static void |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 157 | i915_sched_node_fini(struct drm_i915_private *i915, |
| 158 | struct i915_sched_node *node) |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 159 | { |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 160 | struct i915_dependency *dep, *tmp; |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 161 | |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 162 | GEM_BUG_ON(!list_empty(&node->link)); |
Chris Wilson | 20311bd | 2016-11-14 20:41:03 +0000 | [diff] [blame] | 163 | |
Chris Wilson | 83cc84c | 2018-01-02 15:12:25 +0000 | [diff] [blame] | 164 | /* |
| 165 | * Everyone we depended upon (the fences we wait to be signaled) |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 166 | * should retire before us and remove themselves from our list. |
| 167 | * However, retirement is run independently on each timeline and |
| 168 | * so we may be called out-of-order. |
| 169 | */ |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 170 | list_for_each_entry_safe(dep, tmp, &node->signalers_list, signal_link) { |
| 171 | GEM_BUG_ON(!i915_sched_node_signaled(dep->signaler)); |
Chris Wilson | 83cc84c | 2018-01-02 15:12:25 +0000 | [diff] [blame] | 172 | GEM_BUG_ON(!list_empty(&dep->dfs_link)); |
| 173 | |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 174 | list_del(&dep->wait_link); |
| 175 | if (dep->flags & I915_DEPENDENCY_ALLOC) |
| 176 | i915_dependency_free(i915, dep); |
| 177 | } |
| 178 | |
| 179 | /* Remove ourselves from everyone who depends upon us */ |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 180 | list_for_each_entry_safe(dep, tmp, &node->waiters_list, wait_link) { |
| 181 | GEM_BUG_ON(dep->signaler != node); |
Chris Wilson | 83cc84c | 2018-01-02 15:12:25 +0000 | [diff] [blame] | 182 | GEM_BUG_ON(!list_empty(&dep->dfs_link)); |
| 183 | |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 184 | list_del(&dep->signal_link); |
| 185 | if (dep->flags & I915_DEPENDENCY_ALLOC) |
| 186 | i915_dependency_free(i915, dep); |
| 187 | } |
| 188 | } |
| 189 | |
| 190 | static void |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 191 | i915_sched_node_init(struct i915_sched_node *node) |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 192 | { |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 193 | INIT_LIST_HEAD(&node->signalers_list); |
| 194 | INIT_LIST_HEAD(&node->waiters_list); |
| 195 | INIT_LIST_HEAD(&node->link); |
Chris Wilson | b7268c5 | 2018-04-18 19:40:52 +0100 | [diff] [blame] | 196 | node->attr.priority = I915_PRIORITY_INVALID; |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 197 | } |
| 198 | |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 199 | static int reset_all_global_seqno(struct drm_i915_private *i915, u32 seqno) |
| 200 | { |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 201 | struct intel_engine_cs *engine; |
| 202 | enum intel_engine_id id; |
| 203 | int ret; |
| 204 | |
| 205 | /* Carefully retire all requests without writing to the rings */ |
| 206 | ret = i915_gem_wait_for_idle(i915, |
| 207 | I915_WAIT_INTERRUPTIBLE | |
| 208 | I915_WAIT_LOCKED); |
| 209 | if (ret) |
| 210 | return ret; |
| 211 | |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 212 | GEM_BUG_ON(i915->gt.active_requests); |
| 213 | |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 214 | /* If the seqno wraps around, we need to clear the breadcrumb rbtree */ |
| 215 | for_each_engine(engine, i915, id) { |
Chris Wilson | ae351be | 2017-03-30 15:50:41 +0100 | [diff] [blame] | 216 | struct i915_gem_timeline *timeline; |
| 217 | struct intel_timeline *tl = engine->timeline; |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 218 | |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 219 | GEM_TRACE("%s seqno %d (current %d) -> %d\n", |
| 220 | engine->name, |
| 221 | tl->seqno, |
| 222 | intel_engine_get_seqno(engine), |
| 223 | seqno); |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 224 | |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 225 | if (!i915_seqno_passed(seqno, tl->seqno)) { |
Chris Wilson | f41d19b | 2018-03-06 13:01:43 +0000 | [diff] [blame] | 226 | /* Flush any waiters before we reuse the seqno */ |
| 227 | intel_engine_disarm_breadcrumbs(engine); |
Chris Wilson | 93eef7d | 2018-03-06 13:01:42 +0000 | [diff] [blame] | 228 | GEM_BUG_ON(!list_empty(&engine->breadcrumbs.signals)); |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 229 | } |
| 230 | |
Chris Wilson | 4d53568 | 2017-07-21 13:32:26 +0100 | [diff] [blame] | 231 | /* Check we are idle before we fiddle with hw state! */ |
| 232 | GEM_BUG_ON(!intel_engine_is_idle(engine)); |
| 233 | GEM_BUG_ON(i915_gem_active_isset(&engine->timeline->last_request)); |
| 234 | |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 235 | /* Finally reset hw state */ |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 236 | intel_engine_init_global_seqno(engine, seqno); |
Chris Wilson | 2ca9faa | 2017-04-05 16:30:54 +0100 | [diff] [blame] | 237 | tl->seqno = seqno; |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 238 | |
Chris Wilson | ae351be | 2017-03-30 15:50:41 +0100 | [diff] [blame] | 239 | list_for_each_entry(timeline, &i915->gt.timelines, link) |
Chris Wilson | 7e8894e | 2017-05-03 10:39:22 +0100 | [diff] [blame] | 240 | memset(timeline->engine[id].global_sync, 0, |
| 241 | sizeof(timeline->engine[id].global_sync)); |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 242 | } |
| 243 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 244 | i915->gt.request_serial = seqno; |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 245 | return 0; |
| 246 | } |
| 247 | |
| 248 | int i915_gem_set_global_seqno(struct drm_device *dev, u32 seqno) |
| 249 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 250 | struct drm_i915_private *i915 = to_i915(dev); |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 251 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 252 | lockdep_assert_held(&i915->drm.struct_mutex); |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 253 | |
| 254 | if (seqno == 0) |
| 255 | return -EINVAL; |
| 256 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 257 | /* HWS page needs to be set less than what we will inject to ring */ |
| 258 | return reset_all_global_seqno(i915, seqno - 1); |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 259 | } |
| 260 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 261 | static int reserve_gt(struct drm_i915_private *i915) |
Chris Wilson | 636918f | 2017-08-17 15:47:19 +0100 | [diff] [blame] | 262 | { |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 263 | int ret; |
| 264 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 265 | /* |
| 266 | * Reservation is fine until we may need to wrap around |
| 267 | * |
| 268 | * By incrementing the serial for every request, we know that no |
| 269 | * individual engine may exceed that serial (as each is reset to 0 |
| 270 | * on any wrap). This protects even the most pessimistic of migrations |
| 271 | * of every request from all engines onto just one. |
| 272 | */ |
| 273 | while (unlikely(++i915->gt.request_serial == 0)) { |
Chris Wilson | 636918f | 2017-08-17 15:47:19 +0100 | [diff] [blame] | 274 | ret = reset_all_global_seqno(i915, 0); |
| 275 | if (ret) { |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 276 | i915->gt.request_serial--; |
Chris Wilson | 636918f | 2017-08-17 15:47:19 +0100 | [diff] [blame] | 277 | return ret; |
| 278 | } |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 279 | } |
| 280 | |
Chris Wilson | 636918f | 2017-08-17 15:47:19 +0100 | [diff] [blame] | 281 | if (!i915->gt.active_requests++) |
Chris Wilson | e4d2006 | 2018-04-06 16:51:44 +0100 | [diff] [blame] | 282 | i915_gem_unpark(i915); |
Chris Wilson | 636918f | 2017-08-17 15:47:19 +0100 | [diff] [blame] | 283 | |
Chris Wilson | 12d3173 | 2017-02-23 07:44:09 +0000 | [diff] [blame] | 284 | return 0; |
| 285 | } |
| 286 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 287 | static void unreserve_gt(struct drm_i915_private *i915) |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 288 | { |
Chris Wilson | e4d2006 | 2018-04-06 16:51:44 +0100 | [diff] [blame] | 289 | if (!--i915->gt.active_requests) |
| 290 | i915_gem_park(i915); |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 291 | } |
| 292 | |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 293 | void i915_gem_retire_noop(struct i915_gem_active *active, |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 294 | struct i915_request *request) |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 295 | { |
| 296 | /* Space left intentionally blank */ |
| 297 | } |
| 298 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 299 | static void advance_ring(struct i915_request *request) |
Chris Wilson | cbb60b4 | 2017-04-06 18:00:28 +0100 | [diff] [blame] | 300 | { |
| 301 | unsigned int tail; |
| 302 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 303 | /* |
| 304 | * We know the GPU must have read the request to have |
Chris Wilson | cbb60b4 | 2017-04-06 18:00:28 +0100 | [diff] [blame] | 305 | * sent us the seqno + interrupt, so use the position |
| 306 | * of tail of the request to update the last known position |
| 307 | * of the GPU head. |
| 308 | * |
| 309 | * Note this requires that we are always called in request |
| 310 | * completion order. |
| 311 | */ |
Chris Wilson | e6ba999 | 2017-04-25 14:00:49 +0100 | [diff] [blame] | 312 | if (list_is_last(&request->ring_link, &request->ring->request_list)) { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 313 | /* |
| 314 | * We may race here with execlists resubmitting this request |
Chris Wilson | e6ba999 | 2017-04-25 14:00:49 +0100 | [diff] [blame] | 315 | * as we retire it. The resubmission will move the ring->tail |
| 316 | * forwards (to request->wa_tail). We either read the |
| 317 | * current value that was written to hw, or the value that |
| 318 | * is just about to be. Either works, if we miss the last two |
| 319 | * noops - they are safe to be replayed on a reset. |
| 320 | */ |
Chris Wilson | 3662003 | 2018-03-07 13:42:23 +0000 | [diff] [blame] | 321 | tail = READ_ONCE(request->tail); |
Chris Wilson | e6ba999 | 2017-04-25 14:00:49 +0100 | [diff] [blame] | 322 | } else { |
Chris Wilson | cbb60b4 | 2017-04-06 18:00:28 +0100 | [diff] [blame] | 323 | tail = request->postfix; |
Chris Wilson | e6ba999 | 2017-04-25 14:00:49 +0100 | [diff] [blame] | 324 | } |
Chris Wilson | cbb60b4 | 2017-04-06 18:00:28 +0100 | [diff] [blame] | 325 | list_del(&request->ring_link); |
| 326 | |
| 327 | request->ring->head = tail; |
| 328 | } |
| 329 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 330 | static void free_capture_list(struct i915_request *request) |
Chris Wilson | b0fd47a | 2017-04-15 10:39:02 +0100 | [diff] [blame] | 331 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 332 | struct i915_capture_list *capture; |
Chris Wilson | b0fd47a | 2017-04-15 10:39:02 +0100 | [diff] [blame] | 333 | |
| 334 | capture = request->capture_list; |
| 335 | while (capture) { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 336 | struct i915_capture_list *next = capture->next; |
Chris Wilson | b0fd47a | 2017-04-15 10:39:02 +0100 | [diff] [blame] | 337 | |
| 338 | kfree(capture); |
| 339 | capture = next; |
| 340 | } |
| 341 | } |
| 342 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 343 | static void i915_request_retire(struct i915_request *request) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 344 | { |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 345 | struct intel_engine_cs *engine = request->engine; |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 346 | struct i915_gem_active *active, *next; |
| 347 | |
Tvrtko Ursulin | 0c5c7df | 2018-04-06 13:35:14 +0100 | [diff] [blame] | 348 | GEM_TRACE("%s fence %llx:%d, global=%d, current %d\n", |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 349 | engine->name, |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 350 | request->fence.context, request->fence.seqno, |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 351 | request->global_seqno, |
| 352 | intel_engine_get_seqno(engine)); |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 353 | |
Chris Wilson | 4c7d62c | 2016-10-28 13:58:32 +0100 | [diff] [blame] | 354 | lockdep_assert_held(&request->i915->drm.struct_mutex); |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 355 | GEM_BUG_ON(!i915_sw_fence_signaled(&request->submit)); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 356 | GEM_BUG_ON(!i915_request_completed(request)); |
Chris Wilson | 4302055 | 2016-11-15 16:46:20 +0000 | [diff] [blame] | 357 | GEM_BUG_ON(!request->i915->gt.active_requests); |
Chris Wilson | 4c7d62c | 2016-10-28 13:58:32 +0100 | [diff] [blame] | 358 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 359 | trace_i915_request_retire(request); |
Chris Wilson | 80b204b | 2016-10-28 13:58:58 +0100 | [diff] [blame] | 360 | |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 361 | spin_lock_irq(&engine->timeline->lock); |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 362 | list_del_init(&request->link); |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 363 | spin_unlock_irq(&engine->timeline->lock); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 364 | |
Chris Wilson | cbb60b4 | 2017-04-06 18:00:28 +0100 | [diff] [blame] | 365 | advance_ring(request); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 366 | |
Chris Wilson | b0fd47a | 2017-04-15 10:39:02 +0100 | [diff] [blame] | 367 | free_capture_list(request); |
| 368 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 369 | /* |
| 370 | * Walk through the active list, calling retire on each. This allows |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 371 | * objects to track their GPU activity and mark themselves as idle |
| 372 | * when their *last* active request is completed (updating state |
| 373 | * tracking lists for eviction, active references for GEM, etc). |
| 374 | * |
| 375 | * As the ->retire() may free the node, we decouple it first and |
| 376 | * pass along the auxiliary information (to avoid dereferencing |
| 377 | * the node after the callback). |
| 378 | */ |
| 379 | list_for_each_entry_safe(active, next, &request->active_list, link) { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 380 | /* |
| 381 | * In microbenchmarks or focusing upon time inside the kernel, |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 382 | * we may spend an inordinate amount of time simply handling |
| 383 | * the retirement of requests and processing their callbacks. |
| 384 | * Of which, this loop itself is particularly hot due to the |
| 385 | * cache misses when jumping around the list of i915_gem_active. |
| 386 | * So we try to keep this loop as streamlined as possible and |
| 387 | * also prefetch the next i915_gem_active to try and hide |
| 388 | * the likely cache miss. |
| 389 | */ |
| 390 | prefetchw(next); |
| 391 | |
| 392 | INIT_LIST_HEAD(&active->link); |
Chris Wilson | 0eafec6 | 2016-08-04 16:32:41 +0100 | [diff] [blame] | 393 | RCU_INIT_POINTER(active->request, NULL); |
Chris Wilson | fa545cb | 2016-08-04 07:52:35 +0100 | [diff] [blame] | 394 | |
| 395 | active->retire(active, request); |
| 396 | } |
| 397 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 398 | i915_request_remove_from_client(request); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 399 | |
Mika Kuoppala | e5e1fc4 | 2016-11-16 17:20:31 +0200 | [diff] [blame] | 400 | /* Retirement decays the ban score as it is a sign of ctx progress */ |
Chris Wilson | 77b25a9 | 2017-07-21 13:32:30 +0100 | [diff] [blame] | 401 | atomic_dec_if_positive(&request->ctx->ban_score); |
Mika Kuoppala | e5e1fc4 | 2016-11-16 17:20:31 +0200 | [diff] [blame] | 402 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 403 | /* |
| 404 | * The backing object for the context is done after switching to the |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 405 | * *next* context. Therefore we cannot retire the previous context until |
| 406 | * the next context has already started running. However, since we |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 407 | * cannot take the required locks at i915_request_submit() we |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 408 | * defer the unpinning of the active context to now, retirement of |
| 409 | * the subsequent request. |
| 410 | */ |
| 411 | if (engine->last_retired_context) |
Chris Wilson | ab82a06 | 2018-04-30 14:15:01 +0100 | [diff] [blame^] | 412 | intel_context_unpin(engine->last_retired_context, engine); |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 413 | engine->last_retired_context = request->ctx; |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 414 | |
Chris Wilson | 7b92c1b | 2017-06-28 13:35:48 +0100 | [diff] [blame] | 415 | spin_lock_irq(&request->lock); |
Chris Wilson | b7a3f33 | 2018-02-03 10:19:14 +0000 | [diff] [blame] | 416 | if (!test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &request->fence.flags)) |
| 417 | dma_fence_signal_locked(&request->fence); |
| 418 | if (test_bit(DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT, &request->fence.flags)) |
| 419 | intel_engine_cancel_signaling(request); |
Chris Wilson | 253a281 | 2018-02-06 14:31:37 +0000 | [diff] [blame] | 420 | if (request->waitboost) { |
| 421 | GEM_BUG_ON(!atomic_read(&request->i915->gt_pm.rps.num_waiters)); |
| 422 | atomic_dec(&request->i915->gt_pm.rps.num_waiters); |
| 423 | } |
Chris Wilson | 7b92c1b | 2017-06-28 13:35:48 +0100 | [diff] [blame] | 424 | spin_unlock_irq(&request->lock); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 425 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 426 | unreserve_gt(request->i915); |
| 427 | |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 428 | i915_sched_node_fini(request->i915, &request->sched); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 429 | i915_request_put(request); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 430 | } |
| 431 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 432 | void i915_request_retire_upto(struct i915_request *rq) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 433 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 434 | struct intel_engine_cs *engine = rq->engine; |
| 435 | struct i915_request *tmp; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 436 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 437 | lockdep_assert_held(&rq->i915->drm.struct_mutex); |
| 438 | GEM_BUG_ON(!i915_request_completed(rq)); |
Chris Wilson | 4ffd6e0 | 2016-11-25 13:17:15 +0000 | [diff] [blame] | 439 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 440 | if (list_empty(&rq->link)) |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 441 | return; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 442 | |
| 443 | do { |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 444 | tmp = list_first_entry(&engine->timeline->requests, |
Chris Wilson | efdf7c0 | 2016-08-04 07:52:33 +0100 | [diff] [blame] | 445 | typeof(*tmp), link); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 446 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 447 | i915_request_retire(tmp); |
| 448 | } while (tmp != rq); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 449 | } |
| 450 | |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 451 | static u32 timeline_get_seqno(struct intel_timeline *tl) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 452 | { |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 453 | return ++tl->seqno; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 454 | } |
| 455 | |
Chris Wilson | 4ccfee9 | 2018-03-22 13:10:34 +0000 | [diff] [blame] | 456 | static void move_to_timeline(struct i915_request *request, |
| 457 | struct intel_timeline *timeline) |
| 458 | { |
| 459 | GEM_BUG_ON(request->timeline == request->engine->timeline); |
| 460 | lockdep_assert_held(&request->engine->timeline->lock); |
| 461 | |
| 462 | spin_lock(&request->timeline->lock); |
| 463 | list_move_tail(&request->link, &timeline->requests); |
| 464 | spin_unlock(&request->timeline->lock); |
| 465 | } |
| 466 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 467 | void __i915_request_submit(struct i915_request *request) |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 468 | { |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 469 | struct intel_engine_cs *engine = request->engine; |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 470 | u32 seqno; |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 471 | |
Tvrtko Ursulin | 0c5c7df | 2018-04-06 13:35:14 +0100 | [diff] [blame] | 472 | GEM_TRACE("%s fence %llx:%d -> global=%d, current %d\n", |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 473 | engine->name, |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 474 | request->fence.context, request->fence.seqno, |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 475 | engine->timeline->seqno + 1, |
| 476 | intel_engine_get_seqno(engine)); |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 477 | |
Chris Wilson | e60a870 | 2017-03-02 11:51:30 +0000 | [diff] [blame] | 478 | GEM_BUG_ON(!irqs_disabled()); |
Chris Wilson | 6752041 | 2017-03-02 13:28:01 +0000 | [diff] [blame] | 479 | lockdep_assert_held(&engine->timeline->lock); |
Chris Wilson | e60a870 | 2017-03-02 11:51:30 +0000 | [diff] [blame] | 480 | |
Chris Wilson | 2d453c7 | 2017-12-22 14:19:59 +0000 | [diff] [blame] | 481 | GEM_BUG_ON(request->global_seqno); |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 482 | |
Chris Wilson | 4ccfee9 | 2018-03-22 13:10:34 +0000 | [diff] [blame] | 483 | seqno = timeline_get_seqno(engine->timeline); |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 484 | GEM_BUG_ON(!seqno); |
| 485 | GEM_BUG_ON(i915_seqno_passed(intel_engine_get_seqno(engine), seqno)); |
| 486 | |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 487 | /* We may be recursing from the signal callback of another i915 fence */ |
| 488 | spin_lock_nested(&request->lock, SINGLE_DEPTH_NESTING); |
| 489 | request->global_seqno = seqno; |
| 490 | if (test_bit(DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT, &request->fence.flags)) |
Chris Wilson | f7b02a5 | 2017-04-26 09:06:59 +0100 | [diff] [blame] | 491 | intel_engine_enable_signaling(request, false); |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 492 | spin_unlock(&request->lock); |
| 493 | |
Chris Wilson | caddfe7 | 2016-10-28 13:58:52 +0100 | [diff] [blame] | 494 | engine->emit_breadcrumb(request, |
| 495 | request->ring->vaddr + request->postfix); |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 496 | |
Chris Wilson | 4ccfee9 | 2018-03-22 13:10:34 +0000 | [diff] [blame] | 497 | /* Transfer from per-context onto the global per-engine timeline */ |
| 498 | move_to_timeline(request, engine->timeline); |
Chris Wilson | 80b204b | 2016-10-28 13:58:58 +0100 | [diff] [blame] | 499 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 500 | trace_i915_request_execute(request); |
Tvrtko Ursulin | 158863f | 2018-02-20 10:47:42 +0000 | [diff] [blame] | 501 | |
Chris Wilson | fe49789 | 2017-02-23 07:44:13 +0000 | [diff] [blame] | 502 | wake_up_all(&request->execute); |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 503 | } |
Chris Wilson | 23902e4 | 2016-11-14 20:40:58 +0000 | [diff] [blame] | 504 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 505 | void i915_request_submit(struct i915_request *request) |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 506 | { |
| 507 | struct intel_engine_cs *engine = request->engine; |
| 508 | unsigned long flags; |
| 509 | |
| 510 | /* Will be called from irq-context when using foreign fences. */ |
| 511 | spin_lock_irqsave(&engine->timeline->lock, flags); |
| 512 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 513 | __i915_request_submit(request); |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 514 | |
| 515 | spin_unlock_irqrestore(&engine->timeline->lock, flags); |
| 516 | } |
| 517 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 518 | void __i915_request_unsubmit(struct i915_request *request) |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 519 | { |
| 520 | struct intel_engine_cs *engine = request->engine; |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 521 | |
Tvrtko Ursulin | 0c5c7df | 2018-04-06 13:35:14 +0100 | [diff] [blame] | 522 | GEM_TRACE("%s fence %llx:%d <- global=%d, current %d\n", |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 523 | engine->name, |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 524 | request->fence.context, request->fence.seqno, |
Chris Wilson | e770276 | 2018-03-27 22:01:57 +0100 | [diff] [blame] | 525 | request->global_seqno, |
| 526 | intel_engine_get_seqno(engine)); |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 527 | |
Chris Wilson | e60a870 | 2017-03-02 11:51:30 +0000 | [diff] [blame] | 528 | GEM_BUG_ON(!irqs_disabled()); |
Chris Wilson | 6752041 | 2017-03-02 13:28:01 +0000 | [diff] [blame] | 529 | lockdep_assert_held(&engine->timeline->lock); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 530 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 531 | /* |
| 532 | * Only unwind in reverse order, required so that the per-context list |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 533 | * is kept in seqno/ring order. |
| 534 | */ |
Chris Wilson | 2d453c7 | 2017-12-22 14:19:59 +0000 | [diff] [blame] | 535 | GEM_BUG_ON(!request->global_seqno); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 536 | GEM_BUG_ON(request->global_seqno != engine->timeline->seqno); |
Chris Wilson | c7cc144 | 2018-01-29 09:49:12 +0000 | [diff] [blame] | 537 | GEM_BUG_ON(i915_seqno_passed(intel_engine_get_seqno(engine), |
| 538 | request->global_seqno)); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 539 | engine->timeline->seqno--; |
| 540 | |
| 541 | /* We may be recursing from the signal callback of another i915 fence */ |
| 542 | spin_lock_nested(&request->lock, SINGLE_DEPTH_NESTING); |
| 543 | request->global_seqno = 0; |
| 544 | if (test_bit(DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT, &request->fence.flags)) |
| 545 | intel_engine_cancel_signaling(request); |
| 546 | spin_unlock(&request->lock); |
| 547 | |
| 548 | /* Transfer back from the global per-engine timeline to per-context */ |
Chris Wilson | 4ccfee9 | 2018-03-22 13:10:34 +0000 | [diff] [blame] | 549 | move_to_timeline(request, request->timeline); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 550 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 551 | /* |
| 552 | * We don't need to wake_up any waiters on request->execute, they |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 553 | * will get woken by any other event or us re-adding this request |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 554 | * to the engine timeline (__i915_request_submit()). The waiters |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 555 | * should be quite adapt at finding that the request now has a new |
| 556 | * global_seqno to the one they went to sleep on. |
| 557 | */ |
| 558 | } |
| 559 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 560 | void i915_request_unsubmit(struct i915_request *request) |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 561 | { |
| 562 | struct intel_engine_cs *engine = request->engine; |
| 563 | unsigned long flags; |
| 564 | |
| 565 | /* Will be called from irq-context when using foreign fences. */ |
| 566 | spin_lock_irqsave(&engine->timeline->lock, flags); |
| 567 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 568 | __i915_request_unsubmit(request); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 569 | |
| 570 | spin_unlock_irqrestore(&engine->timeline->lock, flags); |
| 571 | } |
| 572 | |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 573 | static int __i915_sw_fence_call |
| 574 | submit_notify(struct i915_sw_fence *fence, enum i915_sw_fence_notify state) |
| 575 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 576 | struct i915_request *request = |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 577 | container_of(fence, typeof(*request), submit); |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 578 | |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 579 | switch (state) { |
| 580 | case FENCE_COMPLETE: |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 581 | trace_i915_request_submit(request); |
Daniel Vetter | af7a8ff | 2017-10-11 11:10:19 +0200 | [diff] [blame] | 582 | /* |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 583 | * We need to serialize use of the submit_request() callback |
| 584 | * with its hotplugging performed during an emergency |
| 585 | * i915_gem_set_wedged(). We use the RCU mechanism to mark the |
| 586 | * critical section in order to force i915_gem_set_wedged() to |
| 587 | * wait until the submit_request() is completed before |
| 588 | * proceeding. |
Daniel Vetter | af7a8ff | 2017-10-11 11:10:19 +0200 | [diff] [blame] | 589 | */ |
| 590 | rcu_read_lock(); |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 591 | request->engine->submit_request(request); |
Daniel Vetter | af7a8ff | 2017-10-11 11:10:19 +0200 | [diff] [blame] | 592 | rcu_read_unlock(); |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 593 | break; |
| 594 | |
| 595 | case FENCE_FREE: |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 596 | i915_request_put(request); |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 597 | break; |
Chris Wilson | d55ac5b | 2016-11-14 20:40:59 +0000 | [diff] [blame] | 598 | } |
Chris Wilson | 80b204b | 2016-10-28 13:58:58 +0100 | [diff] [blame] | 599 | |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 600 | return NOTIFY_DONE; |
| 601 | } |
| 602 | |
Chris Wilson | 8e63717 | 2016-08-02 22:50:26 +0100 | [diff] [blame] | 603 | /** |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 604 | * i915_request_alloc - allocate a request structure |
Chris Wilson | 8e63717 | 2016-08-02 22:50:26 +0100 | [diff] [blame] | 605 | * |
| 606 | * @engine: engine that we wish to issue the request on. |
| 607 | * @ctx: context that the request will be associated with. |
Chris Wilson | 8e63717 | 2016-08-02 22:50:26 +0100 | [diff] [blame] | 608 | * |
| 609 | * Returns a pointer to the allocated request if successful, |
| 610 | * or an error code if not. |
| 611 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 612 | struct i915_request * |
| 613 | i915_request_alloc(struct intel_engine_cs *engine, struct i915_gem_context *ctx) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 614 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 615 | struct drm_i915_private *i915 = engine->i915; |
| 616 | struct i915_request *rq; |
Chris Wilson | 266a240 | 2017-05-04 10:33:08 +0100 | [diff] [blame] | 617 | struct intel_ring *ring; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 618 | int ret; |
| 619 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 620 | lockdep_assert_held(&i915->drm.struct_mutex); |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 621 | |
Chris Wilson | e7af311 | 2017-10-03 21:34:48 +0100 | [diff] [blame] | 622 | /* |
| 623 | * Preempt contexts are reserved for exclusive use to inject a |
| 624 | * preemption context switch. They are never to be used for any trivial |
| 625 | * request! |
| 626 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 627 | GEM_BUG_ON(ctx == i915->preempt_context); |
Chris Wilson | e7af311 | 2017-10-03 21:34:48 +0100 | [diff] [blame] | 628 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 629 | /* |
| 630 | * ABI: Before userspace accesses the GPU (e.g. execbuffer), report |
Chris Wilson | 6ffb7d0 | 2017-01-14 16:23:33 +0000 | [diff] [blame] | 631 | * EIO if the GPU is already wedged. |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 632 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 633 | if (i915_terminally_wedged(&i915->gpu_error)) |
Chris Wilson | 6ffb7d0 | 2017-01-14 16:23:33 +0000 | [diff] [blame] | 634 | return ERR_PTR(-EIO); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 635 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 636 | /* |
| 637 | * Pinning the contexts may generate requests in order to acquire |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 638 | * GGTT space, so do this first before we reserve a seqno for |
| 639 | * ourselves. |
| 640 | */ |
Chris Wilson | ab82a06 | 2018-04-30 14:15:01 +0100 | [diff] [blame^] | 641 | ring = intel_context_pin(ctx, engine); |
Chris Wilson | 266a240 | 2017-05-04 10:33:08 +0100 | [diff] [blame] | 642 | if (IS_ERR(ring)) |
| 643 | return ERR_CAST(ring); |
| 644 | GEM_BUG_ON(!ring); |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 645 | |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 646 | ret = reserve_gt(i915); |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 647 | if (ret) |
| 648 | goto err_unpin; |
| 649 | |
Chris Wilson | 3fef5cd | 2017-11-20 10:20:02 +0000 | [diff] [blame] | 650 | ret = intel_ring_wait_for_space(ring, MIN_SPACE_FOR_ADD_REQUEST); |
| 651 | if (ret) |
| 652 | goto err_unreserve; |
| 653 | |
Chris Wilson | 9b5f4e5 | 2016-07-20 09:21:09 +0100 | [diff] [blame] | 654 | /* Move the oldest request to the slab-cache (if not in use!) */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 655 | rq = list_first_entry_or_null(&engine->timeline->requests, |
| 656 | typeof(*rq), link); |
| 657 | if (rq && i915_request_completed(rq)) |
| 658 | i915_request_retire(rq); |
Chris Wilson | 9b5f4e5 | 2016-07-20 09:21:09 +0100 | [diff] [blame] | 659 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 660 | /* |
| 661 | * Beware: Dragons be flying overhead. |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 662 | * |
| 663 | * We use RCU to look up requests in flight. The lookups may |
| 664 | * race with the request being allocated from the slab freelist. |
| 665 | * That is the request we are writing to here, may be in the process |
Chris Wilson | 1426f71 | 2016-08-09 17:03:22 +0100 | [diff] [blame] | 666 | * of being read by __i915_gem_active_get_rcu(). As such, |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 667 | * we have to be very careful when overwriting the contents. During |
| 668 | * the RCU lookup, we change chase the request->engine pointer, |
Chris Wilson | 65e4760 | 2016-10-28 13:58:49 +0100 | [diff] [blame] | 669 | * read the request->global_seqno and increment the reference count. |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 670 | * |
| 671 | * The reference count is incremented atomically. If it is zero, |
| 672 | * the lookup knows the request is unallocated and complete. Otherwise, |
| 673 | * it is either still in use, or has been reallocated and reset |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 674 | * with dma_fence_init(). This increment is safe for release as we |
| 675 | * check that the request we have a reference to and matches the active |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 676 | * request. |
| 677 | * |
| 678 | * Before we increment the refcount, we chase the request->engine |
| 679 | * pointer. We must not call kmem_cache_zalloc() or else we set |
| 680 | * that pointer to NULL and cause a crash during the lookup. If |
| 681 | * we see the request is completed (based on the value of the |
| 682 | * old engine and seqno), the lookup is complete and reports NULL. |
| 683 | * If we decide the request is not completed (new engine or seqno), |
| 684 | * then we grab a reference and double check that it is still the |
| 685 | * active request - which it won't be and restart the lookup. |
| 686 | * |
| 687 | * Do not use kmem_cache_zalloc() here! |
| 688 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 689 | rq = kmem_cache_alloc(i915->requests, |
| 690 | GFP_KERNEL | __GFP_RETRY_MAYFAIL | __GFP_NOWARN); |
| 691 | if (unlikely(!rq)) { |
Chris Wilson | 31c70f9 | 2017-12-12 18:06:52 +0000 | [diff] [blame] | 692 | /* Ratelimit ourselves to prevent oom from malicious clients */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 693 | ret = i915_gem_wait_for_idle(i915, |
Chris Wilson | 31c70f9 | 2017-12-12 18:06:52 +0000 | [diff] [blame] | 694 | I915_WAIT_LOCKED | |
| 695 | I915_WAIT_INTERRUPTIBLE); |
| 696 | if (ret) |
| 697 | goto err_unreserve; |
| 698 | |
Chris Wilson | f0111b0 | 2018-01-19 14:46:57 +0000 | [diff] [blame] | 699 | /* |
| 700 | * We've forced the client to stall and catch up with whatever |
| 701 | * backlog there might have been. As we are assuming that we |
| 702 | * caused the mempressure, now is an opportune time to |
| 703 | * recover as much memory from the request pool as is possible. |
| 704 | * Having already penalized the client to stall, we spend |
| 705 | * a little extra time to re-optimise page allocation. |
| 706 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 707 | kmem_cache_shrink(i915->requests); |
Chris Wilson | f0111b0 | 2018-01-19 14:46:57 +0000 | [diff] [blame] | 708 | rcu_barrier(); /* Recover the TYPESAFE_BY_RCU pages */ |
| 709 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 710 | rq = kmem_cache_alloc(i915->requests, GFP_KERNEL); |
| 711 | if (!rq) { |
Chris Wilson | 31c70f9 | 2017-12-12 18:06:52 +0000 | [diff] [blame] | 712 | ret = -ENOMEM; |
| 713 | goto err_unreserve; |
| 714 | } |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 715 | } |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 716 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 717 | rq->timeline = i915_gem_context_lookup_timeline(ctx, engine); |
| 718 | GEM_BUG_ON(rq->timeline == engine->timeline); |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 719 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 720 | spin_lock_init(&rq->lock); |
| 721 | dma_fence_init(&rq->fence, |
Chris Wilson | f54d186 | 2016-10-25 13:00:45 +0100 | [diff] [blame] | 722 | &i915_fence_ops, |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 723 | &rq->lock, |
| 724 | rq->timeline->fence_context, |
| 725 | timeline_get_seqno(rq->timeline)); |
Chris Wilson | 0476965 | 2016-07-20 09:21:11 +0100 | [diff] [blame] | 726 | |
Chris Wilson | 48bc2a4 | 2016-11-25 13:17:17 +0000 | [diff] [blame] | 727 | /* We bump the ref for the fence chain */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 728 | i915_sw_fence_init(&i915_request_get(rq)->submit, submit_notify); |
| 729 | init_waitqueue_head(&rq->execute); |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 730 | |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 731 | i915_sched_node_init(&rq->sched); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 732 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 733 | INIT_LIST_HEAD(&rq->active_list); |
| 734 | rq->i915 = i915; |
| 735 | rq->engine = engine; |
| 736 | rq->ctx = ctx; |
| 737 | rq->ring = ring; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 738 | |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 739 | /* No zalloc, must clear what we need by hand */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 740 | rq->global_seqno = 0; |
| 741 | rq->signaling.wait.seqno = 0; |
| 742 | rq->file_priv = NULL; |
| 743 | rq->batch = NULL; |
| 744 | rq->capture_list = NULL; |
| 745 | rq->waitboost = false; |
Chris Wilson | 5a198b8 | 2016-08-09 09:23:34 +0100 | [diff] [blame] | 746 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 747 | /* |
| 748 | * Reserve space in the ring buffer for all the commands required to |
| 749 | * eventually emit this request. This is to guarantee that the |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 750 | * i915_request_add() call can't fail. Note that the reserve may need |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 751 | * to be redone if the request is not actually submitted straight |
| 752 | * away, e.g. because a GPU scheduler has deferred it. |
| 753 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 754 | rq->reserved_space = MIN_SPACE_FOR_ADD_REQUEST; |
| 755 | GEM_BUG_ON(rq->reserved_space < engine->emit_breadcrumb_sz); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 756 | |
Chris Wilson | 2113184 | 2017-11-20 10:20:01 +0000 | [diff] [blame] | 757 | /* |
| 758 | * Record the position of the start of the request so that |
Chris Wilson | d045446 | 2016-08-15 10:48:40 +0100 | [diff] [blame] | 759 | * should we detect the updated seqno part-way through the |
| 760 | * GPU processing the request, we never over-estimate the |
| 761 | * position of the head. |
| 762 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 763 | rq->head = rq->ring->emit; |
Chris Wilson | d045446 | 2016-08-15 10:48:40 +0100 | [diff] [blame] | 764 | |
Chris Wilson | 2113184 | 2017-11-20 10:20:01 +0000 | [diff] [blame] | 765 | /* Unconditionally invalidate GPU caches and TLBs. */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 766 | ret = engine->emit_flush(rq, EMIT_INVALIDATE); |
Chris Wilson | 2113184 | 2017-11-20 10:20:01 +0000 | [diff] [blame] | 767 | if (ret) |
Chris Wilson | b1c24a6 | 2017-11-23 15:26:30 +0000 | [diff] [blame] | 768 | goto err_unwind; |
Chris Wilson | 2113184 | 2017-11-20 10:20:01 +0000 | [diff] [blame] | 769 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 770 | ret = engine->request_alloc(rq); |
Chris Wilson | b1c24a6 | 2017-11-23 15:26:30 +0000 | [diff] [blame] | 771 | if (ret) |
| 772 | goto err_unwind; |
Chris Wilson | 2113184 | 2017-11-20 10:20:01 +0000 | [diff] [blame] | 773 | |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 774 | /* Check that we didn't interrupt ourselves with a new request */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 775 | GEM_BUG_ON(rq->timeline->seqno != rq->fence.seqno); |
| 776 | return rq; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 777 | |
Chris Wilson | b1c24a6 | 2017-11-23 15:26:30 +0000 | [diff] [blame] | 778 | err_unwind: |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 779 | rq->ring->emit = rq->head; |
Chris Wilson | b1c24a6 | 2017-11-23 15:26:30 +0000 | [diff] [blame] | 780 | |
Chris Wilson | 1618bdb | 2016-11-25 13:17:16 +0000 | [diff] [blame] | 781 | /* Make sure we didn't add ourselves to external state before freeing */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 782 | GEM_BUG_ON(!list_empty(&rq->active_list)); |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 783 | GEM_BUG_ON(!list_empty(&rq->sched.signalers_list)); |
| 784 | GEM_BUG_ON(!list_empty(&rq->sched.waiters_list)); |
Chris Wilson | 1618bdb | 2016-11-25 13:17:16 +0000 | [diff] [blame] | 785 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 786 | kmem_cache_free(i915->requests, rq); |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 787 | err_unreserve: |
Chris Wilson | 52d7f16 | 2018-04-30 14:15:00 +0100 | [diff] [blame] | 788 | unreserve_gt(i915); |
Chris Wilson | e8a9c58 | 2016-12-18 15:37:20 +0000 | [diff] [blame] | 789 | err_unpin: |
Chris Wilson | ab82a06 | 2018-04-30 14:15:01 +0100 | [diff] [blame^] | 790 | intel_context_unpin(ctx, engine); |
Chris Wilson | 8e63717 | 2016-08-02 22:50:26 +0100 | [diff] [blame] | 791 | return ERR_PTR(ret); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 792 | } |
| 793 | |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 794 | static int |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 795 | i915_request_await_request(struct i915_request *to, struct i915_request *from) |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 796 | { |
Chris Wilson | 85e17f5 | 2016-10-28 13:58:53 +0100 | [diff] [blame] | 797 | int ret; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 798 | |
| 799 | GEM_BUG_ON(to == from); |
Chris Wilson | ceae14b | 2017-05-03 10:39:20 +0100 | [diff] [blame] | 800 | GEM_BUG_ON(to->timeline == from->timeline); |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 801 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 802 | if (i915_request_completed(from)) |
Chris Wilson | ade0b0c | 2017-04-22 09:15:37 +0100 | [diff] [blame] | 803 | return 0; |
| 804 | |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 805 | if (to->engine->schedule) { |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 806 | ret = i915_sched_node_add_dependency(to->i915, |
| 807 | &to->sched, |
| 808 | &from->sched); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 809 | if (ret < 0) |
| 810 | return ret; |
| 811 | } |
| 812 | |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 813 | if (to->engine == from->engine) { |
| 814 | ret = i915_sw_fence_await_sw_fence_gfp(&to->submit, |
| 815 | &from->submit, |
Chris Wilson | 2abe2f8 | 2017-12-12 18:06:51 +0000 | [diff] [blame] | 816 | I915_FENCE_GFP); |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 817 | return ret < 0 ? ret : 0; |
| 818 | } |
| 819 | |
Chris Wilson | 6b56708 | 2017-06-08 12:14:05 +0100 | [diff] [blame] | 820 | if (to->engine->semaphore.sync_to) { |
| 821 | u32 seqno; |
Chris Wilson | 65e4760 | 2016-10-28 13:58:49 +0100 | [diff] [blame] | 822 | |
Chris Wilson | 49f0859 | 2017-05-03 10:39:24 +0100 | [diff] [blame] | 823 | GEM_BUG_ON(!from->engine->semaphore.signal); |
| 824 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 825 | seqno = i915_request_global_seqno(from); |
Chris Wilson | 6b56708 | 2017-06-08 12:14:05 +0100 | [diff] [blame] | 826 | if (!seqno) |
| 827 | goto await_dma_fence; |
| 828 | |
Chris Wilson | fc9d4d2 | 2017-05-03 10:39:23 +0100 | [diff] [blame] | 829 | if (seqno <= to->timeline->global_sync[from->engine->id]) |
| 830 | return 0; |
| 831 | |
| 832 | trace_i915_gem_ring_sync_to(to, from); |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 833 | ret = to->engine->semaphore.sync_to(to, from); |
| 834 | if (ret) |
| 835 | return ret; |
Chris Wilson | fc9d4d2 | 2017-05-03 10:39:23 +0100 | [diff] [blame] | 836 | |
| 837 | to->timeline->global_sync[from->engine->id] = seqno; |
Chris Wilson | 6b56708 | 2017-06-08 12:14:05 +0100 | [diff] [blame] | 838 | return 0; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 839 | } |
| 840 | |
Chris Wilson | fc9d4d2 | 2017-05-03 10:39:23 +0100 | [diff] [blame] | 841 | await_dma_fence: |
| 842 | ret = i915_sw_fence_await_dma_fence(&to->submit, |
| 843 | &from->fence, 0, |
Chris Wilson | 2abe2f8 | 2017-12-12 18:06:51 +0000 | [diff] [blame] | 844 | I915_FENCE_GFP); |
Chris Wilson | fc9d4d2 | 2017-05-03 10:39:23 +0100 | [diff] [blame] | 845 | return ret < 0 ? ret : 0; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 846 | } |
| 847 | |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 848 | int |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 849 | i915_request_await_dma_fence(struct i915_request *rq, struct dma_fence *fence) |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 850 | { |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 851 | struct dma_fence **child = &fence; |
| 852 | unsigned int nchild = 1; |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 853 | int ret; |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 854 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 855 | /* |
| 856 | * Note that if the fence-array was created in signal-on-any mode, |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 857 | * we should *not* decompose it into its individual fences. However, |
| 858 | * we don't currently store which mode the fence-array is operating |
| 859 | * in. Fortunately, the only user of signal-on-any is private to |
| 860 | * amdgpu and we should not see any incoming fence-array from |
| 861 | * sync-file being in signal-on-any mode. |
| 862 | */ |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 863 | if (dma_fence_is_array(fence)) { |
| 864 | struct dma_fence_array *array = to_dma_fence_array(fence); |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 865 | |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 866 | child = array->fences; |
| 867 | nchild = array->num_fences; |
| 868 | GEM_BUG_ON(!nchild); |
| 869 | } |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 870 | |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 871 | do { |
| 872 | fence = *child++; |
| 873 | if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags)) |
| 874 | continue; |
| 875 | |
Chris Wilson | ceae14b | 2017-05-03 10:39:20 +0100 | [diff] [blame] | 876 | /* |
| 877 | * Requests on the same timeline are explicitly ordered, along |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 878 | * with their dependencies, by i915_request_add() which ensures |
Chris Wilson | ceae14b | 2017-05-03 10:39:20 +0100 | [diff] [blame] | 879 | * that requests are submitted in-order through each ring. |
| 880 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 881 | if (fence->context == rq->fence.context) |
Chris Wilson | ceae14b | 2017-05-03 10:39:20 +0100 | [diff] [blame] | 882 | continue; |
| 883 | |
Chris Wilson | 4797948 | 2017-05-03 10:39:21 +0100 | [diff] [blame] | 884 | /* Squash repeated waits to the same timelines */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 885 | if (fence->context != rq->i915->mm.unordered_timeline && |
| 886 | intel_timeline_sync_is_later(rq->timeline, fence)) |
Chris Wilson | 4797948 | 2017-05-03 10:39:21 +0100 | [diff] [blame] | 887 | continue; |
| 888 | |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 889 | if (dma_fence_is_i915(fence)) |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 890 | ret = i915_request_await_request(rq, to_request(fence)); |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 891 | else |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 892 | ret = i915_sw_fence_await_dma_fence(&rq->submit, fence, |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 893 | I915_FENCE_TIMEOUT, |
Chris Wilson | 2abe2f8 | 2017-12-12 18:06:51 +0000 | [diff] [blame] | 894 | I915_FENCE_GFP); |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 895 | if (ret < 0) |
| 896 | return ret; |
Chris Wilson | 4797948 | 2017-05-03 10:39:21 +0100 | [diff] [blame] | 897 | |
| 898 | /* Record the latest fence used against each timeline */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 899 | if (fence->context != rq->i915->mm.unordered_timeline) |
| 900 | intel_timeline_sync_set(rq->timeline, fence); |
Chris Wilson | 29ef3fa | 2017-05-03 10:39:19 +0100 | [diff] [blame] | 901 | } while (--nchild); |
Chris Wilson | b52992c | 2016-10-28 13:58:24 +0100 | [diff] [blame] | 902 | |
| 903 | return 0; |
| 904 | } |
| 905 | |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 906 | /** |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 907 | * i915_request_await_object - set this request to (async) wait upon a bo |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 908 | * @to: request we are wishing to use |
| 909 | * @obj: object which may be in use on another ring. |
Chris Wilson | d880212 | 2018-02-08 11:14:53 +0000 | [diff] [blame] | 910 | * @write: whether the wait is on behalf of a writer |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 911 | * |
| 912 | * This code is meant to abstract object synchronization with the GPU. |
| 913 | * Conceptually we serialise writes between engines inside the GPU. |
| 914 | * We only allow one engine to write into a buffer at any time, but |
| 915 | * multiple readers. To ensure each has a coherent view of memory, we must: |
| 916 | * |
| 917 | * - If there is an outstanding write request to the object, the new |
| 918 | * request must wait for it to complete (either CPU or in hw, requests |
| 919 | * on the same ring will be naturally ordered). |
| 920 | * |
| 921 | * - If we are a write request (pending_write_domain is set), the new |
| 922 | * request must wait for outstanding read requests to complete. |
| 923 | * |
| 924 | * Returns 0 if successful, else propagates up the lower layer error. |
| 925 | */ |
| 926 | int |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 927 | i915_request_await_object(struct i915_request *to, |
| 928 | struct drm_i915_gem_object *obj, |
| 929 | bool write) |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 930 | { |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 931 | struct dma_fence *excl; |
| 932 | int ret = 0; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 933 | |
| 934 | if (write) { |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 935 | struct dma_fence **shared; |
| 936 | unsigned int count, i; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 937 | |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 938 | ret = reservation_object_get_fences_rcu(obj->resv, |
| 939 | &excl, &count, &shared); |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 940 | if (ret) |
| 941 | return ret; |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 942 | |
| 943 | for (i = 0; i < count; i++) { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 944 | ret = i915_request_await_dma_fence(to, shared[i]); |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 945 | if (ret) |
| 946 | break; |
| 947 | |
| 948 | dma_fence_put(shared[i]); |
| 949 | } |
| 950 | |
| 951 | for (; i < count; i++) |
| 952 | dma_fence_put(shared[i]); |
| 953 | kfree(shared); |
| 954 | } else { |
| 955 | excl = reservation_object_get_excl_rcu(obj->resv); |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 956 | } |
| 957 | |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 958 | if (excl) { |
| 959 | if (ret == 0) |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 960 | ret = i915_request_await_dma_fence(to, excl); |
Chris Wilson | d07f0e5 | 2016-10-28 13:58:44 +0100 | [diff] [blame] | 961 | |
| 962 | dma_fence_put(excl); |
| 963 | } |
| 964 | |
| 965 | return ret; |
Chris Wilson | a2bc469 | 2016-09-09 14:11:56 +0100 | [diff] [blame] | 966 | } |
| 967 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 968 | /* |
| 969 | * NB: This function is not allowed to fail. Doing so would mean the the |
| 970 | * request is not being tracked for completion but the work itself is |
| 971 | * going to happen on the hardware. This would be a Bad Thing(tm). |
| 972 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 973 | void __i915_request_add(struct i915_request *request, bool flush_caches) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 974 | { |
Chris Wilson | 95b2ab5 | 2016-08-15 10:48:46 +0100 | [diff] [blame] | 975 | struct intel_engine_cs *engine = request->engine; |
| 976 | struct intel_ring *ring = request->ring; |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 977 | struct intel_timeline *timeline = request->timeline; |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 978 | struct i915_request *prev; |
Tvrtko Ursulin | 73dec95 | 2017-02-14 11:32:42 +0000 | [diff] [blame] | 979 | u32 *cs; |
Chris Wilson | caddfe7 | 2016-10-28 13:58:52 +0100 | [diff] [blame] | 980 | int err; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 981 | |
Chris Wilson | d9b13c4 | 2018-03-15 13:14:50 +0000 | [diff] [blame] | 982 | GEM_TRACE("%s fence %llx:%d\n", |
| 983 | engine->name, request->fence.context, request->fence.seqno); |
| 984 | |
Chris Wilson | 4c7d62c | 2016-10-28 13:58:32 +0100 | [diff] [blame] | 985 | lockdep_assert_held(&request->i915->drm.struct_mutex); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 986 | trace_i915_request_add(request); |
Chris Wilson | 0f25dff | 2016-09-09 14:11:55 +0100 | [diff] [blame] | 987 | |
Chris Wilson | 8ac71d1 | 2018-02-07 08:43:50 +0000 | [diff] [blame] | 988 | /* |
| 989 | * Make sure that no request gazumped us - if it was allocated after |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 990 | * our i915_request_alloc() and called __i915_request_add() before |
Chris Wilson | c781c97 | 2017-01-11 14:08:58 +0000 | [diff] [blame] | 991 | * us, the timeline will hold its seqno which is later than ours. |
| 992 | */ |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 993 | GEM_BUG_ON(timeline->seqno != request->fence.seqno); |
Chris Wilson | c781c97 | 2017-01-11 14:08:58 +0000 | [diff] [blame] | 994 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 995 | /* |
| 996 | * To ensure that this call will not fail, space for its emissions |
| 997 | * should already have been reserved in the ring buffer. Let the ring |
| 998 | * know that it is time to use that space up. |
| 999 | */ |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1000 | request->reserved_space = 0; |
| 1001 | |
| 1002 | /* |
| 1003 | * Emit any outstanding flushes - execbuf can fail to emit the flush |
| 1004 | * after having emitted the batchbuffer command. Hence we need to fix |
| 1005 | * things up similar to emitting the lazy request. The difference here |
| 1006 | * is that the flush _must_ happen before the next request, no matter |
| 1007 | * what. |
| 1008 | */ |
| 1009 | if (flush_caches) { |
Chris Wilson | caddfe7 | 2016-10-28 13:58:52 +0100 | [diff] [blame] | 1010 | err = engine->emit_flush(request, EMIT_FLUSH); |
Chris Wilson | c7fe7d2 | 2016-08-02 22:50:24 +0100 | [diff] [blame] | 1011 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1012 | /* Not allowed to fail! */ |
Chris Wilson | caddfe7 | 2016-10-28 13:58:52 +0100 | [diff] [blame] | 1013 | WARN(err, "engine->emit_flush() failed: %d!\n", err); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1014 | } |
| 1015 | |
Chris Wilson | 8ac71d1 | 2018-02-07 08:43:50 +0000 | [diff] [blame] | 1016 | /* |
| 1017 | * Record the position of the start of the breadcrumb so that |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1018 | * should we detect the updated seqno part-way through the |
| 1019 | * GPU processing the request, we never over-estimate the |
Chris Wilson | d045446 | 2016-08-15 10:48:40 +0100 | [diff] [blame] | 1020 | * position of the ring's HEAD. |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1021 | */ |
Tvrtko Ursulin | 73dec95 | 2017-02-14 11:32:42 +0000 | [diff] [blame] | 1022 | cs = intel_ring_begin(request, engine->emit_breadcrumb_sz); |
| 1023 | GEM_BUG_ON(IS_ERR(cs)); |
| 1024 | request->postfix = intel_ring_offset(request, cs); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1025 | |
Chris Wilson | 8ac71d1 | 2018-02-07 08:43:50 +0000 | [diff] [blame] | 1026 | /* |
| 1027 | * Seal the request and mark it as pending execution. Note that |
Chris Wilson | 0f25dff | 2016-09-09 14:11:55 +0100 | [diff] [blame] | 1028 | * we may inspect this state, without holding any locks, during |
| 1029 | * hangcheck. Hence we apply the barrier to ensure that we do not |
| 1030 | * see a more recent value in the hws than we are tracking. |
| 1031 | */ |
Chris Wilson | 0a046a0 | 2016-09-09 14:12:00 +0100 | [diff] [blame] | 1032 | |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 1033 | prev = i915_gem_active_raw(&timeline->last_request, |
Chris Wilson | 0a046a0 | 2016-09-09 14:12:00 +0100 | [diff] [blame] | 1034 | &request->i915->drm.struct_mutex); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1035 | if (prev && !i915_request_completed(prev)) { |
Chris Wilson | 0a046a0 | 2016-09-09 14:12:00 +0100 | [diff] [blame] | 1036 | i915_sw_fence_await_sw_fence(&request->submit, &prev->submit, |
| 1037 | &request->submitq); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 1038 | if (engine->schedule) |
Chris Wilson | 0c7112a | 2018-04-18 19:40:51 +0100 | [diff] [blame] | 1039 | __i915_sched_node_add_dependency(&request->sched, |
| 1040 | &prev->sched, |
| 1041 | &request->dep, |
| 1042 | 0); |
Chris Wilson | 52e5420 | 2016-11-14 20:41:02 +0000 | [diff] [blame] | 1043 | } |
Chris Wilson | 0a046a0 | 2016-09-09 14:12:00 +0100 | [diff] [blame] | 1044 | |
Chris Wilson | 80b204b | 2016-10-28 13:58:58 +0100 | [diff] [blame] | 1045 | spin_lock_irq(&timeline->lock); |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 1046 | list_add_tail(&request->link, &timeline->requests); |
Chris Wilson | 80b204b | 2016-10-28 13:58:58 +0100 | [diff] [blame] | 1047 | spin_unlock_irq(&timeline->lock); |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 1048 | |
Chris Wilson | 9b6586a | 2017-02-23 07:44:08 +0000 | [diff] [blame] | 1049 | GEM_BUG_ON(timeline->seqno != request->fence.seqno); |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 1050 | i915_gem_active_set(&timeline->last_request, request); |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 1051 | |
Chris Wilson | 0f25dff | 2016-09-09 14:11:55 +0100 | [diff] [blame] | 1052 | list_add_tail(&request->ring_link, &ring->request_list); |
Chris Wilson | f2d1329 | 2016-10-28 13:58:57 +0100 | [diff] [blame] | 1053 | request->emitted_jiffies = jiffies; |
Chris Wilson | 0f25dff | 2016-09-09 14:11:55 +0100 | [diff] [blame] | 1054 | |
Chris Wilson | 8ac71d1 | 2018-02-07 08:43:50 +0000 | [diff] [blame] | 1055 | /* |
| 1056 | * Let the backend know a new request has arrived that may need |
Chris Wilson | 0de9136 | 2016-11-14 20:41:01 +0000 | [diff] [blame] | 1057 | * to adjust the existing execution schedule due to a high priority |
| 1058 | * request - i.e. we may want to preempt the current request in order |
| 1059 | * to run a high priority dependency chain *before* we can execute this |
| 1060 | * request. |
| 1061 | * |
| 1062 | * This is called before the request is ready to run so that we can |
| 1063 | * decide whether to preempt the entire chain so that it is ready to |
| 1064 | * run at the earliest possible convenience. |
| 1065 | */ |
Chris Wilson | 47650db | 2018-03-07 13:42:25 +0000 | [diff] [blame] | 1066 | rcu_read_lock(); |
Chris Wilson | 0de9136 | 2016-11-14 20:41:01 +0000 | [diff] [blame] | 1067 | if (engine->schedule) |
Chris Wilson | b7268c5 | 2018-04-18 19:40:52 +0100 | [diff] [blame] | 1068 | engine->schedule(request, &request->ctx->sched); |
Chris Wilson | 47650db | 2018-03-07 13:42:25 +0000 | [diff] [blame] | 1069 | rcu_read_unlock(); |
Chris Wilson | 0de9136 | 2016-11-14 20:41:01 +0000 | [diff] [blame] | 1070 | |
Chris Wilson | 5590af3 | 2016-09-09 14:11:54 +0100 | [diff] [blame] | 1071 | local_bh_disable(); |
| 1072 | i915_sw_fence_commit(&request->submit); |
| 1073 | local_bh_enable(); /* Kick the execlists tasklet if just scheduled */ |
Chris Wilson | c22b355 | 2018-02-07 08:43:49 +0000 | [diff] [blame] | 1074 | |
| 1075 | /* |
| 1076 | * In typical scenarios, we do not expect the previous request on |
| 1077 | * the timeline to be still tracked by timeline->last_request if it |
| 1078 | * has been completed. If the completed request is still here, that |
| 1079 | * implies that request retirement is a long way behind submission, |
| 1080 | * suggesting that we haven't been retiring frequently enough from |
| 1081 | * the combination of retire-before-alloc, waiters and the background |
| 1082 | * retirement worker. So if the last request on this timeline was |
| 1083 | * already completed, do a catch up pass, flushing the retirement queue |
| 1084 | * up to this client. Since we have now moved the heaviest operations |
| 1085 | * during retirement onto secondary workers, such as freeing objects |
| 1086 | * or contexts, retiring a bunch of requests is mostly list management |
| 1087 | * (and cache misses), and so we should not be overly penalizing this |
| 1088 | * client by performing excess work, though we may still performing |
| 1089 | * work on behalf of others -- but instead we should benefit from |
| 1090 | * improved resource management. (Well, that's the theory at least.) |
| 1091 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1092 | if (prev && i915_request_completed(prev)) |
| 1093 | i915_request_retire_upto(prev); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1094 | } |
| 1095 | |
| 1096 | static unsigned long local_clock_us(unsigned int *cpu) |
| 1097 | { |
| 1098 | unsigned long t; |
| 1099 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1100 | /* |
| 1101 | * Cheaply and approximately convert from nanoseconds to microseconds. |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1102 | * The result and subsequent calculations are also defined in the same |
| 1103 | * approximate microseconds units. The principal source of timing |
| 1104 | * error here is from the simple truncation. |
| 1105 | * |
| 1106 | * Note that local_clock() is only defined wrt to the current CPU; |
| 1107 | * the comparisons are no longer valid if we switch CPUs. Instead of |
| 1108 | * blocking preemption for the entire busywait, we can detect the CPU |
| 1109 | * switch and use that as indicator of system load and a reason to |
| 1110 | * stop busywaiting, see busywait_stop(). |
| 1111 | */ |
| 1112 | *cpu = get_cpu(); |
| 1113 | t = local_clock() >> 10; |
| 1114 | put_cpu(); |
| 1115 | |
| 1116 | return t; |
| 1117 | } |
| 1118 | |
| 1119 | static bool busywait_stop(unsigned long timeout, unsigned int cpu) |
| 1120 | { |
| 1121 | unsigned int this_cpu; |
| 1122 | |
| 1123 | if (time_after(local_clock_us(&this_cpu), timeout)) |
| 1124 | return true; |
| 1125 | |
| 1126 | return this_cpu != cpu; |
| 1127 | } |
| 1128 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1129 | static bool __i915_spin_request(const struct i915_request *rq, |
Chris Wilson | b2f2f0f | 2017-09-22 13:03:33 +0100 | [diff] [blame] | 1130 | u32 seqno, int state, unsigned long timeout_us) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1131 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1132 | struct intel_engine_cs *engine = rq->engine; |
Chris Wilson | c33ed06 | 2017-02-17 15:13:01 +0000 | [diff] [blame] | 1133 | unsigned int irq, cpu; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1134 | |
Chris Wilson | b2f2f0f | 2017-09-22 13:03:33 +0100 | [diff] [blame] | 1135 | GEM_BUG_ON(!seqno); |
| 1136 | |
| 1137 | /* |
| 1138 | * Only wait for the request if we know it is likely to complete. |
| 1139 | * |
| 1140 | * We don't track the timestamps around requests, nor the average |
| 1141 | * request length, so we do not have a good indicator that this |
| 1142 | * request will complete within the timeout. What we do know is the |
| 1143 | * order in which requests are executed by the engine and so we can |
| 1144 | * tell if the request has started. If the request hasn't started yet, |
| 1145 | * it is a fair assumption that it will not complete within our |
| 1146 | * relatively short timeout. |
| 1147 | */ |
| 1148 | if (!i915_seqno_passed(intel_engine_get_seqno(engine), seqno - 1)) |
| 1149 | return false; |
| 1150 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1151 | /* |
| 1152 | * When waiting for high frequency requests, e.g. during synchronous |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1153 | * rendering split between the CPU and GPU, the finite amount of time |
| 1154 | * required to set up the irq and wait upon it limits the response |
| 1155 | * rate. By busywaiting on the request completion for a short while we |
| 1156 | * can service the high frequency waits as quick as possible. However, |
| 1157 | * if it is a slow request, we want to sleep as quickly as possible. |
| 1158 | * The tradeoff between waiting and sleeping is roughly the time it |
| 1159 | * takes to sleep on a request, on the order of a microsecond. |
| 1160 | */ |
| 1161 | |
Chris Wilson | c33ed06 | 2017-02-17 15:13:01 +0000 | [diff] [blame] | 1162 | irq = atomic_read(&engine->irq_count); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1163 | timeout_us += local_clock_us(&cpu); |
| 1164 | do { |
Chris Wilson | b2f2f0f | 2017-09-22 13:03:33 +0100 | [diff] [blame] | 1165 | if (i915_seqno_passed(intel_engine_get_seqno(engine), seqno)) |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1166 | return seqno == i915_request_global_seqno(rq); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1167 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1168 | /* |
| 1169 | * Seqno are meant to be ordered *before* the interrupt. If |
Chris Wilson | c33ed06 | 2017-02-17 15:13:01 +0000 | [diff] [blame] | 1170 | * we see an interrupt without a corresponding seqno advance, |
| 1171 | * assume we won't see one in the near future but require |
| 1172 | * the engine->seqno_barrier() to fixup coherency. |
| 1173 | */ |
| 1174 | if (atomic_read(&engine->irq_count) != irq) |
| 1175 | break; |
| 1176 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1177 | if (signal_pending_state(state, current)) |
| 1178 | break; |
| 1179 | |
| 1180 | if (busywait_stop(timeout_us, cpu)) |
| 1181 | break; |
| 1182 | |
Christian Borntraeger | f2f09a4 | 2016-10-25 11:03:14 +0200 | [diff] [blame] | 1183 | cpu_relax(); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1184 | } while (!need_resched()); |
| 1185 | |
| 1186 | return false; |
| 1187 | } |
| 1188 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1189 | static bool __i915_wait_request_check_and_reset(struct i915_request *request) |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1190 | { |
Chris Wilson | d0667e9 | 2018-04-06 23:03:54 +0100 | [diff] [blame] | 1191 | struct i915_gpu_error *error = &request->i915->gpu_error; |
| 1192 | |
| 1193 | if (likely(!i915_reset_handoff(error))) |
Chris Wilson | e070511 | 2017-02-23 07:44:20 +0000 | [diff] [blame] | 1194 | return false; |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1195 | |
Chris Wilson | e070511 | 2017-02-23 07:44:20 +0000 | [diff] [blame] | 1196 | __set_current_state(TASK_RUNNING); |
Chris Wilson | d0667e9 | 2018-04-06 23:03:54 +0100 | [diff] [blame] | 1197 | i915_reset(request->i915, error->stalled_mask, error->reason); |
Chris Wilson | e070511 | 2017-02-23 07:44:20 +0000 | [diff] [blame] | 1198 | return true; |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1199 | } |
| 1200 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1201 | /** |
Michel Thierry | e532be8 | 2018-02-22 09:24:05 -0800 | [diff] [blame] | 1202 | * i915_request_wait - wait until execution of request has finished |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1203 | * @rq: the request to wait upon |
Chris Wilson | ea746f3 | 2016-09-09 14:11:49 +0100 | [diff] [blame] | 1204 | * @flags: how to wait |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1205 | * @timeout: how long to wait in jiffies |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1206 | * |
Michel Thierry | e532be8 | 2018-02-22 09:24:05 -0800 | [diff] [blame] | 1207 | * i915_request_wait() waits for the request to be completed, for a |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1208 | * maximum of @timeout jiffies (with MAX_SCHEDULE_TIMEOUT implying an |
| 1209 | * unbounded wait). |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1210 | * |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1211 | * If the caller holds the struct_mutex, the caller must pass I915_WAIT_LOCKED |
| 1212 | * in via the flags, and vice versa if the struct_mutex is not held, the caller |
| 1213 | * must not specify that the wait is locked. |
| 1214 | * |
| 1215 | * Returns the remaining time (in jiffies) if the request completed, which may |
| 1216 | * be zero or -ETIME if the request is unfinished after the timeout expires. |
| 1217 | * May return -EINTR is called with I915_WAIT_INTERRUPTIBLE and a signal is |
| 1218 | * pending before the request completes. |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1219 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1220 | long i915_request_wait(struct i915_request *rq, |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1221 | unsigned int flags, |
| 1222 | long timeout) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1223 | { |
Chris Wilson | ea746f3 | 2016-09-09 14:11:49 +0100 | [diff] [blame] | 1224 | const int state = flags & I915_WAIT_INTERRUPTIBLE ? |
| 1225 | TASK_INTERRUPTIBLE : TASK_UNINTERRUPTIBLE; |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1226 | wait_queue_head_t *errq = &rq->i915->gpu_error.wait_queue; |
Chris Wilson | a49625f | 2017-02-23 07:44:19 +0000 | [diff] [blame] | 1227 | DEFINE_WAIT_FUNC(reset, default_wake_function); |
| 1228 | DEFINE_WAIT_FUNC(exec, default_wake_function); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1229 | struct intel_wait wait; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1230 | |
| 1231 | might_sleep(); |
Chris Wilson | 22dd3bb | 2016-09-09 14:11:50 +0100 | [diff] [blame] | 1232 | #if IS_ENABLED(CONFIG_LOCKDEP) |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1233 | GEM_BUG_ON(debug_locks && |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1234 | !!lockdep_is_held(&rq->i915->drm.struct_mutex) != |
Chris Wilson | 22dd3bb | 2016-09-09 14:11:50 +0100 | [diff] [blame] | 1235 | !!(flags & I915_WAIT_LOCKED)); |
| 1236 | #endif |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1237 | GEM_BUG_ON(timeout < 0); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1238 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1239 | if (i915_request_completed(rq)) |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1240 | return timeout; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1241 | |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1242 | if (!timeout) |
| 1243 | return -ETIME; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1244 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1245 | trace_i915_request_wait_begin(rq, flags); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1246 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1247 | add_wait_queue(&rq->execute, &exec); |
Chris Wilson | 7de53bf | 2017-02-23 07:44:11 +0000 | [diff] [blame] | 1248 | if (flags & I915_WAIT_LOCKED) |
| 1249 | add_wait_queue(errq, &reset); |
| 1250 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1251 | intel_wait_init(&wait, rq); |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1252 | |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 1253 | restart: |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1254 | do { |
| 1255 | set_current_state(state); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1256 | if (intel_wait_update_request(&wait, rq)) |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1257 | break; |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1258 | |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1259 | if (flags & I915_WAIT_LOCKED && |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1260 | __i915_wait_request_check_and_reset(rq)) |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1261 | continue; |
Chris Wilson | 541ca6e | 2017-02-23 07:44:12 +0000 | [diff] [blame] | 1262 | |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1263 | if (signal_pending_state(state, current)) { |
| 1264 | timeout = -ERESTARTSYS; |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1265 | goto complete; |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1266 | } |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1267 | |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1268 | if (!timeout) { |
| 1269 | timeout = -ETIME; |
| 1270 | goto complete; |
| 1271 | } |
Chris Wilson | 541ca6e | 2017-02-23 07:44:12 +0000 | [diff] [blame] | 1272 | |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1273 | timeout = io_schedule_timeout(timeout); |
| 1274 | } while (1); |
Chris Wilson | 541ca6e | 2017-02-23 07:44:12 +0000 | [diff] [blame] | 1275 | |
Chris Wilson | 0f2f61d | 2017-02-23 07:44:22 +0000 | [diff] [blame] | 1276 | GEM_BUG_ON(!intel_wait_has_seqno(&wait)); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1277 | GEM_BUG_ON(!i915_sw_fence_signaled(&rq->submit)); |
Chris Wilson | 4680816b | 2016-10-28 13:58:48 +0100 | [diff] [blame] | 1278 | |
Daniel Vetter | 437c308 | 2016-08-05 18:11:24 +0200 | [diff] [blame] | 1279 | /* Optimistic short spin before touching IRQs */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1280 | if (__i915_spin_request(rq, wait.seqno, state, 5)) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1281 | goto complete; |
| 1282 | |
| 1283 | set_current_state(state); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1284 | if (intel_engine_add_wait(rq->engine, &wait)) |
| 1285 | /* |
| 1286 | * In order to check that we haven't missed the interrupt |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1287 | * as we enabled it, we need to kick ourselves to do a |
| 1288 | * coherent check on the seqno before we sleep. |
| 1289 | */ |
| 1290 | goto wakeup; |
| 1291 | |
Chris Wilson | 24f417e | 2017-02-23 07:44:21 +0000 | [diff] [blame] | 1292 | if (flags & I915_WAIT_LOCKED) |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1293 | __i915_wait_request_check_and_reset(rq); |
Chris Wilson | 24f417e | 2017-02-23 07:44:21 +0000 | [diff] [blame] | 1294 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1295 | for (;;) { |
| 1296 | if (signal_pending_state(state, current)) { |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1297 | timeout = -ERESTARTSYS; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1298 | break; |
| 1299 | } |
| 1300 | |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1301 | if (!timeout) { |
| 1302 | timeout = -ETIME; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1303 | break; |
| 1304 | } |
| 1305 | |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1306 | timeout = io_schedule_timeout(timeout); |
| 1307 | |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1308 | if (intel_wait_complete(&wait) && |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1309 | intel_wait_check_request(&wait, rq)) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1310 | break; |
| 1311 | |
| 1312 | set_current_state(state); |
| 1313 | |
| 1314 | wakeup: |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1315 | /* |
| 1316 | * Carefully check if the request is complete, giving time |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1317 | * for the seqno to be visible following the interrupt. |
| 1318 | * We also have to check in case we are kicked by the GPU |
| 1319 | * reset in order to drop the struct_mutex. |
| 1320 | */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1321 | if (__i915_request_irq_complete(rq)) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1322 | break; |
| 1323 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1324 | /* |
| 1325 | * If the GPU is hung, and we hold the lock, reset the GPU |
Chris Wilson | 221fe79 | 2016-09-09 14:11:51 +0100 | [diff] [blame] | 1326 | * and then check for completion. On a full reset, the engine's |
| 1327 | * HW seqno will be advanced passed us and we are complete. |
| 1328 | * If we do a partial reset, we have to wait for the GPU to |
| 1329 | * resume and update the breadcrumb. |
| 1330 | * |
| 1331 | * If we don't hold the mutex, we can just wait for the worker |
| 1332 | * to come along and update the breadcrumb (either directly |
| 1333 | * itself, or indirectly by recovering the GPU). |
| 1334 | */ |
| 1335 | if (flags & I915_WAIT_LOCKED && |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1336 | __i915_wait_request_check_and_reset(rq)) |
Chris Wilson | 221fe79 | 2016-09-09 14:11:51 +0100 | [diff] [blame] | 1337 | continue; |
Chris Wilson | 221fe79 | 2016-09-09 14:11:51 +0100 | [diff] [blame] | 1338 | |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1339 | /* Only spin if we know the GPU is processing this request */ |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1340 | if (__i915_spin_request(rq, wait.seqno, state, 2)) |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1341 | break; |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 1342 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1343 | if (!intel_wait_check_request(&wait, rq)) { |
| 1344 | intel_engine_remove_wait(rq->engine, &wait); |
Chris Wilson | d6a2289 | 2017-02-23 07:44:17 +0000 | [diff] [blame] | 1345 | goto restart; |
| 1346 | } |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1347 | } |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1348 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1349 | intel_engine_remove_wait(rq->engine, &wait); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1350 | complete: |
Chris Wilson | a49625f | 2017-02-23 07:44:19 +0000 | [diff] [blame] | 1351 | __set_current_state(TASK_RUNNING); |
Chris Wilson | 7de53bf | 2017-02-23 07:44:11 +0000 | [diff] [blame] | 1352 | if (flags & I915_WAIT_LOCKED) |
| 1353 | remove_wait_queue(errq, &reset); |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1354 | remove_wait_queue(&rq->execute, &exec); |
| 1355 | trace_i915_request_wait_end(rq); |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1356 | |
Chris Wilson | e95433c | 2016-10-28 13:58:27 +0100 | [diff] [blame] | 1357 | return timeout; |
Chris Wilson | 05235c5 | 2016-07-20 09:21:08 +0100 | [diff] [blame] | 1358 | } |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1359 | |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 1360 | static void engine_retire_requests(struct intel_engine_cs *engine) |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1361 | { |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1362 | struct i915_request *request, *next; |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1363 | u32 seqno = intel_engine_get_seqno(engine); |
| 1364 | LIST_HEAD(retire); |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1365 | |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1366 | spin_lock_irq(&engine->timeline->lock); |
Chris Wilson | 73cb970 | 2016-10-28 13:58:46 +0100 | [diff] [blame] | 1367 | list_for_each_entry_safe(request, next, |
| 1368 | &engine->timeline->requests, link) { |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1369 | if (!i915_seqno_passed(seqno, request->global_seqno)) |
| 1370 | break; |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1371 | |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1372 | list_move_tail(&request->link, &retire); |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1373 | } |
Chris Wilson | 754c9fd | 2017-02-23 07:44:14 +0000 | [diff] [blame] | 1374 | spin_unlock_irq(&engine->timeline->lock); |
| 1375 | |
| 1376 | list_for_each_entry_safe(request, next, &retire, link) |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1377 | i915_request_retire(request); |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1378 | } |
| 1379 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1380 | void i915_retire_requests(struct drm_i915_private *i915) |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1381 | { |
| 1382 | struct intel_engine_cs *engine; |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 1383 | enum intel_engine_id id; |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1384 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1385 | lockdep_assert_held(&i915->drm.struct_mutex); |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1386 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1387 | if (!i915->gt.active_requests) |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1388 | return; |
| 1389 | |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1390 | for_each_engine(engine, i915, id) |
Chris Wilson | 28176ef | 2016-10-28 13:58:56 +0100 | [diff] [blame] | 1391 | engine_retire_requests(engine); |
Chris Wilson | 4b8de8e | 2016-08-04 07:52:42 +0100 | [diff] [blame] | 1392 | } |
Chris Wilson | c835c55 | 2017-02-13 17:15:21 +0000 | [diff] [blame] | 1393 | |
| 1394 | #if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) |
| 1395 | #include "selftests/mock_request.c" |
Chris Wilson | e61e0f5 | 2018-02-21 09:56:36 +0000 | [diff] [blame] | 1396 | #include "selftests/i915_request.c" |
Chris Wilson | c835c55 | 2017-02-13 17:15:21 +0000 | [diff] [blame] | 1397 | #endif |