[libvirt] [PATCH 0/4] Some small libxl bug fixes

Daniel Veillard veillard at redhat.com
Tue Jun 30 07:02:50 UTC 2015


On Mon, Jun 29, 2015 at 11:30:11AM -0600, Jim Fehlig wrote:
> While working on a patch to add dom0 support to the libxl driver,
> I noticed a few existing bugs which are fixed by this small
> series. IMO 1 and 2 are safe even during the freeze as they fix
> quite annoying bugs. 3 and 4 are probably safe bug fixes too,
> unless I'm confused about the use of
> domainObj->{def,newDef,persistentDef}
> 
> Jim Fehlig (4):
>   libxl: don't overwrite domain state from statedir config
>   libxl: don't remove persistent domain on start failure
>   libxl: honor domainGetXMLDesc() --inactive flag
>   libxl: Set def->vcpus after successfully modifying live vcpu count
> 
>  src/libxl/libxl_driver.c | 33 ++++++++++++++++++++++++++-------
>  1 file changed, 26 insertions(+), 7 deletions(-)

  Those looks reasonable to me but 1/ I'm a bit rusty 2/ locking is
hard so even if I ACK this take it with a grain of salt :-)

Daniel

-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veillard at redhat.com  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list