Re: [PATCH v2 07/25] backup: Implement backup APIs for remote driver

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

 



On Tue, Dec 03, 2019 at 06:17:29PM +0100, Peter Krempa wrote:
From: Eric Blake <eblake@xxxxxxxxxx>

This one is fairly straightforward - the generator already does what
we need.

Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>
Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
src/remote/remote_driver.c   |  2 ++
src/remote/remote_protocol.x | 33 ++++++++++++++++++++++++++++++++-
src/remote_protocol-structs  | 15 +++++++++++++++
3 files changed, 49 insertions(+), 1 deletion(-)

diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index a1384fc655..ddb95914a6 100644
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -8702,6 +8702,8 @@ static virHypervisorDriver hypervisor_driver = {
    .domainCheckpointDelete = remoteDomainCheckpointDelete, /* 5.6.0 */
    .domainGetGuestInfo = remoteDomainGetGuestInfo, /* 5.7.0 */
    .domainAgentSetResponseTimeout = remoteDomainAgentSetResponseTimeout, /* 5.10.0 */
+    .domainBackupBegin = remoteDomainBackupBegin, /* 6.0.0 */
+    .domainBackupGetXMLDesc = remoteDomainBackupGetXMLDesc, /* 6.0.0 */
};

static virNetworkDriver network_driver = {
diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x
index 23e42d17b1..c79cb98ae8 100644
--- a/src/remote/remote_protocol.x
+++ b/src/remote/remote_protocol.x
@@ -3754,6 +3754,23 @@ struct remote_domain_agent_set_response_timeout_ret {
    int result;
};

+
+struct remote_domain_backup_begin_args {
+    remote_nonnull_domain dom;
+    remote_string backup_xml;

Should this be remote_nonnull_string?

+    remote_string checkpoint_xml;
+    unsigned int flags;
+};
+

Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux