On Thu, Nov 22, 2012 at 03:50:51PM +0100, fred.kon...@greensocs.com wrote: > +static const struct VirtioBusInfo virtio_bus_info = { > + .init_cb = virtio_pci_init_cb, > + .exit_cb = virtio_pci_exit_cb, > + > + .virtio_bindings = {
Eventually VirtIOBindings can probably be inlined into VirtioBusInfo. I don't see a need for separate structs. > + .notify = virtio_pci_notify, > + .save_config = virtio_pci_save_config, > + .load_config = virtio_pci_load_config, > + .save_queue = virtio_pci_save_queue, > + .load_queue = virtio_pci_load_queue, > + .get_features = virtio_pci_get_features, > + .query_guest_notifiers = virtio_pci_query_guest_notifiers, > + .set_host_notifier = virtio_pci_set_host_notifier, > + .set_guest_notifiers = virtio_pci_set_guest_notifiers, > + .vmstate_change = virtio_pci_vmstate_change, > + } > +};