]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commit
Revert "libxl: implement virDomainInterfaceStats"
authorJim Fehlig <jfehlig@suse.com>
Tue, 8 Dec 2015 15:04:40 +0000 (08:04 -0700)
committerJim Fehlig <jfehlig@suse.com>
Tue, 8 Dec 2015 15:04:40 +0000 (08:04 -0700)
commite4ac5919ba040a8ee5245ed9a3b25fe507b5921c
tree5ac5edf2124364124a9b3bc0c25497bc0a1412a2
parentda054f35ab3289c9114bc320338f14b3a88709b2
Revert "libxl: implement virDomainInterfaceStats"

This reverts commit d2e5538b16e325d9095f3ccb0dac88bbd9fc98f0.

A migration regression was introduced by this commit. When migrating
a domain, its active XML is sent to the destination libvirtd, where
it is parsed as inactive XML. d2e5538b copied the libxl generated
interface name into the active config, which was being passed to the
migration destination and being parsed into inactive config. Attempting
to start the config could result in failure if an interface with the
same generated name already exists.

The qemu driver behaves similarly, but the parser contains a hack to
skip interface names starting with 'vnet' when parsing inactive XML.
We could extend the hack to skip names starting with 'vif' too, but a
better fix would be to expose these hypervisor-specific interface name
prefixes in capabilities. See the following discussion thread for more
details

https://www.redhat.com/archives/libvir-list/2015-December/msg00262.html

For the pending 1.3.0 release, it is best to revert d2e5538b. It can
be added again post release, after moving the prefix to capabilities.
src/libxl/libxl_domain.c
src/libxl/libxl_driver.c