[libvirt] [PATCH 0/2] virsh: Properly handle detach-interface --live --config.
Cole Robinson
crobinso at redhat.com
Wed May 4 15:43:46 UTC 2016
On 05/04/2016 10:26 AM, Nitesh Konkar wrote:
> The virsh attach virsh detach interface command fails
> when both live and config are set and when the interface
> gets attached to different pci slots on live and config
> xml respectively.
>
> When we attach an interface with both --live and --config,
> the first time they get the same PCI slots, but the second time
> onwards it differs and hence the virsh detach-interface --live
> --config command fails. This patch makes sure that when both
> --live --config are set , qemuDomainDetachDeviceFlags is called
> twice, once with config xml and once with live xml.
>
> Steps to see the issue:
> virsh attach-interface --domain DomainName --type network --source default --mac 52:54:00:4b:76:5f --live --config
> virsh detach-interface --domain DomainName --type network --mac 52:54:00:4b:76:5f --live --config
> virsh attach-interface --domain DomainName --type network --source default --mac 52:54:00:4b:76:5f --live --config
> virsh detach-interface --domain DomainName --type network --mac 52:54:00:4b:76:5f --live --config
>
>
> Nitesh Konkar (2):
> virsh: Introduce virshDomainDetachInterface function
> virsh: Pass the correct live/config xml to virshDomainDetachInterface
>
> tools/virsh-domain.c | 128 +++++++++++++++++++++++++++++----------------------
> 1 file changed, 73 insertions(+), 55 deletions(-)
>
Should have mentioned, the subject should have contained [PATCH v3] or
whatever version we were on. You can do 'git format-patch -v3' for that, or
send-email -v3
Similarly, if you are going through multiple versions, list in the cover
letter what changed between versions. Like
v3:
Fixed indentation
Fix memory leak
or similar
Thanks,
Cole
More information about the libvir-list
mailing list