Re: [PATCH v3 0/4] Fix parsing our own XMLs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 28.04.2016 16:26, Martin Kletzander wrote:
> v3:
>  - Actually use virPCIDeviceAddress.  It was actually ACKed with that
>    changed but I feel like it's way too big of a change for just
>    pushing it.  Also, I'm in no rush with this, so I'l gladly wait
>    after therelease with this.
> 
> v2:
>  - Just a rebase
>  - I did *not* use virPCIDeviceAddress wording instead as discussed in
>    the v1 thread.  That's because we have lot of functions working
>    with virDevicePCIAddress named exactly after that and renaming
>    those would be ugly IMHO.  -
>    https://www.redhat.com/archives/libvir-list/2016-April/msg00714.html
> 
> v1:
>  - https://www.redhat.com/archives/libvir-list/2016-April/msg00081.html
> 
> 
> Martin Kletzander (4):
>   Change virDevicePCIAddress to virPCIDeviceAddress
>   Move capability formatting together
>   schemas: Update nodedev schema to match reality
>   conf: Parse more of our nodedev XML
> 
>  docs/schemas/nodedev.rng                           |  29 +++---
>  src/bhyve/bhyve_device.c                           |   4 +-
>  src/conf/device_conf.c                             |  16 +--
>  src/conf/device_conf.h                             |  25 ++---
>  src/conf/domain_addr.c                             |  24 ++---
>  src/conf/domain_addr.h                             |  22 ++--
>  src/conf/domain_conf.c                             |  26 ++---
>  src/conf/domain_conf.h                             |  10 +-
>  src/conf/network_conf.c                            |   8 +-
>  src/conf/network_conf.h                            |   2 +-
>  src/conf/node_device_conf.c                        | 111 +++++++++++++++++++--
>  src/conf/storage_conf.c                            |  14 +--
>  src/conf/storage_conf.h                            |   2 +-
>  src/libvirt_private.syms                           |   8 +-
>  src/network/bridge_driver.c                        |   4 +-
>  src/qemu/qemu_agent.c                              |   2 +-
>  src/qemu/qemu_domain_address.c                     |  12 +--
>  src/qemu/qemu_hotplug.c                            |  10 +-
>  src/qemu/qemu_monitor.c                            |  12 +--
>  src/qemu/qemu_monitor.h                            |  14 +--
>  src/qemu/qemu_monitor_json.c                       |  12 +--
>  src/qemu/qemu_monitor_json.h                       |  12 +--
>  src/qemu/qemu_monitor_text.c                       |  14 +--
>  src/qemu/qemu_monitor_text.h                       |  12 +--
>  src/storage/storage_backend_scsi.c                 |   2 +-
>  src/util/virpci.c                                  |   2 +-
>  src/util/virpci.h                                  |   3 +-
>  src/xen/xend_internal.c                            |   2 +-
>  .../pci_0000_00_1c_0_header_type.xml               |   2 +-
>  tests/nodedevschemadata/pci_0000_02_10_7_sriov.xml |  23 +++++
>  .../pci_0000_02_10_7_sriov_pf_vfs_all.xml          |  29 ++++++
>  ...i_0000_02_10_7_sriov_pf_vfs_all_header_type.xml |  30 ++++++
>  .../pci_0000_02_10_7_sriov_vfs.xml                 |  26 +++++
>  .../pci_0000_02_10_7_sriov_zero_vfs_max_count.xml  |  21 ++++
>  tests/nodedevxml2xmltest.c                         |   7 ++
>  35 files changed, 391 insertions(+), 161 deletions(-)
>  create mode 100644 tests/nodedevschemadata/pci_0000_02_10_7_sriov.xml
>  create mode 100644 tests/nodedevschemadata/pci_0000_02_10_7_sriov_pf_vfs_all.xml
>  create mode 100644 tests/nodedevschemadata/pci_0000_02_10_7_sriov_pf_vfs_all_header_type.xml
>  create mode 100644 tests/nodedevschemadata/pci_0000_02_10_7_sriov_vfs.xml
>  create mode 100644 tests/nodedevschemadata/pci_0000_02_10_7_sriov_zero_vfs_max_count.xml
> 

ACK series

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]