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

[Jim Fehlig <jfehlig@xxxxxxxx>]

On 04/09/2015 09:03 AM, Michal Privoznik wrote:
> 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.
I fixed the comment for libxlDomainStart as you suggested and pushed the 
series.  Thanks for the review Michal!
Regards,
Jim


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel