From: Ben Skeggs <skeggsb@gmail.com>
Date: Fri, 21 Mar 2008 07:15:58 +0000 (+1100)
Subject: nouveau: use saved fence pointer, not head of unsignalled list
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=83e94189c424303fee4218a9d9380fc73b7ba318;p=mesa.git

nouveau: use saved fence pointer, not head of unsignalled list

It's possible for the unsignalled list head to change during fence_flush.
Fixes valgrind complaint exposed by a previous commit.
---

diff --git a/src/gallium/winsys/dri/nouveau/nouveau_fence.c b/src/gallium/winsys/dri/nouveau/nouveau_fence.c
index 7714e6f2485..e7b0b4ff079 100644
--- a/src/gallium/winsys/dri/nouveau/nouveau_fence.c
+++ b/src/gallium/winsys/dri/nouveau/nouveau_fence.c
@@ -172,14 +172,13 @@ nouveau_fence_flush(struct nouveau_channel *chan)
 		nvfence = nouveau_fence(nvchan->fence_head);
 		if (nvfence->sequence > sequence)
 			break;
-
 		nouveau_fence_del_unsignalled(&nvfence->base);
 		nvfence->signalled = 1;
 
 		if (nvfence->signal_cb) {
 			struct nouveau_fence *fence = NULL;
 
-			nouveau_fence_ref(nvchan->fence_head, &fence);
+			nouveau_fence_ref(&nvfence->base, &fence);
 
 			while (nvfence->signal_cb) {
 				struct nouveau_fence_cb *cb;