xen: remove redundant initialization of variable ret

hiss2018 · bostrovs · commit bba91dd6d70b · 2020-10-04T18:41:33.000-05:00
After commit 9f51c05 ("pvcalls-front: Avoid get_free_pages(GFP_KERNEL) under spinlock"), the variable ret is being initialized with '-ENOMEM' that is meaningless. So remove it. Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com> Link: https://lore.kernel.org/r/20200919031702.32192-1-jingxiangfeng@huawei.com Reviewed-by: Juergen Gross <jgross@suse.com> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c
@@ -371,7 +371,7 @@ static int alloc_active_ring(struct sock_mapping *map)
 static int create_active(struct sock_mapping *map, evtchn_port_t *evtchn)
 {
 	void *bytes;
-	int ret = -ENOMEM, irq = -1, i;
+	int ret, irq = -1, i;
 
 	*evtchn = 0;
 	init_waitqueue_head(&map->active.inflight_conn_req);

Original file line number	Diff line number	Diff line change
`@@ -371,7 +371,7 @@ static int alloc_active_ring(struct sock_mapping *map)`
`371`	`371`	`static int create_active(struct sock_mapping map, evtchn_port_t evtchn)`
`372`	`372`	`{`
`373`	`373`	`void *bytes;`
`374`		`- int ret = -ENOMEM, irq = -1, i;`
	`374`	`+ int ret, irq = -1, i;`
`375`	`375`
`376`	`376`	`*evtchn = 0;`
`377`	`377`	`init_waitqueue_head(&map->active.inflight_conn_req);`