Re: [Qemu-devel] [PATCH 1/2] virtio-pci: build for uninitialized return value in vq_vector_unmask

[Anthony Liguori <aliguori@us.ibm.com>]

Michael Roth <mdroth@linux.vnet.ibm.com> writes:

> Fixes the following:
>
> /home/mdroth/w/qemu2.git/hw/virtio-pci.c: In function
> ‘kvm_virtio_pci_vector_unmask’:
> /home/mdroth/w/qemu2.git/hw/virtio-pci.c:673:12: error: ‘ret’ may be
> used uninitialized in this function [-Werror=uninitialized]
> cc1: all warnings being treated as errors
> make: *** [hw/virtio-pci.o] Error 1
> make: *** Waiting for unfinished jobs....
>
> Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

Looks like I didn't have libaio-devel installed on my new development
box :-/

Regards,

Anthony Liguori

> ---
>  hw/virtio-pci.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c
> index 0b49739..0934246 100644
> --- a/hw/virtio-pci.c
> +++ b/hw/virtio-pci.c
> @@ -616,7 +616,7 @@ static int kvm_virtio_pci_vq_vector_unmask(VirtIOPCIProxy *proxy,
>      VirtQueue *vq = virtio_get_queue(proxy->vdev, queue_no);
>      EventNotifier *n = virtio_queue_get_guest_notifier(vq);
>      VirtIOIRQFD *irqfd = &proxy->vector_irqfd[vector];
> -    int ret;
> +    int ret = 0;
>  
>      if (irqfd->msg.data != msg.data || irqfd->msg.address != msg.address) {
>          ret = kvm_irqchip_update_msi_route(kvm_state, irqfd->virq, msg);
> -- 
> 1.7.9.5