Re: [libvirt] [PATCH V3 0/3] libxl: domain destroy fixes

On 04.04.2015 00:49, Jim Fehlig wrote: > V3 of a small series to fix issues wrt domain destroy > > V1: > https://www.redhat.com/archives/libvir-list/2015-March/msg01337.html > > V2: > https://www.redhat.com/archives/libvir-list/2015-April/msg00072.html > > In this version, patch 3 is changed a bit to provide a wrapper for > unlocking, destroying, and locking the domain. Existing > libxl_domain_destroy callers are changed to use the wrapper. > > Jim Fehlig (3): > libxl: Move job acquisition in libxlDomainStart to callers > libxl: acquire a job when destroying a domain > libxl: drop virDomainObj lock when destroying a domain > > src/libxl/libxl_domain.c | 106 ++++++++++++++++++++++---------------------- > src/libxl/libxl_domain.h | 8 ++-- > src/libxl/libxl_driver.c | 81 ++++++++++++++++++++++++--------- > src/libxl/libxl_migration.c | 4 +- > 4 files changed, 118 insertions(+), 81 deletions(-) > ACK series, but see my comment to the first patch.