On Tue, Jan 08, 2008 at 09:42:13AM -0600, Anthony Liguori wrote: > Marcelo Tosatti wrote: > >Following patch introduces a KVM guest balloon driver. Communication > >to/from the host is performed via virtio. > >
I'll address the other comments. > >+ virtballoon.dev = vdev; > >+ init_waitqueue_head(&virtballoon.balloon_wait); > >+ init_waitqueue_head(&virtballoon.rmmod_wait); > >+ atomic_set(&virtballoon.inflight_bufs, 0); > >+ > >+ err = request_irq(pvdev->pci_dev->irq, balloon_irq, IRQF_SHARED, > >+ pvdev->vdev.dev.bus_id, &virtballoon); > >+ if (err) > >+ goto out_free_vq; > > > > Why is it taking over the irq? This is very, very wrong. A virtio > device cannot be dependent on being used on top of the virtio-pci backend. A notification is necessary whenever the host changes the target value in the config space. So right now this notification is sharing the same IRQ as the main PCI device. Do you have any suggestion on how to retrieve the IRQ of the virtio device, or some other notification mechanism? _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linux-foundation.org/mailman/listinfo/virtualization
