Re: [libvirt] [PATCH 0/4] libxl: fixes related to concurrency improvements

On 05.02.2014 18:39, Jim Fehlig wrote: > While reviving old patches to add job support to the libxl driver, > testing revealed some problems that were difficult to encounter > in the current, more serialized processing approach used in the > driver. > > The first patch is a bug fix, plugging leaks of libxlDomainObjPrivate > objects. The second patch removes the list of libxl timer registrations > maintained in the driver - a hack I was never fond of. The third patch > moves domain shutdown handling to a thread, instead of doing all the > shutdown work in the event handler. The fourth patch fixes an issue wrt > child process handling discussed in this thread > > http://lists.xen.org/archives/html/xen-devel/2014-01/msg01553.html > > Ian Jackson's latest patches on the libxl side are here > > http://lists.xen.org/archives/html/xen-devel/2014-02/msg00124.html > > > Jim Fehlig (4): > libxl: fix leaking libxlDomainObjPrivate > libxl: remove list of timer registrations from libxlDomainObjPrivate > libxl: handle domain shutdown events in a thread > libxl: improve subprocess handling > > src/libxl/libxl_conf.h | 5 +- > src/libxl/libxl_domain.c | 102 ++++++++--------------------------- > src/libxl/libxl_domain.h | 8 +-- > src/libxl/libxl_driver.c | 135 > +++++++++++++++++++++++++++++++---------------- > 4 files changed, 115 insertions(+), 135 deletions(-) > ACK series but see my comment on 3/4 where I'm asking for a pair of fixes prior pushing.