--- d10c85764ee3be2200a73da7045ff0fd21689cc3 +++ d43859f76db248a39a23d39cdb47d73aca8e5393 @@ -95,6 +95,11 @@ static struct pci_device_id virtio_pci_i MODULE_DEVICE_TABLE(pci, virtio_pci_id_table); +/* A PCI device has it's own struct device and so does a virtio device so + * we create a place for the virtio devices to show up in sysfs. I think it + * would make more sense for virtio to not insist on having it's own device. */ +static struct device *virtio_pci_root; + /* Convert a generic virtio device to our structure */ static struct virtio_pci_device *to_vp_device(struct virtio_device *vdev) { @@ -623,7 +628,7 @@ static int __devinit virtio_pci_probe(st if (vp_dev == NULL) return -ENOMEM; - vp_dev->vdev.dev.parent = &pci_dev->dev; + vp_dev->vdev.dev.parent = virtio_pci_root; vp_dev->vdev.dev.release = virtio_pci_release_dev; vp_dev->vdev.config = &virtio_pci_config_ops; vp_dev->pci_dev = pci_dev; @@ -647,7 +652,6 @@ static int __devinit virtio_pci_probe(st goto out_req_regions; pci_set_drvdata(pci_dev, vp_dev); - pci_set_master(pci_dev); /* we use the subsystem vendor/device id as the virtio vendor/device * id. this allows us to use the same PCI vendor/device id for all @@ -711,7 +715,17 @@ static struct pci_driver virtio_pci_driv static int __init virtio_pci_init(void) { - return pci_register_driver(&virtio_pci_driver); + int err; + + virtio_pci_root = root_device_register("virtio-pci"); + if (IS_ERR(virtio_pci_root)) + return PTR_ERR(virtio_pci_root); + + err = pci_register_driver(&virtio_pci_driver); + if (err) + root_device_unregister(virtio_pci_root); + + return err; } module_init(virtio_pci_init); @@ -719,6 +733,7 @@ module_init(virtio_pci_init); static void __exit virtio_pci_exit(void) { pci_unregister_driver(&virtio_pci_driver); + root_device_unregister(virtio_pci_root); } module_exit(virtio_pci_exit);