winsys/svga: Remove some unneeded debug code
authorThomas Hellstrom <thellstrom@vmware.com>
Wed, 19 Oct 2011 11:49:09 +0000 (13:49 +0200)
committerThomas Hellstrom <thellstrom@vmware.com>
Wed, 19 Oct 2011 13:33:26 +0000 (15:33 +0200)
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>
src/gallium/winsys/svga/drm/vmw_buffer.c

index 69f0787..66ab808 100644 (file)
@@ -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
 }