On Wed, 24 Oct 2012, William Dauchy wrote: > Signed-off-by: William Dauchy <will...@gandi.net> > --- > tools/kvm/virtio/pci.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/tools/kvm/virtio/pci.c b/tools/kvm/virtio/pci.c > index b6ac571..ab1119a 100644 > --- a/tools/kvm/virtio/pci.c > +++ b/tools/kvm/virtio/pci.c > @@ -245,7 +245,9 @@ static void virtio_pci__signal_msi(struct kvm *kvm, > struct virtio_pci *vpci, int > .data = vpci->msix_table[vec].msg.data, > }; > > - ioctl(kvm->vm_fd, KVM_SIGNAL_MSI, &msi); > + if (ioctl(kvm->vm_fd, KVM_SIGNAL_MSI, &msi) < 0) { > + pr_warning("Config pci device error: %s", strerror(errno)); > + } > }
Is this something that happens on your machine? I wonder if it makes more sense to return the error from virtio_pci__signal_vq() and virtio_pci__signal_config() and make the callers deal with it. Pekka -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html