Linux Virtualization - Feb 2020 - [PATCH] virtio_balloon: Fix build error seen with CONFIG_BALLOON_COMPACTION=n

0day reports:

drivers//virtio/virtio_balloon.c: In function 'virtballoon_probe':
drivers//virtio/virtio_balloon.c:960:1: error:
	label 'out_del_vqs' defined but not used [-Werror=unused-label]

This is seen with CONFIG_BALLOON_COMPACTION=n.

Reported-by: kbuild test robot <lkp at intel.com>
Fixes: 1ad6f58ea936 ("virtio_balloon: Fix memory leaks on errors in
virtballoon_probe()")
Cc: David Hildenbrand <david at redhat.com>
Cc: Michael S. Tsirkin <mst at redhat.com>
Signed-off-by: Guenter Roeck <linux at roeck-us.net>
---
 drivers/virtio/virtio_balloon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c
index 7bfe365d9372..341458fd95ca 100644
--- a/drivers/virtio/virtio_balloon.c
+++ b/drivers/virtio/virtio_balloon.c
@@ -959,8 +959,8 @@ static int virtballoon_probe(struct virtio_device *vdev)
 	iput(vb->vb_dev_info.inode);
 out_kern_unmount:
 	kern_unmount(balloon_mnt);
-#endif
 out_del_vqs:
+#endif
 	vdev->config->del_vqs(vdev);
 out_free_vb:
 	kfree(vb);
-- 
2.17.1

> Am 20.02.2020 um 03:32 schrieb Guenter Roeck <linux at roeck-us.net>:
> 
> ?0day reports:
> 
> drivers//virtio/virtio_balloon.c: In function 'virtballoon_probe':
> drivers//virtio/virtio_balloon.c:960:1: error:
>    label 'out_del_vqs' defined but not used [-Werror=unused-label]
> 
> This is seen with CONFIG_BALLOON_COMPACTION=n.
> 
> Reported-by: kbuild test robot <lkp at intel.com>
> Fixes: 1ad6f58ea936 ("virtio_balloon: Fix memory leaks on errors in
virtballoon_probe()")
> Cc: David Hildenbrand <david at redhat.com>
> Cc: Michael S. Tsirkin <mst at redhat.com>
> Signed-off-by: Guenter Roeck <linux at roeck-us.net>
> ---
There is already a patch on the list to fix that (and it looks exactly like this
one :) ). Unfortunately, not picked up yet.

Thanks!