Mesa (master): r600g/compute: Fix bug in compute memory pool

Tom Stellard tstellar at kemper.freedesktop.org
Fri Sep 6 00:28:00 UTC 2013


Module: Mesa
Branch: master
Commit: 505fad04f10eee1efdfcd8986b4d484b49d39986
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=505fad04f10eee1efdfcd8986b4d484b49d39986

Author: Tom Stellard <thomas.stellard at amd.com>
Date:   Mon Aug 26 13:06:53 2013 -0700

r600g/compute: Fix bug in compute memory pool

When adding a new buffer to the beginning of the memory pool, we were
accidentally deleting the buffer that was first in the buffer list.
This was caused by a bug in the memory pool's linked list
implementation.

---

 src/gallium/drivers/r600/compute_memory_pool.c |    9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src/gallium/drivers/r600/compute_memory_pool.c
index a02df84..75696d2 100644
--- a/src/gallium/drivers/r600/compute_memory_pool.c
+++ b/src/gallium/drivers/r600/compute_memory_pool.c
@@ -337,14 +337,9 @@ void compute_memory_finalize_pending(struct compute_memory_pool* pool,
 				}
 			} else {
 				/* Add item to the front of the list */
-				item->next = pool->item_list->next;
-				if (pool->item_list->next) {
-					pool->item_list->next->prev = item;
-				}
+				item->next = pool->item_list;
 				item->prev = pool->item_list->prev;
-				if (pool->item_list->prev) {
-					pool->item_list->prev->next = item;
-				}
+				pool->item_list->prev = item;
 				pool->item_list = item;
 			}
 		}




More information about the mesa-commit mailing list