drm/ttm: Simplify ttm_eu_reserve_buffers()
authorTom St Denis <tom.stdenis@amd.com>
Fri, 26 Jan 2018 15:13:42 +0000 (10:13 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 19 Feb 2018 19:19:00 +0000 (14:19 -0500)
Hoist the comparison of the ret to -EDEADLK above
the two code paths to simplify the function.

Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/ttm/ttm_execbuf_util.c

index 373ced0..fa44f7b 100644 (file)
@@ -139,12 +139,14 @@ int ttm_eu_reserve_buffers(struct ww_acquire_ctx *ticket,
                 */
                ttm_eu_backoff_reservation_reverse(list, entry);
 
-               if (ret == -EDEADLK && intr) {
-                       ret = ww_mutex_lock_slow_interruptible(&bo->resv->lock,
-                                                              ticket);
-               } else if (ret == -EDEADLK) {
-                       ww_mutex_lock_slow(&bo->resv->lock, ticket);
-                       ret = 0;
+               if (ret == -EDEADLK) {
+                       if (intr) {
+                               ret = ww_mutex_lock_slow_interruptible(&bo->resv->lock,
+                                                                      ticket);
+                       } else {
+                               ww_mutex_lock_slow(&bo->resv->lock, ticket);
+                               ret = 0;
+                       }
                }
 
                if (!ret && entry->shared)