i915_gem_request.c 20 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720
  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. #include "i915_drv.h"
  25. static const char *i915_fence_get_driver_name(struct fence *fence)
  26. {
  27. return "i915";
  28. }
  29. static const char *i915_fence_get_timeline_name(struct fence *fence)
  30. {
  31. /* Timelines are bound by eviction to a VM. However, since
  32. * we only have a global seqno at the moment, we only have
  33. * a single timeline. Note that each timeline will have
  34. * multiple execution contexts (fence contexts) as we allow
  35. * engines within a single timeline to execute in parallel.
  36. */
  37. return "global";
  38. }
  39. static bool i915_fence_signaled(struct fence *fence)
  40. {
  41. return i915_gem_request_completed(to_request(fence));
  42. }
  43. static bool i915_fence_enable_signaling(struct fence *fence)
  44. {
  45. if (i915_fence_signaled(fence))
  46. return false;
  47. intel_engine_enable_signaling(to_request(fence));
  48. return true;
  49. }
  50. static signed long i915_fence_wait(struct fence *fence,
  51. bool interruptible,
  52. signed long timeout_jiffies)
  53. {
  54. s64 timeout_ns, *timeout;
  55. int ret;
  56. if (timeout_jiffies != MAX_SCHEDULE_TIMEOUT) {
  57. timeout_ns = jiffies_to_nsecs(timeout_jiffies);
  58. timeout = &timeout_ns;
  59. } else {
  60. timeout = NULL;
  61. }
  62. ret = __i915_wait_request(to_request(fence),
  63. interruptible, timeout,
  64. NO_WAITBOOST);
  65. if (ret == -ETIME)
  66. return 0;
  67. if (ret < 0)
  68. return ret;
  69. if (timeout_jiffies != MAX_SCHEDULE_TIMEOUT)
  70. timeout_jiffies = nsecs_to_jiffies(timeout_ns);
  71. return timeout_jiffies;
  72. }
  73. static void i915_fence_value_str(struct fence *fence, char *str, int size)
  74. {
  75. snprintf(str, size, "%u", fence->seqno);
  76. }
  77. static void i915_fence_timeline_value_str(struct fence *fence, char *str,
  78. int size)
  79. {
  80. snprintf(str, size, "%u",
  81. intel_engine_get_seqno(to_request(fence)->engine));
  82. }
  83. static void i915_fence_release(struct fence *fence)
  84. {
  85. struct drm_i915_gem_request *req = to_request(fence);
  86. kmem_cache_free(req->i915->requests, req);
  87. }
  88. const struct fence_ops i915_fence_ops = {
  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,
  95. .fence_value_str = i915_fence_value_str,
  96. .timeline_value_str = i915_fence_timeline_value_str,
  97. };
  98. int i915_gem_request_add_to_client(struct drm_i915_gem_request *req,
  99. struct drm_file *file)
  100. {
  101. struct drm_i915_private *dev_private;
  102. struct drm_i915_file_private *file_priv;
  103. WARN_ON(!req || !file || req->file_priv);
  104. if (!req || !file)
  105. return -EINVAL;
  106. if (req->file_priv)
  107. return -EINVAL;
  108. dev_private = req->i915;
  109. file_priv = file->driver_priv;
  110. spin_lock(&file_priv->mm.lock);
  111. req->file_priv = file_priv;
  112. list_add_tail(&req->client_list, &file_priv->mm.request_list);
  113. spin_unlock(&file_priv->mm.lock);
  114. req->pid = get_pid(task_pid(current));
  115. return 0;
  116. }
  117. static inline void
  118. i915_gem_request_remove_from_client(struct drm_i915_gem_request *request)
  119. {
  120. struct drm_i915_file_private *file_priv = request->file_priv;
  121. if (!file_priv)
  122. return;
  123. spin_lock(&file_priv->mm.lock);
  124. list_del(&request->client_list);
  125. request->file_priv = NULL;
  126. spin_unlock(&file_priv->mm.lock);
  127. put_pid(request->pid);
  128. request->pid = NULL;
  129. }
  130. static void i915_gem_request_retire(struct drm_i915_gem_request *request)
  131. {
  132. trace_i915_gem_request_retire(request);
  133. list_del_init(&request->list);
  134. /* We know the GPU must have read the request to have
  135. * sent us the seqno + interrupt, so use the position
  136. * of tail of the request to update the last known position
  137. * of the GPU head.
  138. *
  139. * Note this requires that we are always called in request
  140. * completion order.
  141. */
  142. request->ring->last_retired_head = request->postfix;
  143. i915_gem_request_remove_from_client(request);
  144. if (request->previous_context) {
  145. if (i915.enable_execlists)
  146. intel_lr_context_unpin(request->previous_context,
  147. request->engine);
  148. }
  149. i915_gem_context_put(request->ctx);
  150. i915_gem_request_put(request);
  151. }
  152. void i915_gem_request_retire_upto(struct drm_i915_gem_request *req)
  153. {
  154. struct intel_engine_cs *engine = req->engine;
  155. struct drm_i915_gem_request *tmp;
  156. lockdep_assert_held(&req->i915->drm.struct_mutex);
  157. if (list_empty(&req->list))
  158. return;
  159. do {
  160. tmp = list_first_entry(&engine->request_list,
  161. typeof(*tmp), list);
  162. i915_gem_request_retire(tmp);
  163. } while (tmp != req);
  164. WARN_ON(i915_verify_lists(engine->dev));
  165. }
  166. static int i915_gem_check_wedge(unsigned int reset_counter, bool interruptible)
  167. {
  168. if (__i915_terminally_wedged(reset_counter))
  169. return -EIO;
  170. if (__i915_reset_in_progress(reset_counter)) {
  171. /* Non-interruptible callers can't handle -EAGAIN, hence return
  172. * -EIO unconditionally for these.
  173. */
  174. if (!interruptible)
  175. return -EIO;
  176. return -EAGAIN;
  177. }
  178. return 0;
  179. }
  180. static int i915_gem_init_seqno(struct drm_i915_private *dev_priv, u32 seqno)
  181. {
  182. struct intel_engine_cs *engine;
  183. int ret;
  184. /* Carefully retire all requests without writing to the rings */
  185. for_each_engine(engine, dev_priv) {
  186. ret = intel_engine_idle(engine);
  187. if (ret)
  188. return ret;
  189. }
  190. i915_gem_retire_requests(dev_priv);
  191. /* If the seqno wraps around, we need to clear the breadcrumb rbtree */
  192. if (!i915_seqno_passed(seqno, dev_priv->next_seqno)) {
  193. while (intel_kick_waiters(dev_priv) ||
  194. intel_kick_signalers(dev_priv))
  195. yield();
  196. }
  197. /* Finally reset hw state */
  198. for_each_engine(engine, dev_priv)
  199. intel_engine_init_seqno(engine, seqno);
  200. return 0;
  201. }
  202. int i915_gem_set_seqno(struct drm_device *dev, u32 seqno)
  203. {
  204. struct drm_i915_private *dev_priv = to_i915(dev);
  205. int ret;
  206. if (seqno == 0)
  207. return -EINVAL;
  208. /* HWS page needs to be set less than what we
  209. * will inject to ring
  210. */
  211. ret = i915_gem_init_seqno(dev_priv, seqno - 1);
  212. if (ret)
  213. return ret;
  214. dev_priv->next_seqno = seqno;
  215. return 0;
  216. }
  217. static int i915_gem_get_seqno(struct drm_i915_private *dev_priv, u32 *seqno)
  218. {
  219. /* reserve 0 for non-seqno */
  220. if (unlikely(dev_priv->next_seqno == 0)) {
  221. int ret;
  222. ret = i915_gem_init_seqno(dev_priv, 0);
  223. if (ret)
  224. return ret;
  225. dev_priv->next_seqno = 1;
  226. }
  227. *seqno = dev_priv->next_seqno++;
  228. return 0;
  229. }
  230. /**
  231. * i915_gem_request_alloc - allocate a request structure
  232. *
  233. * @engine: engine that we wish to issue the request on.
  234. * @ctx: context that the request will be associated with.
  235. * This can be NULL if the request is not directly related to
  236. * any specific user context, in which case this function will
  237. * choose an appropriate context to use.
  238. *
  239. * Returns a pointer to the allocated request if successful,
  240. * or an error code if not.
  241. */
  242. struct drm_i915_gem_request *
  243. i915_gem_request_alloc(struct intel_engine_cs *engine,
  244. struct i915_gem_context *ctx)
  245. {
  246. struct drm_i915_private *dev_priv = engine->i915;
  247. unsigned int reset_counter = i915_reset_counter(&dev_priv->gpu_error);
  248. struct drm_i915_gem_request *req;
  249. u32 seqno;
  250. int ret;
  251. /* ABI: Before userspace accesses the GPU (e.g. execbuffer), report
  252. * EIO if the GPU is already wedged, or EAGAIN to drop the struct_mutex
  253. * and restart.
  254. */
  255. ret = i915_gem_check_wedge(reset_counter, dev_priv->mm.interruptible);
  256. if (ret)
  257. return ERR_PTR(ret);
  258. /* Move the oldest request to the slab-cache (if not in use!) */
  259. req = list_first_entry_or_null(&engine->request_list,
  260. typeof(*req), list);
  261. if (req && i915_gem_request_completed(req))
  262. i915_gem_request_retire(req);
  263. req = kmem_cache_zalloc(dev_priv->requests, GFP_KERNEL);
  264. if (!req)
  265. return ERR_PTR(-ENOMEM);
  266. ret = i915_gem_get_seqno(dev_priv, &seqno);
  267. if (ret)
  268. goto err;
  269. spin_lock_init(&req->lock);
  270. fence_init(&req->fence,
  271. &i915_fence_ops,
  272. &req->lock,
  273. engine->fence_context,
  274. seqno);
  275. req->i915 = dev_priv;
  276. req->engine = engine;
  277. req->ctx = i915_gem_context_get(ctx);
  278. /*
  279. * Reserve space in the ring buffer for all the commands required to
  280. * eventually emit this request. This is to guarantee that the
  281. * i915_add_request() call can't fail. Note that the reserve may need
  282. * to be redone if the request is not actually submitted straight
  283. * away, e.g. because a GPU scheduler has deferred it.
  284. */
  285. req->reserved_space = MIN_SPACE_FOR_ADD_REQUEST;
  286. if (i915.enable_execlists)
  287. ret = intel_logical_ring_alloc_request_extras(req);
  288. else
  289. ret = intel_ring_alloc_request_extras(req);
  290. if (ret)
  291. goto err_ctx;
  292. return req;
  293. err_ctx:
  294. i915_gem_context_put(ctx);
  295. err:
  296. kmem_cache_free(dev_priv->requests, req);
  297. return ERR_PTR(ret);
  298. }
  299. static void i915_gem_mark_busy(const struct intel_engine_cs *engine)
  300. {
  301. struct drm_i915_private *dev_priv = engine->i915;
  302. dev_priv->gt.active_engines |= intel_engine_flag(engine);
  303. if (dev_priv->gt.awake)
  304. return;
  305. intel_runtime_pm_get_noresume(dev_priv);
  306. dev_priv->gt.awake = true;
  307. intel_enable_gt_powersave(dev_priv);
  308. i915_update_gfx_val(dev_priv);
  309. if (INTEL_GEN(dev_priv) >= 6)
  310. gen6_rps_busy(dev_priv);
  311. queue_delayed_work(dev_priv->wq,
  312. &dev_priv->gt.retire_work,
  313. round_jiffies_up_relative(HZ));
  314. }
  315. /*
  316. * NB: This function is not allowed to fail. Doing so would mean the the
  317. * request is not being tracked for completion but the work itself is
  318. * going to happen on the hardware. This would be a Bad Thing(tm).
  319. */
  320. void __i915_add_request(struct drm_i915_gem_request *request,
  321. struct drm_i915_gem_object *obj,
  322. bool flush_caches)
  323. {
  324. struct intel_engine_cs *engine;
  325. struct intel_ring *ring;
  326. u32 request_start;
  327. u32 reserved_tail;
  328. int ret;
  329. if (WARN_ON(!request))
  330. return;
  331. engine = request->engine;
  332. ring = request->ring;
  333. /*
  334. * To ensure that this call will not fail, space for its emissions
  335. * should already have been reserved in the ring buffer. Let the ring
  336. * know that it is time to use that space up.
  337. */
  338. request_start = ring->tail;
  339. reserved_tail = request->reserved_space;
  340. request->reserved_space = 0;
  341. /*
  342. * Emit any outstanding flushes - execbuf can fail to emit the flush
  343. * after having emitted the batchbuffer command. Hence we need to fix
  344. * things up similar to emitting the lazy request. The difference here
  345. * is that the flush _must_ happen before the next request, no matter
  346. * what.
  347. */
  348. if (flush_caches) {
  349. ret = engine->emit_flush(request, EMIT_FLUSH);
  350. /* Not allowed to fail! */
  351. WARN(ret, "engine->emit_flush() failed: %d!\n", ret);
  352. }
  353. trace_i915_gem_request_add(request);
  354. request->head = request_start;
  355. /* Whilst this request exists, batch_obj will be on the
  356. * active_list, and so will hold the active reference. Only when this
  357. * request is retired will the the batch_obj be moved onto the
  358. * inactive_list and lose its active reference. Hence we do not need
  359. * to explicitly hold another reference here.
  360. */
  361. request->batch_obj = obj;
  362. /* Seal the request and mark it as pending execution. Note that
  363. * we may inspect this state, without holding any locks, during
  364. * hangcheck. Hence we apply the barrier to ensure that we do not
  365. * see a more recent value in the hws than we are tracking.
  366. */
  367. request->emitted_jiffies = jiffies;
  368. request->previous_seqno = engine->last_submitted_seqno;
  369. smp_store_mb(engine->last_submitted_seqno, request->fence.seqno);
  370. list_add_tail(&request->list, &engine->request_list);
  371. /* Record the position of the start of the request so that
  372. * should we detect the updated seqno part-way through the
  373. * GPU processing the request, we never over-estimate the
  374. * position of the head.
  375. */
  376. request->postfix = ring->tail;
  377. /* Not allowed to fail! */
  378. ret = engine->emit_request(request);
  379. WARN(ret, "(%s)->emit_request failed: %d!\n", engine->name, ret);
  380. /* Sanity check that the reserved size was large enough. */
  381. ret = ring->tail - request_start;
  382. if (ret < 0)
  383. ret += ring->size;
  384. WARN_ONCE(ret > reserved_tail,
  385. "Not enough space reserved (%d bytes) "
  386. "for adding the request (%d bytes)\n",
  387. reserved_tail, ret);
  388. i915_gem_mark_busy(engine);
  389. engine->submit_request(request);
  390. }
  391. static unsigned long local_clock_us(unsigned int *cpu)
  392. {
  393. unsigned long t;
  394. /* Cheaply and approximately convert from nanoseconds to microseconds.
  395. * The result and subsequent calculations are also defined in the same
  396. * approximate microseconds units. The principal source of timing
  397. * error here is from the simple truncation.
  398. *
  399. * Note that local_clock() is only defined wrt to the current CPU;
  400. * the comparisons are no longer valid if we switch CPUs. Instead of
  401. * blocking preemption for the entire busywait, we can detect the CPU
  402. * switch and use that as indicator of system load and a reason to
  403. * stop busywaiting, see busywait_stop().
  404. */
  405. *cpu = get_cpu();
  406. t = local_clock() >> 10;
  407. put_cpu();
  408. return t;
  409. }
  410. static bool busywait_stop(unsigned long timeout, unsigned int cpu)
  411. {
  412. unsigned int this_cpu;
  413. if (time_after(local_clock_us(&this_cpu), timeout))
  414. return true;
  415. return this_cpu != cpu;
  416. }
  417. bool __i915_spin_request(const struct drm_i915_gem_request *req,
  418. int state, unsigned long timeout_us)
  419. {
  420. unsigned int cpu;
  421. /* When waiting for high frequency requests, e.g. during synchronous
  422. * rendering split between the CPU and GPU, the finite amount of time
  423. * required to set up the irq and wait upon it limits the response
  424. * rate. By busywaiting on the request completion for a short while we
  425. * can service the high frequency waits as quick as possible. However,
  426. * if it is a slow request, we want to sleep as quickly as possible.
  427. * The tradeoff between waiting and sleeping is roughly the time it
  428. * takes to sleep on a request, on the order of a microsecond.
  429. */
  430. timeout_us += local_clock_us(&cpu);
  431. do {
  432. if (i915_gem_request_completed(req))
  433. return true;
  434. if (signal_pending_state(state, current))
  435. break;
  436. if (busywait_stop(timeout_us, cpu))
  437. break;
  438. cpu_relax_lowlatency();
  439. } while (!need_resched());
  440. return false;
  441. }
  442. /**
  443. * __i915_wait_request - wait until execution of request has finished
  444. * @req: duh!
  445. * @interruptible: do an interruptible wait (normally yes)
  446. * @timeout: in - how long to wait (NULL forever); out - how much time remaining
  447. * @rps: client to charge for RPS boosting
  448. *
  449. * Note: It is of utmost importance that the passed in seqno and reset_counter
  450. * values have been read by the caller in an smp safe manner. Where read-side
  451. * locks are involved, it is sufficient to read the reset_counter before
  452. * unlocking the lock that protects the seqno. For lockless tricks, the
  453. * reset_counter _must_ be read before, and an appropriate smp_rmb must be
  454. * inserted.
  455. *
  456. * Returns 0 if the request was found within the alloted time. Else returns the
  457. * errno with remaining time filled in timeout argument.
  458. */
  459. int __i915_wait_request(struct drm_i915_gem_request *req,
  460. bool interruptible,
  461. s64 *timeout,
  462. struct intel_rps_client *rps)
  463. {
  464. int state = interruptible ? TASK_INTERRUPTIBLE : TASK_UNINTERRUPTIBLE;
  465. DEFINE_WAIT(reset);
  466. struct intel_wait wait;
  467. unsigned long timeout_remain;
  468. int ret = 0;
  469. might_sleep();
  470. if (list_empty(&req->list))
  471. return 0;
  472. if (i915_gem_request_completed(req))
  473. return 0;
  474. timeout_remain = MAX_SCHEDULE_TIMEOUT;
  475. if (timeout) {
  476. if (WARN_ON(*timeout < 0))
  477. return -EINVAL;
  478. if (*timeout == 0)
  479. return -ETIME;
  480. /* Record current time in case interrupted, or wedged */
  481. timeout_remain = nsecs_to_jiffies_timeout(*timeout);
  482. *timeout += ktime_get_raw_ns();
  483. }
  484. trace_i915_gem_request_wait_begin(req);
  485. /* This client is about to stall waiting for the GPU. In many cases
  486. * this is undesirable and limits the throughput of the system, as
  487. * many clients cannot continue processing user input/output whilst
  488. * blocked. RPS autotuning may take tens of milliseconds to respond
  489. * to the GPU load and thus incurs additional latency for the client.
  490. * We can circumvent that by promoting the GPU frequency to maximum
  491. * before we wait. This makes the GPU throttle up much more quickly
  492. * (good for benchmarks and user experience, e.g. window animations),
  493. * but at a cost of spending more power processing the workload
  494. * (bad for battery). Not all clients even want their results
  495. * immediately and for them we should just let the GPU select its own
  496. * frequency to maximise efficiency. To prevent a single client from
  497. * forcing the clocks too high for the whole system, we only allow
  498. * each client to waitboost once in a busy period.
  499. */
  500. if (IS_RPS_CLIENT(rps) && INTEL_GEN(req->i915) >= 6)
  501. gen6_rps_boost(req->i915, rps, req->emitted_jiffies);
  502. /* Optimistic spin for the next ~jiffie before touching IRQs */
  503. if (i915_spin_request(req, state, 5))
  504. goto complete;
  505. set_current_state(state);
  506. add_wait_queue(&req->i915->gpu_error.wait_queue, &reset);
  507. intel_wait_init(&wait, req->fence.seqno);
  508. if (intel_engine_add_wait(req->engine, &wait))
  509. /* In order to check that we haven't missed the interrupt
  510. * as we enabled it, we need to kick ourselves to do a
  511. * coherent check on the seqno before we sleep.
  512. */
  513. goto wakeup;
  514. for (;;) {
  515. if (signal_pending_state(state, current)) {
  516. ret = -ERESTARTSYS;
  517. break;
  518. }
  519. timeout_remain = io_schedule_timeout(timeout_remain);
  520. if (timeout_remain == 0) {
  521. ret = -ETIME;
  522. break;
  523. }
  524. if (intel_wait_complete(&wait))
  525. break;
  526. set_current_state(state);
  527. wakeup:
  528. /* Carefully check if the request is complete, giving time
  529. * for the seqno to be visible following the interrupt.
  530. * We also have to check in case we are kicked by the GPU
  531. * reset in order to drop the struct_mutex.
  532. */
  533. if (__i915_request_irq_complete(req))
  534. break;
  535. /* Only spin if we know the GPU is processing this request */
  536. if (i915_spin_request(req, state, 2))
  537. break;
  538. }
  539. remove_wait_queue(&req->i915->gpu_error.wait_queue, &reset);
  540. intel_engine_remove_wait(req->engine, &wait);
  541. __set_current_state(TASK_RUNNING);
  542. complete:
  543. trace_i915_gem_request_wait_end(req);
  544. if (timeout) {
  545. *timeout -= ktime_get_raw_ns();
  546. if (*timeout < 0)
  547. *timeout = 0;
  548. /*
  549. * Apparently ktime isn't accurate enough and occasionally has a
  550. * bit of mismatch in the jiffies<->nsecs<->ktime loop. So patch
  551. * things up to make the test happy. We allow up to 1 jiffy.
  552. *
  553. * This is a regrssion from the timespec->ktime conversion.
  554. */
  555. if (ret == -ETIME && *timeout < jiffies_to_usecs(1)*1000)
  556. *timeout = 0;
  557. }
  558. if (IS_RPS_USER(rps) &&
  559. req->fence.seqno == req->engine->last_submitted_seqno) {
  560. /* The GPU is now idle and this client has stalled.
  561. * Since no other client has submitted a request in the
  562. * meantime, assume that this client is the only one
  563. * supplying work to the GPU but is unable to keep that
  564. * work supplied because it is waiting. Since the GPU is
  565. * then never kept fully busy, RPS autoclocking will
  566. * keep the clocks relatively low, causing further delays.
  567. * Compensate by giving the synchronous client credit for
  568. * a waitboost next time.
  569. */
  570. spin_lock(&req->i915->rps.client_lock);
  571. list_del_init(&rps->link);
  572. spin_unlock(&req->i915->rps.client_lock);
  573. }
  574. return ret;
  575. }
  576. /**
  577. * Waits for a request to be signaled, and cleans up the
  578. * request and object lists appropriately for that event.
  579. */
  580. int i915_wait_request(struct drm_i915_gem_request *req)
  581. {
  582. int ret;
  583. GEM_BUG_ON(!req);
  584. lockdep_assert_held(&req->i915->drm.struct_mutex);
  585. ret = __i915_wait_request(req, req->i915->mm.interruptible, NULL, NULL);
  586. if (ret)
  587. return ret;
  588. /* If the GPU hung, we want to keep the requests to find the guilty. */
  589. if (!i915_reset_in_progress(&req->i915->gpu_error))
  590. i915_gem_request_retire_upto(req);
  591. return 0;
  592. }