Re: [libvirt] [PATCH V2 00/13] libxl: add job support

2014-02-19 Thread Jim Fehlig
Jim Fehlig wrote:
 This patch series adds job support to the libxl driver, using a simplified
 version of the technique used in the qemu driver.  One benefit of this
 series is no longer blocking get operations during long running modify
 operations.  E.g. with these patches 'vish dominfo dom' will work while
 'virsh save dom ...' is in progress.

 The first two patches are new to the series, following review of V1.
 Patch1 changes the cleanup logic to unconditionally set dom id to -1 on
 domain shutdown.  Patch2 removes libxlVmReap, giving callers more control
 over domain destruction/cleanup.

 The remaining patches are updates to V1 based on Daniel's and Michal's
 comments.
   

Ping.

This series would be a nice little improvement in the libxl driver for
1.2.2 :-).

Regards,
Jim

 Jim Fehlig (13):
   libxl: always set vm id to -1 on shutdown
   libxl: remove libxlVmReap function
   libxl: Add job support to libxl driver
   libxl: use job functions in libxlVmStart
   libxl: use job functions in libxlDomainSetMemoryFlags
   libxl: use job functions in libxlDomain{Suspend,Resume}
   libxl: use job functions when cleaning up a domain
   libxl: use job functions in domain save operations
   libxl: use job functions in libxlDomainCoreDump
   libxl: use job functions in vcpu set and pin functions
   libxl: use job functions in device attach and detach functions
   libxl: use job functions in libxlDomainSetAutostart
   libxl: use job functions in libxlDomainSetSchedulerParametersFlags

  src/libxl/libxl_domain.c | 128 +++
  src/libxl/libxl_domain.h |  38 +
  src/libxl/libxl_driver.c | 405 
 +++
  3 files changed, 432 insertions(+), 139 deletions(-)

   

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


Re: [libvirt] [PATCH V2 00/13] libxl: add job support

2014-02-19 Thread Daniel P. Berrange
On Wed, Feb 19, 2014 at 08:35:21AM -0700, Jim Fehlig wrote:
 Jim Fehlig wrote:
  This patch series adds job support to the libxl driver, using a simplified
  version of the technique used in the qemu driver.  One benefit of this
  series is no longer blocking get operations during long running modify
  operations.  E.g. with these patches 'vish dominfo dom' will work while
  'virsh save dom ...' is in progress.
 
  The first two patches are new to the series, following review of V1.
  Patch1 changes the cleanup logic to unconditionally set dom id to -1 on
  domain shutdown.  Patch2 removes libxlVmReap, giving callers more control
  over domain destruction/cleanup.
 
  The remaining patches are updates to V1 based on Daniel's and Michal's
  comments.

 
 Ping.
 
 This series would be a nice little improvement in the libxl driver for
 1.2.2 :-).

Yep, will re-review this shortly. Shouldn't be any problem geting
it into 1.2.2 since previous posting was almost there.

Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


[libvirt] [PATCH V2 00/13] libxl: add job support

2014-02-12 Thread Jim Fehlig
This patch series adds job support to the libxl driver, using a simplified
version of the technique used in the qemu driver.  One benefit of this
series is no longer blocking get operations during long running modify
operations.  E.g. with these patches 'vish dominfo dom' will work while
'virsh save dom ...' is in progress.

The first two patches are new to the series, following review of V1.
Patch1 changes the cleanup logic to unconditionally set dom id to -1 on
domain shutdown.  Patch2 removes libxlVmReap, giving callers more control
over domain destruction/cleanup.

The remaining patches are updates to V1 based on Daniel's and Michal's
comments.

Jim Fehlig (13):
  libxl: always set vm id to -1 on shutdown
  libxl: remove libxlVmReap function
  libxl: Add job support to libxl driver
  libxl: use job functions in libxlVmStart
  libxl: use job functions in libxlDomainSetMemoryFlags
  libxl: use job functions in libxlDomain{Suspend,Resume}
  libxl: use job functions when cleaning up a domain
  libxl: use job functions in domain save operations
  libxl: use job functions in libxlDomainCoreDump
  libxl: use job functions in vcpu set and pin functions
  libxl: use job functions in device attach and detach functions
  libxl: use job functions in libxlDomainSetAutostart
  libxl: use job functions in libxlDomainSetSchedulerParametersFlags

 src/libxl/libxl_domain.c | 128 +++
 src/libxl/libxl_domain.h |  38 +
 src/libxl/libxl_driver.c | 405 +++
 3 files changed, 432 insertions(+), 139 deletions(-)

-- 
1.8.1.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list