[libvirt] [PATCHv3 00/13] Add new PCIe controllers

Laine Stump laine at laine.org
Sat Jul 25 19:58:24 UTC 2015


Since the first 3 patches of V2 were ACKed and uncontroversial, I
fixed the small problems pointed out in the reviews and pushed
them. Thus, Patches 01-13 here correspond to Patches 04-17 in V2.

Most of these patches were already ACKed in V2 (pending my making
small fixes pointed out in review), but the main things that need
review are:

1) changing of model name from a char* to an enum in Patch 01, and
   corresponding blowback in patches 02, 07, 10, and 13

2) range checking of chassisNr in Patch 03 and chassis+port in patch
   06.

3) check for duplicate <model> in patch 01, and duplicate <target> in
   patch 03.

I did add one new negative test, and reworded some documentation, but I'm
about to go mostly offline for 10 days, and would rather not have
these patches bitrotting during that time if they are okay other than
that. (also, I see both of those tasks as having no practical end, but
do give my word to add more to both in later followups).

If by chance everything is ACKed before DV freezes for RC1, but after
I'm already offline (which will happen Sunday morning U.S. east coast
time), I would appreciate if the reviewer could push the patches so
they'll get the RC testing and be in the 1.2.18 release. (If not, I'll
take care of it when I return).

P.S. I ran "git rebase -i master -x "make -j8 check && make -j8
syntax-check" before sending.

Laine Stump (13):
  conf: add new <model> subelement with name attribute to <controller>
  qemu: implement <model> subelement to <controller>
  conf: add new <target> subelement with chassisNr attribute to
    <controller>
  qemu: implement <target chassisNr='n'/> subelement/attribute of
    <controller>
  qemu: add capabilities bit for device ioh3420
  conf: new pci controller model "pcie-root-port"
  qemu: support new pci controller model "pcie-root-port"
  qemu: add capabilities bit for device x3130-upstream
  conf: new pci controller model "pcie-switch-upstream-port"
  qemu: support new pci controller model "pcie-switch-upstream-port"
  qemu: add capabilities bit for device xio3130-downstream
  conf: new pcie-controller model "pcie-switch-downstream-port"
  qemu: support new pci controller model "pcie-switch-downstream-port"

 docs/formatdomain.html.in                          |  90 +++++++-
 docs/schemas/domaincommon.rng                      |  42 ++++
 src/conf/domain_addr.c                             |  32 ++-
 src/conf/domain_addr.h                             |  12 +-
 src/conf/domain_conf.c                             | 162 +++++++++++++-
 src/conf/domain_conf.h                             |  34 +++
 src/libvirt_private.syms                           |   2 +
 src/qemu/qemu_capabilities.c                       |   8 +-
 src/qemu/qemu_capabilities.h                       |   5 +-
 src/qemu/qemu_command.c                            | 238 ++++++++++++++++++++-
 tests/qemucapabilitiesdata/caps_1.2.2-1.caps       |   3 +
 tests/qemucapabilitiesdata/caps_1.3.1-1.caps       |   3 +
 tests/qemucapabilitiesdata/caps_1.4.2-1.caps       |   3 +
 tests/qemucapabilitiesdata/caps_1.5.3-1.caps       |   3 +
 tests/qemucapabilitiesdata/caps_1.6.0-1.caps       |   3 +
 tests/qemucapabilitiesdata/caps_1.6.50-1.caps      |   3 +
 tests/qemucapabilitiesdata/caps_2.1.1-1.caps       |   3 +
 tests/qemuhelptest.c                               |  10 +-
 .../qemuxml2argv-pcie-root-port-too-many.xml       |  60 ++++++
 .../qemuxml2argv-pcie-root-port.args               |  10 +
 .../qemuxml2argv-pcie-root-port.xml                |  36 ++++
 .../qemuxml2argv-pcie-switch-downstream-port.args  |  18 ++
 .../qemuxml2argv-pcie-switch-downstream-port.xml   |  44 ++++
 .../qemuxml2argv-pcie-switch-upstream-port.args    |  12 ++
 .../qemuxml2argv-pcie-switch-upstream-port.xml     |  37 ++++
 tests/qemuxml2argvdata/qemuxml2argv-q35.args       |   2 +-
 tests/qemuxml2argvdata/qemuxml2argv-q35.xml        |   9 +-
 tests/qemuxml2argvtest.c                           |  33 +++
 tests/qemuxml2xmloutdata/qemuxml2xmlout-q35.xml    |   9 +-
 tests/qemuxml2xmltest.c                            |   4 +
 30 files changed, 909 insertions(+), 21 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-root-port-too-many.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-root-port.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-root-port.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-switch-downstream-port.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-switch-downstream-port.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-switch-upstream-port.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-pcie-switch-upstream-port.xml

-- 
2.1.0




More information about the libvir-list mailing list