From 2cc6c652cf707afec884ecdc41d2de2c8a04d00f Mon Sep 17 00:00:00 2001 From: Martin Kletzander Date: Tue, 16 Jun 2015 14:08:59 +0200 Subject: [PATCH] Revert "admin: Add virAdmHello function" This reverts commit 5792fabb7b712749147e9d03348c798dc1943651. I mistakenly pushed it along with the Admin API series. --- daemon/admin_server.c | 23 ----------------------- include/libvirt/libvirt-admin.h | 1 - po/POTFILES.in | 1 - src/admin/admin_protocol.x | 16 +--------------- src/admin_protocol-structs | 7 ------- src/libvirt-admin.c | 26 -------------------------- src/libvirt_admin.syms | 1 - 7 files changed, 1 insertion(+), 74 deletions(-) diff --git a/daemon/admin_server.c b/daemon/admin_server.c index 76d0b28d74..712a44beca 100644 --- a/daemon/admin_server.c +++ b/daemon/admin_server.c @@ -114,27 +114,4 @@ adminDispatchConnectClose(virNetServerPtr server ATTRIBUTE_UNUSED, return 0; } - -static char * -admHello(virNetDaemonPtr dmn, - unsigned int flags) -{ - char *ret = NULL; - virNetServerPtr srv = NULL; - size_t nclients; - - virCheckFlags(0, NULL); - - if (!(srv = virNetDaemonGetServer(dmn, 0))) - return NULL; - - nclients = virNetServerGetNClients(srv); - if (virAsprintf(&ret, "Clients: %zu", nclients) < 0) - goto cleanup; - - cleanup: - virObjectUnref(srv); - return ret; -} - #include "admin_dispatch.h" diff --git a/include/libvirt/libvirt-admin.h b/include/libvirt/libvirt-admin.h index 27178b45ef..b3cfc93fcb 100644 --- a/include/libvirt/libvirt-admin.h +++ b/include/libvirt/libvirt-admin.h @@ -54,7 +54,6 @@ virAdmConnectPtr virAdmConnectOpen(const char *name, unsigned int flags); int virAdmConnectClose(virAdmConnectPtr conn); int virAdmConnectRef(virAdmConnectPtr conn); -char *virAdmHello(virAdmConnectPtr conn, unsigned int flags); # ifdef __cplusplus } diff --git a/po/POTFILES.in b/po/POTFILES.in index 31f03d9e44..189e2cc2f4 100644 --- a/po/POTFILES.in +++ b/po/POTFILES.in @@ -1,4 +1,3 @@ -daemon/admin_dispatch.h daemon/admin_server.c daemon/libvirtd-config.c daemon/libvirtd.c diff --git a/src/admin/admin_protocol.x b/src/admin/admin_protocol.x index 70954ccb87..1a2e94ee86 100644 --- a/src/admin/admin_protocol.x +++ b/src/admin/admin_protocol.x @@ -43,15 +43,6 @@ struct admin_connect_open_args { unsigned int flags; }; -struct admin_hello_args { - unsigned int flags; -}; - -struct admin_hello_ret { - admin_string greeting; -}; - - /* Define the program number, protocol version and procedure numbers here. */ const ADMIN_PROGRAM = 0x06900690; const ADMIN_PROTOCOL_VERSION = 1; @@ -82,10 +73,5 @@ enum admin_procedure { /** * @generate: client */ - ADMIN_PROC_CONNECT_CLOSE = 2, - - /** - * @generate: both - */ - ADMIN_PROC_HELLO = 3 + ADMIN_PROC_CONNECT_CLOSE = 2 }; diff --git a/src/admin_protocol-structs b/src/admin_protocol-structs index f91f7013ad..3ac31fa253 100644 --- a/src/admin_protocol-structs +++ b/src/admin_protocol-structs @@ -2,14 +2,7 @@ struct admin_connect_open_args { u_int flags; }; -struct admin_hello_args { - u_int flags; -}; -struct admin_hello_ret { - admin_string greeting; -}; enum admin_procedure { ADMIN_PROC_CONNECT_OPEN = 1, ADMIN_PROC_CONNECT_CLOSE = 2, - ADMIN_PROC_HELLO = 3, }; diff --git a/src/libvirt-admin.c b/src/libvirt-admin.c index 107e27e430..11b6fe3238 100644 --- a/src/libvirt-admin.c +++ b/src/libvirt-admin.c @@ -384,29 +384,3 @@ virAdmConnectRef(virAdmConnectPtr conn) return 0; } - - -/** - * virAdmHello: - * @conn: a connection object - * @flags: unused, 0 for now - * - * Testing function. - * - * Returns the number of connected clients as a string. Yes, as a - * string. Because it's just for testing. - */ -char * -virAdmHello(virAdmConnectPtr conn, - unsigned int flags) -{ - char *ret = NULL; - - virResetLastError(); - - ret = remoteAdminHello(conn, flags); - if (!ret) - virDispatchError(NULL); - - return ret; -} diff --git a/src/libvirt_admin.syms b/src/libvirt_admin.syms index a3c9e5fc3e..d9e3c0b1b9 100644 --- a/src/libvirt_admin.syms +++ b/src/libvirt_admin.syms @@ -15,5 +15,4 @@ LIBVIRT_ADMIN_1.3.0 { virAdmConnectOpen; virAdmConnectClose; virAdmConnectRef; - virAdmHello; };