Re: [Mesa-dev] [PATCH] r600g/compute: Fix bug in compute memory pool

2013-08-28 Thread Aaron Watry
The changes look good to me... That seems to be a much more sane way to add the item to the beginning of the linked list. I've tested this on CEDAR (Radeon 5400) without any OpenCL regressions, and the only piglit change was that the new piglit test created for this bug now passes. --Aaron On T

[Mesa-dev] [PATCH] r600g/compute: Fix bug in compute memory pool

2013-08-27 Thread Tom Stellard
From: Tom Stellard 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 ++---