[libvirt] [PATCH v2] 802.1Qbg: use pre-associate state at beginning of incoming migration
Daniel Veillard
veillard at redhat.com
Tue Nov 30 19:41:47 UTC 2010
On Wed, Nov 24, 2010 at 08:39:23AM -0500, Stefan Berger wrote:
[...]
> Gerhard successfully tested the patch with a recent blade network
> 802.1Qbg-compliant switch.
[...]
> The patch should not have any side-effects on the 802.1Qbh support
> in libvirt, but Roopa (cc'ed) may want to verify this.
>
> Signed-off-by: Stefan Berger <stefanb at us.ibm.com>
> Signed-off-by: Gerhard Stenzel <gerhard.stenzel at de.ibm.com>
>
> ---
> src/libvirt_macvtap.syms | 2 +
> src/qemu/qemu_conf.c | 12 +++++--
> src/qemu/qemu_conf.h | 7 +++-
> src/qemu/qemu_driver.c | 72
> +++++++++++++++++++++++++++++++++++++----------
> src/util/macvtap.c | 54 ++++++++++++++++++++++++++++++-----
> src/util/macvtap.h | 20 +++++++++++--
> 6 files changed, 137 insertions(+), 30 deletions(-)
>
[...]
> @@ -238,7 +239,8 @@ int qemudBuildCommandLine
> int **vmfds,
> int *nvmfds,
> const char *migrateFrom,
> - virDomainSnapshotObjPtr
> current_snapshot)
> + virDomainSnapshotObjPtr
> current_snapshot,
> + enum virVMOperationType vmop)
> ATTRIBUTE_NONNULL(1);
this brroke the build
qemuxml2argvtest.c: In function 'testCompareXMLToArgvFiles':
qemuxml2argvtest.c:92:31: error: too few arguments to function
'qemudBuildCommandLine'
../src/qemu/qemu_conf.h:231:5: note: declared here
I'm commiting a fix,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list