]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
drm/vmwgfx: Remove set but not used variable 'restart'
authorYueHaibing <yuehaibing@huawei.com>
Fri, 1 Mar 2019 17:47:43 +0000 (09:47 -0800)
committerDeepak Rawat <drawat@vmware.com>
Mon, 8 Apr 2019 17:29:04 +0000 (10:29 -0700)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c: In function 'vmw_cmdbuf_work_func':
drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c:514:7: warning:
 variable 'restart' set but not used [-Wunused-but-set-variable]

It not used any more after commit dc366364c4ef ("drm/vmwgfx: Fix multiple
command buffer context use")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Deepak Rawat <drawat@vmware.com>
Signed-off-by: Deepak Rawat <drawat@vmware.com>
Fixes: dc366364c4ef ("drm/vmwgfx: Fix multiple command buffer context use")
drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c

index 70dab55e78887220b9f235f28c4cf1f9096097c1..ed15655eacd23bd566515614e904a2ccfec3f156 100644 (file)
@@ -511,17 +511,14 @@ static void vmw_cmdbuf_work_func(struct work_struct *work)
                container_of(work, struct vmw_cmdbuf_man, work);
        struct vmw_cmdbuf_header *entry, *next;
        uint32_t dummy;
-       bool restart[SVGA_CB_CONTEXT_MAX];
        bool send_fence = false;
        struct list_head restart_head[SVGA_CB_CONTEXT_MAX];
        int i;
        struct vmw_cmdbuf_context *ctx;
        bool global_block = false;
 
-       for_each_cmdbuf_ctx(man, i, ctx) {
+       for_each_cmdbuf_ctx(man, i, ctx)
                INIT_LIST_HEAD(&restart_head[i]);
-               restart[i] = false;
-       }
 
        mutex_lock(&man->error_mutex);
        spin_lock(&man->lock);
@@ -533,7 +530,6 @@ static void vmw_cmdbuf_work_func(struct work_struct *work)
                const char *cmd_name;
 
                list_del_init(&entry->list);
-               restart[entry->cb_context] = true;
                global_block = true;
 
                if (!vmw_cmd_describe(header, &error_cmd_size, &cmd_name)) {