Re: [Qemu-devel] [PATCH v7 0/4] Generic PCIE-PCI Bridge

[Aleksandr Bezzubikov]

ср, 23 авг. 2017 г. в 5:46, Michael S. Tsirkin <address@hidden>:

> On Tue, Aug 22, 2017 at 02:43:39PM +0300, Marcel Apfelbaum wrote:
> > On 18/08/2017 2:36, Aleksandr Bezzubikov wrote:
> > > This series introduces a new device - Generic PCI Express to PCI
> bridge,
> > > and also makes all necessary changes to enable hotplug of the bridge
> itself
> > > and any device into the bridge.
> > >
> >
> > Hi,
> >
> > Series
> >    Tested-by: Marcel Apfelbaum <address@hidden>
> > (focused on changes from v6)
> >
> >
> > Michael, will Alecsandr need to re-send it after freeze?
>
> re-send or ping pls.


Michael, it seems that freeze is over, so you can pick this patches.


>
> > I am asking because the GSOC project is ending in a week or so.
> >
> >
> > Thanks,
> > Marcel
> >
> > > Changes v6->v7:
> > > Change IO/MEM/PREF reservation properties type to SIZE.
> > >
> > > Changes v5->v6:
> > > 1. Fix indentation in the cap creation function (addresses Marcel's
> comment)
> > > 2. Simplify capability pref_mem_* fields assignment (addresses
> Marcel's comment)
> > > 3. Documentation fixes:
> > >     - fix mutually exclusive fields definition (addresses Laszlo's
> comment)
> > >     - fix pcie-pci-bridge usage example (addresses Marcel's comment)
> > >
> > > Changes v4->v5:
> > > 1. Change PCIE-PCI Bridge license (addresses Marcel's comment)
> > > 2. The capability layout changes (adress Laszlo' comments):
> > >      - separate pref_mem into pref_mem_32 and pref_mem_64 fields
> (SeaBIOS side has the same changes)
> > >      - accordingly change the Generic Root Port's properties
> > > 3. Do not add the capability to the root port if no valid values are
> provided (adresses Michael's comment)
> > > 4. Rename the capability type to 'RESOURCE_RESERVE' (addresses
> Marcel's comment)
> > > 5. Remove shpc_present check function (addresses Marcel's comment)
> > > 6. Fix the 4th patch message (adresses Michael's comment)
> > > 7. Patch for SHPC enabling in _OSC method has been already merged
> > >
> > > Changes v3->v4:
> > > 1. PCIE-PCI Bridge device: "msi_enable"->"msi", "shpc"->"shpc_bar",
> remove local_err,
> > >     make "msi" property OnOffAuto, shpc_present() is still here
> > >     to avoid SHPC_VMSTATE refactoring (address Marcel's comments).
> > > 2. Change QEMU PCI capability layout (SeaBIOS side has the same
> changes):
> > >    - change reservation fields types: bus_res - uint32_t, others -
> uint64_t
> > >    - rename 'non_pref' and 'pref' fields
> > >    - interpret -1 value as 'ignore'
> > > 3. Use parent_realize in Generic PCI Express Root Port properly.
> > > 4. Fix documentation: fully replace the DMI-PCI bridge references with
> the new PCIE-PCI bridge,
> > > "PCIE"->"PCI Express", small mistakes and typos - address Laszlo's and
> Marcel's comments.
> > > 5. Rename QEMU PCI cap creation fucntion - addresses Marcel's comment.
> > >
> > > Changes v2->v3:
> > > (0). 'do_not_use' capability field flag is still _not_ in here since
> we haven't come to consesus on it yet.
> > > 1. Merge commits 5 (bus_reserve property creation) and 6 (property
> usage) together - addresses Michael's comment.
> > > 2. Add 'bus_reserve' property and QEMU PCI capability only to Generic
> PCIE Root Port - addresses Michael's and Marcel's comments.
> > > 3. Change 'bus_reserve' property's default value to 0 - addresses
> Michael's comment.
> > > 4. Rename QEMU bridge-specific PCI capability creation function -
> addresses Michael's comment.
> > > 5. Init the whole QEMU PCI capability with zeroes - addresses
> Michael's and Laszlo's comments.
> > > 6. Change QEMU PCI capability layout (SeaBIOS side has the same
> changes)
> > >    - add 'type' field to distinguish multiple
> > >      RedHat-specific capabilities - addresses Michael's comment
> > >    - do not mimiс PCI Config space register layout, but use mutually
> exclusive differently
> > >      sized fields for IO and prefetchable memory limits - addresses
> Laszlo's comment
> > > 7. Correct error handling in PCIE-PCI bridge realize function.
> > > 8. Replace a '2' constant with PCI_CAP_FLAGS in the capability
> creation function - addresses Michael's comment.
> > > 9. Remove a comment on _OSC which isn't correct anymore - address
> Marcel's comment.
> > > 10. Add documentation for the Generic PCIE-PCI Bridge and QEMU PCI
> capability - addresses Michael's comment.
> > >
> > > Changes v1->v2:
> > > 1. Enable SHPC for the bridge.
> > > 2. Enable SHPC support for the Q35 machine (ACPI stuff).
> > > 3. Introduce PCI capability to help firmware on the system init.
> > >     This allows the bridge to be hotpluggable. Now it's supported
> > >     only for pcie-root-port. Now it's supposed to used with
> > >     SeaBIOS only, look at the SeaBIOS corresponding series
> > >     "Allow RedHat PCI bridges reserve more buses than necessary during
> init".
> > >
> > > Aleksandr Bezzubikov (4):
> > >    hw/pci: introduce pcie-pci-bridge device
> > >    hw/pci: introduce bridge-only vendor-specific capability to provide
> > >      some hints to firmware
> > >    hw/pci: add QEMU-specific PCI capability to the Generic PCI Express
> > >      Root Port
> > >    docs: update documentation considering PCIE-PCI bridge
> > >
> > >   docs/pcie.txt                      |  49 +++++-----
> > >   docs/pcie_pci_bridge.txt           | 114 ++++++++++++++++++++++
> > >   hw/pci-bridge/Makefile.objs        |   2 +-
> > >   hw/pci-bridge/gen_pcie_root_port.c |  36 +++++++
> > >   hw/pci-bridge/pcie_pci_bridge.c    | 192
> +++++++++++++++++++++++++++++++++++++
> > >   hw/pci/pci_bridge.c                |  46 +++++++++
> > >   include/hw/pci/pci.h               |   1 +
> > >   include/hw/pci/pci_bridge.h        |  25 +++++
> > >   include/hw/pci/pcie_port.h         |   1 +
> > >   9 files changed, 442 insertions(+), 24 deletions(-)
> > >   create mode 100644 docs/pcie_pci_bridge.txt
> > >   create mode 100644 hw/pci-bridge/pcie_pci_bridge.c
> > >
>
-- 
Aleksandr Bezzubikov