Re: [PATCH for 1.3.3 0/5] Fix parsing our own XMLs

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

 



On 03.04.2016 21:27, Martin Kletzander wrote:
> Laine found out that he can't do 'virsh nodedev-detach pci_dev'
> because of some strange error message.  That was caused by my commit,
> but also by all the previous ones that skipped adding tests and
> parsing of new functions.  In order for this to work in 1.3.3, we need
> either a) only the first patch or b) all of them.  I specifically
> created the first one so that it has enough in itself to fix the
> problem and we don't need to push more and more patches into the
> release.  We can push the rest after release.  If someone wants to
> have all in for 1.3.3, well, I hope I added enough tests for that ;)
> 
> 
> Martin Kletzander (5):
>   nodedev: Fix parsing of generated XMLs
>   Change virPCIDeviceAddress to virDevicePCIAddress
>   Move capability formatting together
>   schemas: Update nodedev schema to match reality
>   conf: Parse more of our nodedev XML
> 
>  docs/schemas/nodedev.rng                           |  29 +++--
>  src/conf/device_conf.h                             |  11 +-
>  src/conf/node_device_conf.c                        | 121 ++++++++++++++++++---
>  src/conf/node_device_conf.h                        |   6 +-
>  src/libvirt_private.syms                           |  10 +-
>  src/network/bridge_driver.c                        |   4 +-
>  src/node_device/node_device_linux_sysfs.c          |   6 +-
>  src/util/virhostdev.c                              |  12 +-
>  src/util/virnetdev.c                               |   4 +-
>  src/util/virnetdev.h                               |   2 +-
>  src/util/virpci.c                                  |  80 +++++++-------
>  src/util/virpci.h                                  |  29 ++---
>  .../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                         |   5 +
>  19 files changed, 333 insertions(+), 117 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
> 

I like this. I really do. And if we were not in the freeze, I would ACK
this straight away. But since we are in the freeze, we should push only
(small) bug fixes. And I think the minimum needed here is 1/5, right?
Therefore, I'd suggest pushing that one and saving the rest for after
the release. Is that okay with you?

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]