• M
    Revert "PCI: Convert pci_device_hot_add() to QObject" · 6c6a58ae
    Markus Armbruster 提交于
    Short story: We don't want pci_add in QMP.  Long story follows.
    
    pci_add can do two things:
    
    * Hot plug a PCI NIC.  device_add is more general.
    
    * Hot plug a PCI disk controller, and a drive connected to it.
    
      The controller is either virtio-blk-pci (if=virtio) or lsi53c895a
      (if=scsi).  With the latter, the drive is optional.  Use drive_add to
      hotplug additional SCSI drives.  Except drive_add is not available in
      QMP.
    
      device_add is more general for controllers and the guest part of
      drives.  I'm working on a more general alternative for the host part
      of drives.
    
    Why am I proposing to remove pci_add from QMP before its replacement is
    ready?  I want it out sooner rather than later, because it isn't fully
    functional (errors and drive_add are missing), and we do not plan to
    complete the job.  In other words, it's not really usable over QMP now,
    and it's not what we want for QMP anyway.  Since we don't want it to be
    used over QMP, we should take it out, not leave it around as a trap for
    the uninitiated.
    
    Dan Berrange confirmed that libvirt has no need for pci_add & friends
    over QMP.
    
    This reverts commit 7a344f7a.
    
    Conflicts:
    
    	hw/pci-hotplug.c
    	sysemu.h
    Signed-off-by: NMarkus Armbruster <armbru@redhat.com>
    6c6a58ae
sysemu.h 6.6 KB