[Qemu-devel] [PATCH v3 0/1] s390 pci infrastructure modelling

[Hong Bo Li]

This patch extends the current s390 pci implementation to
provide more flexibility in configuration of s390 specific
device handling.

For each vfio pci device, I create a zpci device to store s390
specific informations. And attach all of these special zpci devices
to the s390 facility bus. A zpci device references the corresponding
PCI device via device id. 

Compare to the old implementation, I moved the actual hotplug/unplug 
codes to s390 pci device hot plug function. Then in the pcihost 
hotplug function, we don't need to do anything special. In the pcihost
unplug function, we need to unplug the corresponding zpci device.

The new design allows to define multiple host bridges, each host bridge
could hold 32 zpci devices at most.

The topology for this implementation could be:

  dev: s390-pcihost, id ""
    bus: pci.0
      type PCI
      dev: vfio-pci, id "vpci1"
        host = "0000:00:00.0"
        ......
      dev: vfio-pci, id "vpci2"
        host = "0001:00:00.0"
        ......
  dev: s390-pci-facility, id ""
    bus: s390-pci-fac-bus.0
      type s390-pci-fac-bus
      dev: zpci, id "zpci1"
        fid = 1 (0x1)
        uid = 2 (0x2)
        pci_id = "vpci1"
      dev: zpci, id "zpci2"
        fid = 6 (0x6)
        uid = 7 (0x7)
        pci_id = "vpci2"

To make the review easier, I keep all of the old names, such as 
S390PCIBusDevice to name a zpci device. I will make a cleanup 
patch later to change these names to a more suitable name.

Comparing to v2, this patch does the following change:
1.use a link list to store all the zpci devices, and sort them by fid. 
2.add a flag "token_valid" to mark whether the list zpci instruction
  should restart or not.

Hong Bo Li (1):
  s390 pci infrastructure modelling

 hw/s390x/s390-pci-bus.c    | 316 +++++++++++++++++++++++++++++++++------------
 hw/s390x/s390-pci-bus.h    |  41 +++++-
 hw/s390x/s390-pci-inst.c   |  12 +-
 hw/s390x/s390-virtio-ccw.c |   5 +-
 4 files changed, 287 insertions(+), 87 deletions(-)

-- 
1.9.3