From 7aaa318fa3ec2a6ac02afb5f79a1b46982b07911 Mon Sep 17 00:00:00 2001
From: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Date: Tue, 15 Jan 2013 14:56:48 +0100
Subject: [PATCH] --- yaml --- r: 358159 b: refs/heads/master c:
 7a1863084c9d90ce4b67d645bf9b0f1612e68f62 h: refs/heads/master i:   358157:
 bdee74be0f0331cab755a8b3a318cc0e30839518   358155:
 1f3dfdfb97eb65706d9b023cd19914f32b2f830b   358151:
 9e982bbe419a2b3fb1f5cebaed30acffab828d6e   358143:
 8a426b6a1e547860c06c0bdb047af4adfdd2fda0 v: v3

---
 [refs]                                       |  2 +-
 trunk/drivers/gpu/drm/ttm/ttm_execbuf_util.c | 53 ++++++++------------
 2 files changed, 22 insertions(+), 33 deletions(-)

diff --git a/[refs] b/[refs]
index 17340601480f..3eb42fa1529b 100644
--- a/[refs]
+++ b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 63d0a4195560362e2e00a3ad38fc331d34e1da9b
+refs/heads/master: 7a1863084c9d90ce4b67d645bf9b0f1612e68f62
diff --git a/trunk/drivers/gpu/drm/ttm/ttm_execbuf_util.c b/trunk/drivers/gpu/drm/ttm/ttm_execbuf_util.c
index bd37b5cb8553..c7d323657798 100644
--- a/trunk/drivers/gpu/drm/ttm/ttm_execbuf_util.c
+++ b/trunk/drivers/gpu/drm/ttm/ttm_execbuf_util.c
@@ -82,22 +82,6 @@ static void ttm_eu_list_ref_sub(struct list_head *list)
 	}
 }
 
-static int ttm_eu_wait_unreserved_locked(struct list_head *list,
-					 struct ttm_buffer_object *bo)
-{
-	struct ttm_bo_global *glob = bo->glob;
-	int ret;
-
-	ttm_eu_del_from_lru_locked(list);
-	spin_unlock(&glob->lru_lock);
-	ret = ttm_bo_wait_unreserved(bo, true);
-	spin_lock(&glob->lru_lock);
-	if (unlikely(ret != 0))
-		ttm_eu_backoff_reservation_locked(list);
-	return ret;
-}
-
-
 void ttm_eu_backoff_reservation(struct list_head *list)
 {
 	struct ttm_validate_buffer *entry;
@@ -152,19 +136,23 @@ int ttm_eu_reserve_buffers(struct list_head *list)
 	list_for_each_entry(entry, list, head) {
 		struct ttm_buffer_object *bo = entry->bo;
 
-retry_this_bo:
 		ret = ttm_bo_reserve_nolru(bo, true, true, true, val_seq);
 		switch (ret) {
 		case 0:
 			break;
 		case -EBUSY:
-			ret = ttm_eu_wait_unreserved_locked(list, bo);
-			if (unlikely(ret != 0)) {
-				spin_unlock(&glob->lru_lock);
-				ttm_eu_list_ref_sub(list);
-				return ret;
-			}
-			goto retry_this_bo;
+			ttm_eu_del_from_lru_locked(list);
+			spin_unlock(&glob->lru_lock);
+			ret = ttm_bo_reserve_nolru(bo, true, false,
+						   true, val_seq);
+			spin_lock(&glob->lru_lock);
+			if (!ret)
+				break;
+
+			if (unlikely(ret != -EAGAIN))
+				goto err;
+
+			/* fallthrough */
 		case -EAGAIN:
 			ttm_eu_backoff_reservation_locked(list);
 			spin_unlock(&glob->lru_lock);
@@ -174,18 +162,13 @@ int ttm_eu_reserve_buffers(struct list_head *list)
 				return ret;
 			goto retry;
 		default:
-			ttm_eu_backoff_reservation_locked(list);
-			spin_unlock(&glob->lru_lock);
-			ttm_eu_list_ref_sub(list);
-			return ret;
+			goto err;
 		}
 
 		entry->reserved = true;
 		if (unlikely(atomic_read(&bo->cpu_writers) > 0)) {
-			ttm_eu_backoff_reservation_locked(list);
-			spin_unlock(&glob->lru_lock);
-			ttm_eu_list_ref_sub(list);
-			return -EBUSY;
+			ret = -EBUSY;
+			goto err;
 		}
 	}
 
@@ -194,6 +177,12 @@ int ttm_eu_reserve_buffers(struct list_head *list)
 	ttm_eu_list_ref_sub(list);
 
 	return 0;
+
+err:
+	ttm_eu_backoff_reservation_locked(list);
+	spin_unlock(&glob->lru_lock);
+	ttm_eu_list_ref_sub(list);
+	return ret;
 }
 EXPORT_SYMBOL(ttm_eu_reserve_buffers);