On Mon, Jan 8, 2024 at 4:57 PM Ian Forbes wrote:
>
> Without this definition device errors will display the command name
> as (null) when debug logging is enabled.
>
> Signed-off-by: Ian Forbes
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
> b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
> index 36987ef3fc30..472c4821528f 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
> @@ -3603,6 +3603,8 @@ static const struct vmw_cmd_entry
> vmw_cmd_entries[SVGA_3D_CMD_MAX] = {
> _cmd_dx_bind_streamoutput, true, false, true),
> VMW_CMD_DEF(SVGA_3D_CMD_DX_DEFINE_RASTERIZER_STATE_V2,
> _cmd_dx_so_define, true, false, true),
> + VMW_CMD_DEF(SVGA_3D_CMD_DEFINE_GB_SURFACE_V4,
> + _cmd_invalid, false, false, true),
> };
>
> bool vmw_cmd_describe(const void *buf, u32 *size, char const **cmd)
> --
> 2.34.1
>
Looks good, but in the future you want to both find the change that
initially added SVGA_3D_CMD_DEFINE_GB_SURFACE_V4 and do "dim fixes" to
get the proper "Fixes:..." line to add to the commit description and
either use ./scripts/get_maintainer.pl or at least make sure the
maintainers are included in the patch by hand.
z
