drm/vmwgfx: Add support for indirect and dispatch commands
authorDeepak Rawat <drawat.floss@gmail.com>
Thu, 13 Dec 2018 22:00:18 +0000 (14:00 -0800)
committerRoland Scheidegger <sroland@vmware.com>
Mon, 23 Mar 2020 21:39:34 +0000 (22:39 +0100)
Validate indirect and dispatch commands in command buffer.

Signed-off-by: Deepak Rawat <drawat.floss@gmail.com>
Reviewed-by: Thomas Hellström (VMware) <thomas_os@shipmail.org>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
Signed-off-by: Roland Scheidegger <sroland@vmware.com>
drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c

index ed56d9918c6a25b40472863e056397d727450d93..ecab6cbbc8d329fdda456484043432bd2ed1feb0 100644 (file)
@@ -2816,6 +2816,16 @@ static int vmw_cmd_intra_surface_copy(struct vmw_private *dev_priv,
                                 &cmd->body.surface.sid, NULL);
 }
 
+static int vmw_cmd_sm5(struct vmw_private *dev_priv,
+                      struct vmw_sw_context *sw_context,
+                      SVGA3dCmdHeader *header)
+{
+       if (!has_sm5_context(dev_priv))
+               return -EINVAL;
+
+       return 0;
+}
+
 static int vmw_cmd_sm5_view_define(struct vmw_private *dev_priv,
                                   struct vmw_sw_context *sw_context,
                                   SVGA3dCmdHeader *header)
@@ -2938,6 +2948,57 @@ static int vmw_cmd_set_cs_uav(struct vmw_private *dev_priv,
        return ret;
 }
 
+static int vmw_cmd_indexed_instanced_indirect(struct vmw_private *dev_priv,
+                                             struct vmw_sw_context *sw_context,
+                                             SVGA3dCmdHeader *header)
+{
+       struct vmw_draw_indexed_instanced_indirect_cmd {
+               SVGA3dCmdHeader header;
+               SVGA3dCmdDXDrawIndexedInstancedIndirect body;
+       } *cmd = container_of(header, typeof(*cmd), header);
+
+       if (!has_sm5_context(dev_priv))
+               return -EINVAL;
+
+       return vmw_cmd_res_check(dev_priv, sw_context, vmw_res_surface,
+                                VMW_RES_DIRTY_NONE, user_surface_converter,
+                                &cmd->body.argsBufferSid, NULL);
+}
+
+static int vmw_cmd_instanced_indirect(struct vmw_private *dev_priv,
+                                     struct vmw_sw_context *sw_context,
+                                     SVGA3dCmdHeader *header)
+{
+       struct vmw_draw_instanced_indirect_cmd {
+               SVGA3dCmdHeader header;
+               SVGA3dCmdDXDrawInstancedIndirect body;
+       } *cmd = container_of(header, typeof(*cmd), header);
+
+       if (!has_sm5_context(dev_priv))
+               return -EINVAL;
+
+       return vmw_cmd_res_check(dev_priv, sw_context, vmw_res_surface,
+                                VMW_RES_DIRTY_NONE, user_surface_converter,
+                                &cmd->body.argsBufferSid, NULL);
+}
+
+static int vmw_cmd_dispatch_indirect(struct vmw_private *dev_priv,
+                                    struct vmw_sw_context *sw_context,
+                                    SVGA3dCmdHeader *header)
+{
+       struct vmw_dispatch_indirect_cmd {
+               SVGA3dCmdHeader header;
+               SVGA3dCmdDXDispatchIndirect body;
+       } *cmd = container_of(header, typeof(*cmd), header);
+
+       if (!has_sm5_context(dev_priv))
+               return -EINVAL;
+
+       return vmw_cmd_res_check(dev_priv, sw_context, vmw_res_surface,
+                                VMW_RES_DIRTY_NONE, user_surface_converter,
+                                &cmd->body.argsBufferSid, NULL);
+}
+
 static int vmw_cmd_check_not_3d(struct vmw_private *dev_priv,
                                struct vmw_sw_context *sw_context,
                                void *buf, uint32_t *size)
@@ -3303,8 +3364,17 @@ static const struct vmw_cmd_entry vmw_cmd_entries[SVGA_3D_CMD_MAX] = {
                    false, true),
        VMW_CMD_DEF(SVGA_3D_CMD_DX_SET_UA_VIEWS, &vmw_cmd_set_uav, true, false,
                    true),
+       VMW_CMD_DEF(SVGA_3D_CMD_DX_DRAW_INDEXED_INSTANCED_INDIRECT,
+                   &vmw_cmd_indexed_instanced_indirect, true, false, true),
+       VMW_CMD_DEF(SVGA_3D_CMD_DX_DRAW_INSTANCED_INDIRECT,
+                   &vmw_cmd_instanced_indirect, true, false, true),
+       VMW_CMD_DEF(SVGA_3D_CMD_DX_DISPATCH, &vmw_cmd_sm5, true, false, true),
+       VMW_CMD_DEF(SVGA_3D_CMD_DX_DISPATCH_INDIRECT,
+                   &vmw_cmd_dispatch_indirect, true, false, true),
        VMW_CMD_DEF(SVGA_3D_CMD_DX_SET_CS_UA_VIEWS, &vmw_cmd_set_cs_uav, true,
                    false, true),
+       VMW_CMD_DEF(SVGA_3D_CMD_DX_DEFINE_DEPTHSTENCIL_VIEW_V2,
+                   &vmw_cmd_sm5_view_define, true, false, true),
 };
 
 bool vmw_cmd_describe(const void *buf, u32 *size, char const **cmd)