svga: add some comments in the svga_surface struct
authorBrian Paul <brianp@vmware.com>
Fri, 26 Aug 2016 00:04:51 +0000 (18:04 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 26 Aug 2016 20:20:19 +0000 (14:20 -0600)
Give more info about backing resources/surfaces.

Reviewed-by: Neha Bhende <bhenden@vmware.com>
src/gallium/drivers/svga/svga_surface.h

index c166791e1e7b240fa6cc4dec6329e52c5e8e613d..2f003b22cc60a255d2b90aaf624630dbfb02ebdd 100644 (file)
@@ -45,6 +45,12 @@ struct svga_surface
    struct pipe_surface base;
 
    struct svga_host_surface_cache_key key;
+
+   /*
+    * Note that the handle may point at a secondary / backing resource
+    * created by svga_texture_view_surface() which is something other
+    * than svga_texture(base->texture)->handle.
+    */
    struct svga_winsys_surface *handle;
 
    unsigned real_layer;
@@ -55,6 +61,16 @@ struct svga_surface
 
    /* VGPU10 */
    SVGA3dRenderTargetViewId view_id;
+
+   /*
+    * As with 'handle' above, this may point to a secondary / backing resource.
+    * We can't have one resource bound as both a render target and a shader
+    * resource at the same time.  But we sometimes want to do that, such as
+    * for mipmap generation where we sample from one level and render into
+    * another.
+    * In this situation, the backed surface is the render target while the
+    * original surface is the shader resource.
+    */
    struct svga_surface *backed;
 };