Re: [PATCH] xen/pvcalls: remove redundant check for irq >= 0
On 11/03/2017 06:01 AM, Juergen Gross wrote: > On 03/11/17 10:20, Colin King wrote: >> From: Colin Ian King>> >> This is a moot point, but irq is always less than zero at the label >> out_error, so the check for irq >= 0 is redundant and can be removed. >> >> Detected by CoverityScan, CID#1460371 ("Logically dead code") >> >> Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") >> Signed-off-by: Colin Ian King > Reviewed-by: Juergen Gross Applied to for-linus-4.15. -boris
Re: [PATCH] xen/pvcalls: remove redundant check for irq >= 0
On 11/03/2017 06:01 AM, Juergen Gross wrote: > On 03/11/17 10:20, Colin King wrote: >> From: Colin Ian King >> >> This is a moot point, but irq is always less than zero at the label >> out_error, so the check for irq >= 0 is redundant and can be removed. >> >> Detected by CoverityScan, CID#1460371 ("Logically dead code") >> >> Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") >> Signed-off-by: Colin Ian King > Reviewed-by: Juergen Gross Applied to for-linus-4.15. -boris
Re: [PATCH] xen/pvcalls: remove redundant check for irq >= 0
On 03/11/17 10:20, Colin King wrote: > From: Colin Ian King> > This is a moot point, but irq is always less than zero at the label > out_error, so the check for irq >= 0 is redundant and can be removed. > > Detected by CoverityScan, CID#1460371 ("Logically dead code") > > Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") > Signed-off-by: Colin Ian King Reviewed-by: Juergen Gross Juergen
Re: [PATCH] xen/pvcalls: remove redundant check for irq >= 0
On 03/11/17 10:20, Colin King wrote: > From: Colin Ian King > > This is a moot point, but irq is always less than zero at the label > out_error, so the check for irq >= 0 is redundant and can be removed. > > Detected by CoverityScan, CID#1460371 ("Logically dead code") > > Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") > Signed-off-by: Colin Ian King Reviewed-by: Juergen Gross Juergen
[PATCH] xen/pvcalls: remove redundant check for irq >= 0
From: Colin Ian KingThis is a moot point, but irq is always less than zero at the label out_error, so the check for irq >= 0 is redundant and can be removed. Detected by CoverityScan, CID#1460371 ("Logically dead code") Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") Signed-off-by: Colin Ian King --- drivers/xen/pvcalls-front.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c index de8a470351a5..b08569998046 100644 --- a/drivers/xen/pvcalls-front.c +++ b/drivers/xen/pvcalls-front.c @@ -351,9 +351,7 @@ static int create_active(struct sock_mapping *map, int *evtchn) return 0; out_error: - if (irq >= 0) - unbind_from_irqhandler(irq, map); - else if (*evtchn >= 0) + if (*evtchn >= 0) xenbus_free_evtchn(pvcalls_front_dev, *evtchn); kfree(map->active.data.in); kfree(map->active.ring); -- 2.14.1
[PATCH] xen/pvcalls: remove redundant check for irq >= 0
From: Colin Ian King This is a moot point, but irq is always less than zero at the label out_error, so the check for irq >= 0 is redundant and can be removed. Detected by CoverityScan, CID#1460371 ("Logically dead code") Fixes: cb1c7d9bbc87 ("xen/pvcalls: implement connect command") Signed-off-by: Colin Ian King --- drivers/xen/pvcalls-front.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c index de8a470351a5..b08569998046 100644 --- a/drivers/xen/pvcalls-front.c +++ b/drivers/xen/pvcalls-front.c @@ -351,9 +351,7 @@ static int create_active(struct sock_mapping *map, int *evtchn) return 0; out_error: - if (irq >= 0) - unbind_from_irqhandler(irq, map); - else if (*evtchn >= 0) + if (*evtchn >= 0) xenbus_free_evtchn(pvcalls_front_dev, *evtchn); kfree(map->active.data.in); kfree(map->active.ring); -- 2.14.1