On 05/23/2017 09:27 AM, Farhan Ali wrote: > Add testcases for loadparm > > Signed-off-by: Farhan Ali <[email protected]> > Reviewed-by: Boris Fiuczynski <[email protected]> > Reviewed-by: Bjoern Walk <[email protected]> > Reviewed-by: Marc Hartmayer <[email protected]> > --- > ...-machine-loadparm-multiple-disks-nets-s390.args | 28 ++++++++++++++ > ...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 > ++++++++++++++++++++++ > .../qemuxml2argv-machine-loadparm-net-s390.args | 20 ++++++++++ > .../qemuxml2argv-machine-loadparm-net-s390.xml | 26 +++++++++++++ > ...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 +++++++++++++ > ...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 +++++++++++++ > .../qemuxml2argv-machine-loadparm-s390.args | 20 ++++++++++ > .../qemuxml2argv-machine-loadparm-s390.xml | 26 +++++++++++++ > tests/qemuxml2argvtest.c | 19 ++++++++++ > 9 files changed, 234 insertions(+) > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args > create mode 100644 > tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml >
These all seem reasonable - but could be merged with patch2 which had the qemu changes. John -- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
