[libvirt] [PATCHv2 00/13] New API: virDominDestroyFlags
Michal Privoznik
mprivozn at redhat.com
Thu Jul 21 18:43:59 UTC 2011
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
More information about the libvir-list
mailing list