[libvirt] [PATCH 00/16] Xen: remove xend config version
Jim Fehlig
jfehlig at suse.com
Fri Dec 18 04:57:51 UTC 2015
On 12/17/2015 05:42 AM, Michal Privoznik wrote:
> On 15.12.2015 22:32, Jim Fehlig wrote:
>> Hi All,
>>
>> Ian Campbell recently attempted [1] to fix and issue around MAX_VIRT_VPUS
>> on ARM that required adding a new XEND_CONFIG_VERSION. After some
>> discussion [2] we decided to drop support for all of the old xend config
>> versions and go with the version supported in Xen 4.0, since the xl syntax
>> was originally based on (and intended to be compatible with) xm circa that
>> point in time.
>>
>> This series removes all traces of xend config version from the codebase,
>> essentially removing support for Xen 3.x. Hopefully I succeeding in making
>> the rather large series reviewable. The series is also available on the
>> remove-xend-config-version branch of my libvirt github clone [2].
>>
>> [1] https://www.redhat.com/archives/libvir-list/2015-November/msg01153.html
>> [2] https://www.redhat.com/archives/libvir-list/2015-December/msg00148.html
>> [3] https://github.com/jfehlig/libvirt/tree/remove-xend-config-version
>>
>> Jim Fehlig (16):
>> Xen: tests: remove old xm config tests
>> Xen: tests: remove net-ioemu xm config test
>> Xen: tests: remove old sexpr2xml tests
>> Xen: tests: remove old xml2sexpr tests
>> Xen: tests: use latest XEND_CONFIG_VERSION in xm/xl tests
>> Xen: xenconfig: remove XEND_CONFIG_VERSION in common code
>> Xen: xenconfig: remove use of XEND_CONFIG_VERSION in xen_xm
>> Xen: xenconfig: remove xendConfigVersion from public functions
>> Xen: tests: use latest XEND_CONFIG_VERSION in sexpr2xml tests
>> Xen: xenconfig: remove disks from '(image)' sexpr
>> Xen: tests: use latest XEND_CONFIG_VERSION in xml2sexpr tests
>> Xen: xenconfig: remove use of XEND_CONFIG_VERSION in xen_sxpr
>> Xen: xen_driver: remove use of XEND_CONFIG_VERSION
>> Xen: xend: remove use of XEND_CONFIG_VERSION
>> Xen: xenconfig: remove xendConfigVersion from public sexpr functions
>> Xen: remove xendConfigVersion from driver private struct
>>
>> src/libxl/libxl_driver.c | 9 +-
>> src/xen/xen_driver.c | 296 ++++-----------
>> src/xen/xen_driver.h | 2 -
>> src/xen/xend_internal.c | 224 ++---------
>> src/xen/xm_internal.c | 9 +-
>> src/xenconfig/xen_common.c | 211 ++++-------
>> src/xenconfig/xen_common.h | 7 +-
>> src/xenconfig/xen_sxpr.c | 411 ++++++---------------
>> src/xenconfig/xen_sxpr.h | 21 +-
>> src/xenconfig/xen_xl.c | 9 +-
>> src/xenconfig/xen_xl.h | 7 +-
>> src/xenconfig/xen_xm.c | 57 +--
>> src/xenconfig/xen_xm.h | 5 +-
>> src/xenconfig/xenxs_private.h | 8 -
>> tests/sexpr2xmldata/sexpr2xml-fv-autoport.xml | 2 +-
>> .../sexpr2xmldata/sexpr2xml-fv-empty-kernel.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-empty-kernel.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-force-hpet.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-force-hpet.xml | 4 +-
>> .../sexpr2xmldata/sexpr2xml-fv-force-nohpet.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-force-nohpet.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-kernel.xml | 2 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-localtime.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-localtime.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.sexpr | 9 -
>> tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml | 48 ---
>> .../sexpr2xmldata/sexpr2xml-fv-net-netfront.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-net-netfront.xml | 4 +-
>> .../sexpr2xmldata/sexpr2xml-fv-parallel-tcp.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-parallel-tcp.xml | 4 +-
>> .../sexpr2xml-fv-serial-dev-2-ports.sexpr | 5 +-
>> .../sexpr2xml-fv-serial-dev-2-ports.xml | 4 +-
>> .../sexpr2xml-fv-serial-dev-2nd-port.sexpr | 4 +-
>> .../sexpr2xml-fv-serial-dev-2nd-port.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-file.sexpr | 7 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-file.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-null.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-null.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-pipe.sexpr | 7 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-pipe.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-pty.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-pty.xml | 4 +-
>> .../sexpr2xmldata/sexpr2xml-fv-serial-stdio.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-stdio.xml | 4 +-
>> .../sexpr2xml-fv-serial-tcp-telnet.sexpr | 3 +-
>> .../sexpr2xml-fv-serial-tcp-telnet.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-tcp.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-tcp.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-udp.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-udp.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-unix.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-serial-unix.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-sound-all.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-sound-all.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-sound.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-sound.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-usbmouse.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-usbmouse.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-usbtablet.sexpr | 3 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-usbtablet.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-utc.sexpr | 2 +
>> tests/sexpr2xmldata/sexpr2xml-fv-utc.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv-v2.xml | 2 +-
>> tests/sexpr2xmldata/sexpr2xml-fv.sexpr | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-fv.xml | 4 +-
>> tests/sexpr2xmldata/sexpr2xml-no-source-cdrom.xml | 2 +-
>> tests/sexpr2xmldata/sexpr2xml-pv-vfb-orig.sexpr | 9 -
>> tests/sexpr2xmldata/sexpr2xml-pv-vfb-orig.xml | 34 --
>> tests/sexpr2xmltest.c | 130 ++++---
>> .../test-fullvirt-direct-kernel-boot.cfg | 3 +-
>> .../test-fullvirt-direct-kernel-boot.xml | 2 +-
>> tests/xlconfigdata/test-fullvirt-multiusb.cfg | 3 +-
>> tests/xlconfigdata/test-fullvirt-multiusb.xml | 2 +-
>> tests/xlconfigdata/test-new-disk.cfg | 3 +-
>> tests/xlconfigdata/test-new-disk.xml | 2 +-
>> tests/xlconfigdata/test-spice-features.cfg | 3 +-
>> tests/xlconfigdata/test-spice-features.xml | 2 +-
>> tests/xlconfigdata/test-spice.cfg | 3 +-
>> tests/xlconfigdata/test-spice.xml | 2 +-
>> tests/xlconfigtest.c | 29 +-
>> tests/xmconfigdata/test-escape-paths.cfg | 5 +-
>> tests/xmconfigdata/test-escape-paths.xml | 2 +-
>> .../xmconfigdata/test-fullvirt-default-feature.cfg | 5 +-
>> .../xmconfigdata/test-fullvirt-default-feature.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-force-hpet.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-force-hpet.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-force-nohpet.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-force-nohpet.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-localtime.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-localtime.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-net-ioemu.cfg | 25 --
>> tests/xmconfigdata/test-fullvirt-net-ioemu.xml | 49 ---
>> tests/xmconfigdata/test-fullvirt-net-netfront.cfg | 3 +
>> tests/xmconfigdata/test-fullvirt-net-netfront.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-new-cdrom.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-new-cdrom.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-old-cdrom.cfg | 26 --
>> tests/xmconfigdata/test-fullvirt-old-cdrom.xml | 49 ---
>> tests/xmconfigdata/test-fullvirt-parallel-tcp.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-parallel-tcp.xml | 2 +-
>> .../test-fullvirt-serial-dev-2-ports.cfg | 5 +-
>> .../test-fullvirt-serial-dev-2-ports.xml | 2 +-
>> .../test-fullvirt-serial-dev-2nd-port.cfg | 5 +-
>> .../test-fullvirt-serial-dev-2nd-port.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-file.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-file.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-null.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-null.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-pipe.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-pipe.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-pty.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-pty.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-stdio.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-stdio.xml | 2 +-
>> .../test-fullvirt-serial-tcp-telnet.cfg | 5 +-
>> .../test-fullvirt-serial-tcp-telnet.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-tcp.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-tcp.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-udp.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-udp.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-serial-unix.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-serial-unix.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-sound.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-sound.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-usbmouse.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-usbmouse.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-usbtablet.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-usbtablet.xml | 2 +-
>> tests/xmconfigdata/test-fullvirt-utc.cfg | 5 +-
>> tests/xmconfigdata/test-fullvirt-utc.xml | 2 +-
>> tests/xmconfigdata/test-no-source-cdrom.cfg | 5 +-
>> tests/xmconfigdata/test-no-source-cdrom.xml | 2 +-
>> .../test-paravirt-old-pvfb-vncdisplay.cfg | 18 -
>> .../test-paravirt-old-pvfb-vncdisplay.xml | 36 --
>> tests/xmconfigdata/test-paravirt-old-pvfb.cfg | 17 -
>> tests/xmconfigdata/test-paravirt-old-pvfb.xml | 36 --
>> tests/xmconfigdata/test-pci-devs.cfg | 5 +-
>> tests/xmconfigdata/test-pci-devs.xml | 2 +-
>> tests/xmconfigtest.c | 94 +++--
>> tests/xml2sexprdata/xml2sexpr-boot-grub.sexpr | 4 +-
>> tests/xml2sexprdata/xml2sexpr-bridge-ipaddr.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-curmem.sexpr | 2 +-
>> .../xml2sexpr-disk-block-shareable.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-disk-block.sexpr | 2 +-
>> .../xml2sexprdata/xml2sexpr-disk-drv-blkback.sexpr | 2 +-
>> .../xml2sexpr-disk-drv-blktap-qcow.sexpr | 2 +-
>> .../xml2sexpr-disk-drv-blktap-raw.sexpr | 2 +-
>> .../xml2sexprdata/xml2sexpr-disk-drv-blktap.sexpr | 2 +-
>> .../xml2sexpr-disk-drv-blktap2-raw.sexpr | 2 +-
>> .../xml2sexprdata/xml2sexpr-disk-drv-blktap2.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-disk-drv-loop.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-disk-file.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-escape.sexpr | 6 +-
>> tests/xml2sexprdata/xml2sexpr-fv-force-hpet.sexpr | 10 +-
>> .../xml2sexprdata/xml2sexpr-fv-force-nohpet.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-kernel.sexpr | 5 +-
>> tests/xml2sexprdata/xml2sexpr-fv-localtime.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-net-ioemu.sexpr | 9 -
>> tests/xml2sexprdata/xml2sexpr-fv-net-ioemu.xml | 31 --
>> .../xml2sexprdata/xml2sexpr-fv-net-netfront.sexpr | 5 +-
>> .../xml2sexprdata/xml2sexpr-fv-parallel-tcp.sexpr | 10 +-
>> .../xml2sexpr-fv-serial-dev-2-ports.sexpr | 13 +-
>> .../xml2sexpr-fv-serial-dev-2nd-port.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-file.sexpr | 13 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-null.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-pipe.sexpr | 13 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-pty.sexpr | 10 +-
>> .../xml2sexprdata/xml2sexpr-fv-serial-stdio.sexpr | 10 +-
>> .../xml2sexpr-fv-serial-tcp-telnet.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-tcp.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-udp.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-serial-unix.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-sound.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-usbmouse.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-utc.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-fv-v2.sexpr | 4 +-
>> tests/xml2sexprdata/xml2sexpr-fv-vncunused.sexpr | 4 +-
>> tests/xml2sexprdata/xml2sexpr-fv.sexpr | 10 +-
>> tests/xml2sexprdata/xml2sexpr-net-bridged.sexpr | 3 +-
>> tests/xml2sexprdata/xml2sexpr-net-e1000.sexpr | 3 +-
>> tests/xml2sexprdata/xml2sexpr-net-routed.sexpr | 3 +-
>> .../xml2sexprdata/xml2sexpr-no-source-cdrom.sexpr | 6 +-
>> tests/xml2sexprdata/xml2sexpr-pci-devs.sexpr | 2 +-
>> .../xml2sexpr-pv-bootloader-cmdline.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-pv-bootloader.sexpr | 2 +-
>> tests/xml2sexprdata/xml2sexpr-pv-localtime.sexpr | 6 +-
>> tests/xml2sexprdata/xml2sexpr-pv-vcpus.sexpr | 3 +-
>> .../xml2sexprdata/xml2sexpr-pv-vfb-new-auto.sexpr | 3 +-
>> tests/xml2sexprdata/xml2sexpr-pv-vfb-new.sexpr | 3 +-
>> tests/xml2sexprdata/xml2sexpr-pv-vfb-orig.sexpr | 10 -
>> tests/xml2sexprdata/xml2sexpr-pv-vfb-orig.xml | 23 --
>> tests/xml2sexprdata/xml2sexpr-pv.sexpr | 3 +-
>> tests/xml2sexprtest.c | 120 +++---
>> 193 files changed, 919 insertions(+), 1833 deletions(-)
>> delete mode 100644 tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.sexpr
>> delete mode 100644 tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml
>> delete mode 100644 tests/sexpr2xmldata/sexpr2xml-pv-vfb-orig.sexpr
>> delete mode 100644 tests/sexpr2xmldata/sexpr2xml-pv-vfb-orig.xml
>> delete mode 100644 tests/xmconfigdata/test-fullvirt-net-ioemu.cfg
>> delete mode 100644 tests/xmconfigdata/test-fullvirt-net-ioemu.xml
>> delete mode 100755 tests/xmconfigdata/test-fullvirt-old-cdrom.cfg
>> delete mode 100644 tests/xmconfigdata/test-fullvirt-old-cdrom.xml
>> delete mode 100644 tests/xmconfigdata/test-paravirt-old-pvfb-vncdisplay.cfg
>> delete mode 100644 tests/xmconfigdata/test-paravirt-old-pvfb-vncdisplay.xml
>> delete mode 100755 tests/xmconfigdata/test-paravirt-old-pvfb.cfg
>> delete mode 100644 tests/xmconfigdata/test-paravirt-old-pvfb.xml
>> delete mode 100644 tests/xml2sexprdata/xml2sexpr-fv-net-ioemu.sexpr
>> delete mode 100644 tests/xml2sexprdata/xml2sexpr-fv-net-ioemu.xml
>> delete mode 100644 tests/xml2sexprdata/xml2sexpr-pv-vfb-orig.sexpr
>> delete mode 100644 tests/xml2sexprdata/xml2sexpr-pv-vfb-orig.xml
>>
> ACK series
Thanks! I removed 'xendConfigVersion' from the comments in xen_driver.c (patch
13/16) noted by Ian and pushed the series.
This was a lot of work in preparation for Ian's patch to fix ARM tests where
maxvcpus > 1
https://www.redhat.com/archives/libvir-list/2015-December/msg00631.html
Ian has 'Tested-by' and 'Acked-by' that patch on top of this series. Any
objections to pushing the fruit of all this labour?
Regards,
Jim
More information about the libvir-list
mailing list