Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 1 | /* |
| 2 | * Copyright © 2008-2010 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 | * Authors: |
| 24 | * Eric Anholt <eric@anholt.net> |
| 25 | * Chris Wilson <chris@chris-wilson.co.uuk> |
| 26 | * |
| 27 | */ |
| 28 | |
David Howells | 760285e | 2012-10-02 18:01:07 +0100 | [diff] [blame] | 29 | #include <drm/drmP.h> |
David Howells | 760285e | 2012-10-02 18:01:07 +0100 | [diff] [blame] | 30 | #include <drm/i915_drm.h> |
Chris Wilson | 74e21ac | 2014-01-20 10:17:37 +0000 | [diff] [blame] | 31 | |
| 32 | #include "i915_drv.h" |
| 33 | #include "intel_drv.h" |
Chris Wilson | db53a30 | 2011-02-03 11:57:46 +0000 | [diff] [blame] | 34 | #include "i915_trace.h" |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 35 | |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 36 | static bool |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 37 | gpu_is_idle(struct drm_i915_private *dev_priv) |
| 38 | { |
| 39 | struct intel_engine_cs *engine; |
| 40 | |
| 41 | for_each_engine(engine, dev_priv) { |
Chris Wilson | dcff85c | 2016-08-05 10:14:11 +0100 | [diff] [blame] | 42 | if (intel_engine_is_active(engine)) |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 43 | return false; |
| 44 | } |
| 45 | |
| 46 | return true; |
| 47 | } |
| 48 | |
| 49 | static bool |
Chris Wilson | 8211887 | 2016-08-18 17:17:05 +0100 | [diff] [blame] | 50 | mark_free(struct i915_vma *vma, unsigned int flags, struct list_head *unwind) |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 51 | { |
Chris Wilson | 20dfbde | 2016-08-04 16:32:30 +0100 | [diff] [blame] | 52 | if (i915_vma_is_pinned(vma)) |
Chris Wilson | 1b50247 | 2012-04-24 15:47:30 +0100 | [diff] [blame] | 53 | return false; |
| 54 | |
Daniel Vetter | b93dab6 | 2013-08-26 11:23:47 +0200 | [diff] [blame] | 55 | if (WARN_ON(!list_empty(&vma->exec_list))) |
| 56 | return false; |
| 57 | |
Chris Wilson | 8211887 | 2016-08-18 17:17:05 +0100 | [diff] [blame] | 58 | if (flags & PIN_NONFAULT && vma->obj->fault_mappable) |
| 59 | return false; |
| 60 | |
Ben Widawsky | 82a55ad | 2013-08-14 11:38:34 +0200 | [diff] [blame] | 61 | list_add(&vma->exec_list, unwind); |
Ben Widawsky | 2f63315 | 2013-07-17 12:19:03 -0700 | [diff] [blame] | 62 | return drm_mm_scan_add_block(&vma->node); |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 63 | } |
| 64 | |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 65 | /** |
| 66 | * i915_gem_evict_something - Evict vmas to make room for binding a new one |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 67 | * @vm: address space to evict from |
Daniel Vetter | 7838a63 | 2015-01-05 14:36:59 +0100 | [diff] [blame] | 68 | * @min_size: size of the desired free space |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 69 | * @alignment: alignment constraint of the desired free space |
| 70 | * @cache_level: cache_level for the desired space |
Daniel Vetter | 7838a63 | 2015-01-05 14:36:59 +0100 | [diff] [blame] | 71 | * @start: start (inclusive) of the range from which to evict objects |
| 72 | * @end: end (exclusive) of the range from which to evict objects |
| 73 | * @flags: additional flags to control the eviction algorithm |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 74 | * |
| 75 | * This function will try to evict vmas until a free space satisfying the |
| 76 | * requirements is found. Callers must check first whether any such hole exists |
| 77 | * already before calling this function. |
| 78 | * |
| 79 | * This function is used by the object/vma binding code. |
| 80 | * |
Daniel Vetter | eb0b44a | 2015-03-18 14:47:59 +0100 | [diff] [blame] | 81 | * Since this function is only used to free up virtual address space it only |
| 82 | * ignores pinned vmas, and not object where the backing storage itself is |
| 83 | * pinned. Hence obj->pages_pin_count does not protect against eviction. |
| 84 | * |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 85 | * To clarify: This is for freeing up virtual address space, not for freeing |
| 86 | * memory in e.g. the shrinker. |
| 87 | */ |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 88 | int |
Chris Wilson | e522ac2 | 2016-08-04 16:32:18 +0100 | [diff] [blame] | 89 | i915_gem_evict_something(struct i915_address_space *vm, |
Chris Wilson | 2ffffd0 | 2016-08-04 16:32:22 +0100 | [diff] [blame] | 90 | u64 min_size, u64 alignment, |
| 91 | unsigned cache_level, |
| 92 | u64 start, u64 end, |
Daniel Vetter | 1ec9e26 | 2014-02-14 14:01:11 +0100 | [diff] [blame] | 93 | unsigned flags) |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 94 | { |
Chris Wilson | e522ac2 | 2016-08-04 16:32:18 +0100 | [diff] [blame] | 95 | struct drm_i915_private *dev_priv = to_i915(vm->dev); |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 96 | struct list_head eviction_list; |
| 97 | struct list_head *phases[] = { |
| 98 | &vm->inactive_list, |
| 99 | &vm->active_list, |
| 100 | NULL, |
| 101 | }, **phase; |
| 102 | struct i915_vma *vma, *next; |
| 103 | int ret; |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 104 | |
Chris Wilson | e522ac2 | 2016-08-04 16:32:18 +0100 | [diff] [blame] | 105 | trace_i915_gem_evict(vm, min_size, alignment, flags); |
Chris Wilson | db53a30 | 2011-02-03 11:57:46 +0000 | [diff] [blame] | 106 | |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 107 | /* |
| 108 | * The goal is to evict objects and amalgamate space in LRU order. |
| 109 | * The oldest idle objects reside on the inactive list, which is in |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 110 | * retirement order. The next objects to retire are those in flight, |
| 111 | * on the active list, again in retirement order. |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 112 | * |
| 113 | * The retirement sequence is thus: |
| 114 | * 1. Inactive objects (already retired) |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 115 | * 2. Active objects (will stall on unbinding) |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 116 | * |
| 117 | * On each list, the oldest objects lie at the HEAD with the freshest |
| 118 | * object on the TAIL. |
| 119 | */ |
Chris Wilson | d23db88 | 2014-05-23 08:48:08 +0200 | [diff] [blame] | 120 | if (start != 0 || end != vm->total) { |
Ben Widawsky | 5cef07e | 2013-07-16 16:50:08 -0700 | [diff] [blame] | 121 | drm_mm_init_scan_with_range(&vm->mm, min_size, |
Chris Wilson | d23db88 | 2014-05-23 08:48:08 +0200 | [diff] [blame] | 122 | alignment, cache_level, |
| 123 | start, end); |
Ben Widawsky | f6cd1f1 | 2013-07-31 17:00:11 -0700 | [diff] [blame] | 124 | } else |
Ben Widawsky | 5cef07e | 2013-07-16 16:50:08 -0700 | [diff] [blame] | 125 | drm_mm_init_scan(&vm->mm, min_size, alignment, cache_level); |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 126 | |
Daniel Vetter | 1ec9e26 | 2014-02-14 14:01:11 +0100 | [diff] [blame] | 127 | if (flags & PIN_NONBLOCK) |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 128 | phases[1] = NULL; |
Chris Wilson | 86a1ee2 | 2012-08-11 15:41:04 +0100 | [diff] [blame] | 129 | |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 130 | search_again: |
| 131 | INIT_LIST_HEAD(&eviction_list); |
| 132 | phase = phases; |
| 133 | do { |
| 134 | list_for_each_entry(vma, *phase, vm_link) |
Chris Wilson | 8211887 | 2016-08-18 17:17:05 +0100 | [diff] [blame] | 135 | if (mark_free(vma, flags, &eviction_list)) |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 136 | goto found; |
| 137 | } while (*++phase); |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 138 | |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 139 | /* Nothing found, clean up and bail out! */ |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 140 | list_for_each_entry_safe(vma, next, &eviction_list, exec_list) { |
Ben Widawsky | 2f63315 | 2013-07-17 12:19:03 -0700 | [diff] [blame] | 141 | ret = drm_mm_scan_remove_block(&vma->node); |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 142 | BUG_ON(ret); |
Chris Wilson | 092de6f | 2011-01-10 14:21:05 +0000 | [diff] [blame] | 143 | |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 144 | INIT_LIST_HEAD(&vma->exec_list); |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 145 | } |
| 146 | |
Chris Wilson | ad071ac | 2013-12-09 10:37:24 +0000 | [diff] [blame] | 147 | /* Can we unpin some objects such as idle hw contents, |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 148 | * or pending flips? But since only the GGTT has global entries |
| 149 | * such as scanouts, rinbuffers and contexts, we can skip the |
| 150 | * purge when inspecting per-process local address spaces. |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 151 | */ |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 152 | if (!i915_is_ggtt(vm) || flags & PIN_NONBLOCK) |
Chris Wilson | 74e21ac | 2014-01-20 10:17:37 +0000 | [diff] [blame] | 153 | return -ENOSPC; |
Chris Wilson | ad071ac | 2013-12-09 10:37:24 +0000 | [diff] [blame] | 154 | |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 155 | if (gpu_is_idle(dev_priv)) { |
| 156 | /* If we still have pending pageflip completions, drop |
| 157 | * back to userspace to give our workqueues time to |
| 158 | * acquire our locks and unpin the old scanouts. |
| 159 | */ |
Chris Wilson | e522ac2 | 2016-08-04 16:32:18 +0100 | [diff] [blame] | 160 | return intel_has_pending_fb_unpin(vm->dev) ? -EAGAIN : -ENOSPC; |
Chris Wilson | 74e21ac | 2014-01-20 10:17:37 +0000 | [diff] [blame] | 161 | } |
| 162 | |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 163 | /* Not everything in the GGTT is tracked via vma (otherwise we |
| 164 | * could evict as required with minimal stalling) so we are forced |
| 165 | * to idle the GPU and explicitly retire outstanding requests in |
| 166 | * the hopes that we can then remove contexts and the like only |
| 167 | * bound by their active reference. |
Chris Wilson | 74e21ac | 2014-01-20 10:17:37 +0000 | [diff] [blame] | 168 | */ |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 169 | ret = i915_gem_switch_to_kernel_context(dev_priv); |
| 170 | if (ret) |
| 171 | return ret; |
| 172 | |
Chris Wilson | 22dd3bb | 2016-09-09 14:11:50 +0100 | [diff] [blame] | 173 | ret = i915_gem_wait_for_idle(dev_priv, |
| 174 | I915_WAIT_INTERRUPTIBLE | |
| 175 | I915_WAIT_LOCKED); |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 176 | if (ret) |
| 177 | return ret; |
| 178 | |
| 179 | i915_gem_retire_requests(dev_priv); |
| 180 | goto search_again; |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 181 | |
| 182 | found: |
Chris Wilson | e39a015 | 2010-09-29 22:23:05 +0100 | [diff] [blame] | 183 | /* drm_mm doesn't allow any other other operations while |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 184 | * scanning, therefore store to-be-evicted objects on a |
| 185 | * temporary list and take a reference for all before |
| 186 | * calling unbind (which may remove the active reference |
| 187 | * of any of our objects, thus corrupting the list). |
| 188 | */ |
| 189 | list_for_each_entry_safe(vma, next, &eviction_list, exec_list) { |
| 190 | if (drm_mm_scan_remove_block(&vma->node)) |
Chris Wilson | 20dfbde | 2016-08-04 16:32:30 +0100 | [diff] [blame] | 191 | __i915_vma_pin(vma); |
Chris Wilson | 9332f3b | 2016-08-04 16:32:17 +0100 | [diff] [blame] | 192 | else |
| 193 | list_del_init(&vma->exec_list); |
Chris Wilson | cd377ea | 2010-08-07 11:01:24 +0100 | [diff] [blame] | 194 | } |
| 195 | |
Chris Wilson | e39a015 | 2010-09-29 22:23:05 +0100 | [diff] [blame] | 196 | /* Unbinding will emit any required flushes */ |
| 197 | while (!list_empty(&eviction_list)) { |
Ben Widawsky | 82a55ad | 2013-08-14 11:38:34 +0200 | [diff] [blame] | 198 | vma = list_first_entry(&eviction_list, |
| 199 | struct i915_vma, |
Chris Wilson | 432e58e | 2010-11-25 19:32:06 +0000 | [diff] [blame] | 200 | exec_list); |
Ben Widawsky | 8637b40 | 2013-08-16 13:29:33 -0700 | [diff] [blame] | 201 | |
Ben Widawsky | 8637b40 | 2013-08-16 13:29:33 -0700 | [diff] [blame] | 202 | list_del_init(&vma->exec_list); |
Chris Wilson | 20dfbde | 2016-08-04 16:32:30 +0100 | [diff] [blame] | 203 | __i915_vma_unpin(vma); |
Chris Wilson | e39a015 | 2010-09-29 22:23:05 +0100 | [diff] [blame] | 204 | if (ret == 0) |
Ben Widawsky | 82a55ad | 2013-08-14 11:38:34 +0200 | [diff] [blame] | 205 | ret = i915_vma_unbind(vma); |
Chris Wilson | e39a015 | 2010-09-29 22:23:05 +0100 | [diff] [blame] | 206 | } |
Chris Wilson | e39a015 | 2010-09-29 22:23:05 +0100 | [diff] [blame] | 207 | return ret; |
Chris Wilson | b47eb4a | 2010-08-07 11:01:23 +0100 | [diff] [blame] | 208 | } |
| 209 | |
Chris Wilson | 506a8e8 | 2015-12-08 11:55:07 +0000 | [diff] [blame] | 210 | int |
| 211 | i915_gem_evict_for_vma(struct i915_vma *target) |
| 212 | { |
| 213 | struct drm_mm_node *node, *next; |
| 214 | |
| 215 | list_for_each_entry_safe(node, next, |
| 216 | &target->vm->mm.head_node.node_list, |
| 217 | node_list) { |
| 218 | struct i915_vma *vma; |
| 219 | int ret; |
| 220 | |
| 221 | if (node->start + node->size <= target->node.start) |
| 222 | continue; |
| 223 | if (node->start >= target->node.start + target->node.size) |
| 224 | break; |
| 225 | |
| 226 | vma = container_of(node, typeof(*vma), node); |
| 227 | |
Chris Wilson | 20dfbde | 2016-08-04 16:32:30 +0100 | [diff] [blame] | 228 | if (i915_vma_is_pinned(vma)) { |
| 229 | if (!vma->exec_entry || i915_vma_pin_count(vma) > 1) |
Chris Wilson | 506a8e8 | 2015-12-08 11:55:07 +0000 | [diff] [blame] | 230 | /* Object is pinned for some other use */ |
| 231 | return -EBUSY; |
| 232 | |
| 233 | /* We need to evict a buffer in the same batch */ |
| 234 | if (vma->exec_entry->flags & EXEC_OBJECT_PINNED) |
| 235 | /* Overlapping fixed objects in the same batch */ |
| 236 | return -EINVAL; |
| 237 | |
| 238 | return -ENOSPC; |
| 239 | } |
| 240 | |
| 241 | ret = i915_vma_unbind(vma); |
| 242 | if (ret) |
| 243 | return ret; |
| 244 | } |
| 245 | |
| 246 | return 0; |
| 247 | } |
| 248 | |
Ben Widawsky | 68c8c17 | 2013-09-11 14:57:50 -0700 | [diff] [blame] | 249 | /** |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 250 | * i915_gem_evict_vm - Evict all idle vmas from a vm |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 251 | * @vm: Address space to cleanse |
Ben Widawsky | 68c8c17 | 2013-09-11 14:57:50 -0700 | [diff] [blame] | 252 | * @do_idle: Boolean directing whether to idle first. |
| 253 | * |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 254 | * This function evicts all idles vmas from a vm. If all unpinned vmas should be |
| 255 | * evicted the @do_idle needs to be set to true. |
Ben Widawsky | 68c8c17 | 2013-09-11 14:57:50 -0700 | [diff] [blame] | 256 | * |
Daniel Vetter | c2c1d49 | 2014-01-29 22:07:11 +0100 | [diff] [blame] | 257 | * This is used by the execbuf code as a last-ditch effort to defragment the |
| 258 | * address space. |
| 259 | * |
| 260 | * To clarify: This is for freeing up virtual address space, not for freeing |
| 261 | * memory in e.g. the shrinker. |
Ben Widawsky | 68c8c17 | 2013-09-11 14:57:50 -0700 | [diff] [blame] | 262 | */ |
| 263 | int i915_gem_evict_vm(struct i915_address_space *vm, bool do_idle) |
Ben Widawsky | 7b79612 | 2013-09-11 14:57:49 -0700 | [diff] [blame] | 264 | { |
| 265 | struct i915_vma *vma, *next; |
| 266 | int ret; |
| 267 | |
Ben Widawsky | b9b5dce | 2014-12-23 17:16:04 +0000 | [diff] [blame] | 268 | WARN_ON(!mutex_is_locked(&vm->dev->struct_mutex)); |
Ben Widawsky | bcccff8 | 2013-09-24 09:57:56 -0700 | [diff] [blame] | 269 | trace_i915_gem_evict_vm(vm); |
| 270 | |
Ben Widawsky | 7b79612 | 2013-09-11 14:57:49 -0700 | [diff] [blame] | 271 | if (do_idle) { |
Chris Wilson | 6e5a5be | 2016-06-24 14:55:57 +0100 | [diff] [blame] | 272 | struct drm_i915_private *dev_priv = to_i915(vm->dev); |
| 273 | |
Chris Wilson | 883445d | 2016-06-24 14:55:58 +0100 | [diff] [blame] | 274 | if (i915_is_ggtt(vm)) { |
Chris Wilson | 945657b | 2016-07-15 14:56:19 +0100 | [diff] [blame] | 275 | ret = i915_gem_switch_to_kernel_context(dev_priv); |
Chris Wilson | 883445d | 2016-06-24 14:55:58 +0100 | [diff] [blame] | 276 | if (ret) |
| 277 | return ret; |
| 278 | } |
Ben Widawsky | 7b79612 | 2013-09-11 14:57:49 -0700 | [diff] [blame] | 279 | |
Chris Wilson | 22dd3bb | 2016-09-09 14:11:50 +0100 | [diff] [blame] | 280 | ret = i915_gem_wait_for_idle(dev_priv, |
| 281 | I915_WAIT_INTERRUPTIBLE | |
| 282 | I915_WAIT_LOCKED); |
Chris Wilson | 6e5a5be | 2016-06-24 14:55:57 +0100 | [diff] [blame] | 283 | if (ret) |
| 284 | return ret; |
| 285 | |
| 286 | i915_gem_retire_requests(dev_priv); |
Ben Widawsky | b9b5dce | 2014-12-23 17:16:04 +0000 | [diff] [blame] | 287 | WARN_ON(!list_empty(&vm->active_list)); |
Ben Widawsky | 7b79612 | 2013-09-11 14:57:49 -0700 | [diff] [blame] | 288 | } |
| 289 | |
Chris Wilson | 1c7f4bc | 2016-02-26 11:03:19 +0000 | [diff] [blame] | 290 | list_for_each_entry_safe(vma, next, &vm->inactive_list, vm_link) |
Chris Wilson | 20dfbde | 2016-08-04 16:32:30 +0100 | [diff] [blame] | 291 | if (!i915_vma_is_pinned(vma)) |
Ben Widawsky | 7b79612 | 2013-09-11 14:57:49 -0700 | [diff] [blame] | 292 | WARN_ON(i915_vma_unbind(vma)); |
| 293 | |
| 294 | return 0; |
| 295 | } |