svga: simplify failure code in emit_rss_vgpu9()
authorBrian Paul <brianp@vmware.com>
Fri, 29 Dec 2017 17:03:43 +0000 (10:03 -0700)
committerBrian Paul <brianp@vmware.com>
Thu, 11 Jan 2018 15:06:38 +0000 (08:06 -0700)
No need for a goto.

Reviewed-by: Neha Bhende <bhenden@vmware.com>
Reviewed-by: Charmaine Lee <charmainel@vmware.com>
src/gallium/drivers/svga/svga_state_rss.c

index 515155d3ae8cf410ac6330f212935bf15eaa26d1..0cf8be8efbd3369e23674705096ba3875cee2b3f 100644 (file)
@@ -275,29 +275,24 @@ emit_rss_vgpu9(struct svga_context *svga, unsigned dirty)
    if (queue.rs_count) {
       SVGA3dRenderState *rs;
 
-      if (SVGA3D_BeginSetRenderState(svga->swc,
-                                      &rs,
-                                      queue.rs_count) != PIPE_OK)
-         goto fail;
+      if (SVGA3D_BeginSetRenderState(svga->swc, &rs, queue.rs_count)
+          != PIPE_OK) {
+         /* XXX: need to poison cached hardware state on failure to ensure
+          * dirty state gets re-emitted.  Fix this by re-instating partial
+          * FIFOCommit command and only updating cached hw state once the
+          * initial allocation has succeeded.
+          */
+         memset(svga->state.hw_draw.rs, 0xcd, sizeof(svga->state.hw_draw.rs));
+
+         return PIPE_ERROR_OUT_OF_MEMORY;
+      }
 
-      memcpy(rs,
-              queue.rs,
-              queue.rs_count * sizeof queue.rs[0]);
+      memcpy(rs, queue.rs, queue.rs_count * sizeof queue.rs[0]);
 
       SVGA_FIFOCommitAll(svga->swc);
    }
 
    return PIPE_OK;
-
-fail:
-   /* XXX: need to poison cached hardware state on failure to ensure
-    * dirty state gets re-emitted.  Fix this by re-instating partial
-    * FIFOCommit command and only updating cached hw state once the
-    * initial allocation has succeeded.
-    */
-   memset(svga->state.hw_draw.rs, 0xcd, sizeof(svga->state.hw_draw.rs));
-
-   return PIPE_ERROR_OUT_OF_MEMORY;
 }