From: Thomas Hellstrom <thellstrom@vmware.com>
Date: Wed, 19 Oct 2011 11:49:09 +0000 (+0200)
Subject: winsys/svga: Remove some unneeded debug code
X-Git-Tag: mesa-8.0-rc1~1632
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=60f67cd1f17d13c0be3caa9a8c0b9a4600970230;p=platform%2Fupstream%2Fmesa.git

winsys/svga: Remove some unneeded debug code

This code isn't really relevant since the kernel takes care not
to destroy busy GMR buffers.

Also with the advent of fence objects, the code was incorrect since
it didn't refcount fence handles.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Jakob Bornecrantz <jakob@vmware.com>
---

diff --git a/src/gallium/winsys/svga/drm/vmw_buffer.c b/src/gallium/winsys/svga/drm/vmw_buffer.c
index 69f0787..66ab808 100644
--- a/src/gallium/winsys/svga/drm/vmw_buffer.c
+++ b/src/gallium/winsys/svga/drm/vmw_buffer.c
@@ -63,10 +63,6 @@ struct vmw_gmr_buffer
    
    struct vmw_region *region;
    void *map;
-   
-#ifdef DEBUG
-   struct pipe_fence_handle *last_fence;
-#endif
 };
 
 
@@ -103,13 +99,6 @@ vmw_gmr_buffer_destroy(struct pb_buffer *_buf)
 {
    struct vmw_gmr_buffer *buf = vmw_gmr_buffer(_buf);
 
-#ifdef DEBUG
-   if(buf->last_fence) {
-      struct svga_winsys_screen *sws = &buf->mgr->vws->base;
-      assert(sws->fence_signalled(sws, buf->last_fence, 0) == 0);
-   }
-#endif
-
    vmw_ioctl_region_unmap(buf->region);
    
    vmw_ioctl_region_destroy(buf->region);
@@ -162,11 +151,6 @@ vmw_gmr_buffer_fence( struct pb_buffer *_buf,
 {
    /* We don't need to do anything, as the pipebuffer library
     * will take care of delaying the destruction of fenced buffers */  
-#ifdef DEBUG
-   struct vmw_gmr_buffer *buf = vmw_gmr_buffer(_buf);
-   if(fence)
-      buf->last_fence = fence;
-#endif
 }