[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCHv2 00/13] New API: virDominDestroyFlags



On 21.07.2011 18:55, Daniel P. Berrange wrote:
On Thu, Jul 21, 2011 at 06:08:31PM +0200, Michal Privoznik wrote:
This series creates new API which adds flags to
virDomainDestroy. As mentioned here many times, it is
a bad luck all APIs don't have flags parameter.

Parameter for this might be needed to select different
ways of destroying a domain. For example, qemu has 'quit'
command, which force it to flush internal caches a thus
preventing from data loss. Or vice versa - in cases
where we are doing hypervisor assisted destroy we might
want to send signals, because hypervisor is not responding.

Currently, no new functionality is implemented, but
this paves the way for creating multiple ways of domain
destroying. Therefore, calling this new API is the same
as calling its predecessor.

diff to v1:
-rename virDominDestroyWithFlags() to
  virDominDestroyFlags()
-rename enum virDominDestroyFlags to
  virDominDestroyFlagsValues

Michal Privoznik (13):
   destroy: Define new public API virDomainDestroyFlags
   destroy: Wire up the remote protocol
   destroy: Implement internal API for qemu driver
   destroy: Implement internal API for ESX driver
   destroy: Implement internal API for libxl driver
   destroy: Implement internal API for lxc driver
   destroy: Implement internal API for openvz driver
   destroy: Implement internal API for phyp driver
   destroy: Implement internal API for uml driver
   destroy: Implement internal API for vbox driver
   destroy: Implement internal API for vmware driver
   destroy: Implement internal API for xen driver
   destroy: Implement internal API for xenapi driver

  include/libvirt/libvirt.h.in |    7 +++++
  src/driver.h                 |    4 +++
  src/esx/esx_driver.c         |   12 +++++++++-
  src/libvirt.c                |   52 ++++++++++++++++++++++++++++++++++++++++++
  src/libvirt_public.syms      |    1 +
  src/libxl/libxl_driver.c     |   12 +++++++++-
  src/lxc/lxc_driver.c         |   23 +++++++++++++++++-
  src/openvz/openvz_driver.c   |   13 +++++++++-
  src/phyp/phyp_driver.c       |   12 +++++++++-
  src/qemu/qemu_driver.c       |   13 +++++++++-
  src/remote/remote_driver.c   |    1 +
  src/remote/remote_protocol.x |    8 +++++-
  src/remote_protocol-structs  |    5 ++++
  src/uml/uml_driver.c         |   14 ++++++++++-
  src/vbox/vbox_tmpl.c         |   14 ++++++++++-
  src/vmware/vmware_driver.c   |   12 +++++++++-
  src/xen/xen_driver.c         |   28 ++++++++++++++++++++++
  src/xen/xen_driver.h         |    1 +
  src/xen/xen_hypervisor.c     |   24 +++++++++++++++++-
  src/xen/xen_hypervisor.h     |    3 ++
  src/xen/xen_inotify.c        |    1 +
  src/xen/xend_internal.c      |   24 +++++++++++++++++-
  src/xen/xm_internal.c        |    1 +
  src/xen/xs_internal.c        |    1 +
  src/xenapi/xenapi_driver.c   |   22 ++++++++++++++++-
  25 files changed, 291 insertions(+), 17 deletions(-)

ACK to the whole series

Daniel

Thanks, incorporated Eric's suggestions and pushed.

Michal


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]