summaryrefslogtreecommitdiff
path: root/src/admin
diff options
context:
space:
mode:
authorMartin Kletzander <mkletzan@redhat.com>2015-11-25 16:59:30 +0100
committerMartin Kletzander <mkletzan@redhat.com>2015-12-01 11:44:48 +0100
commitdf8192aaf42266b1d7aafe8f70d8e4ec910cac49 (patch)
tree3b8b834ec9d74b9fb7fb1fd8b4610a6731099808 /src/admin
parent15b0a6d641cde9289aaebfce2fee26d2d838256d (diff)
downloadlibvirt-df8192aaf42266b1d7aafe8f70d8e4ec910cac49.tar.gz
admin: Rename virAdmConnect to virAdmDaemon
virAdmConnect was named after virConnect, but after some discussions, most of the APIs called will be working with remote daemon and starting them virAdmDaemon will make more sense. Only possibly controversal name is CloseCallback (de)registration, and connecting to the daemon (which will still be Open/Close), but even this makes sense if one thinks about the daemon being opened and closed, e.g. as file, etc. This way all the APIs working with the daemon will start with virAdmDaemon prefix, they will accept virAdmDaemonPtr as first parameter and that will better suit with other namings as well (virDomain*, virAdmServer*, etc.). Because in virt-admin, the connection name does not refer to a struct that would have a connect in its name, also adjust 'connname' in clients. And because it is not used anywhere in the vsh code, move it from there into each client. Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Diffstat (limited to 'src/admin')
-rw-r--r--src/admin/admin_protocol.x10
-rw-r--r--src/admin/admin_remote.c36
2 files changed, 23 insertions, 23 deletions
diff --git a/src/admin/admin_protocol.x b/src/admin/admin_protocol.x
index 878983d2ec..b168f6fffd 100644
--- a/src/admin/admin_protocol.x
+++ b/src/admin/admin_protocol.x
@@ -37,11 +37,11 @@ typedef string admin_nonnull_string<ADMIN_STRING_MAX>;
typedef admin_nonnull_string *admin_string;
/*----- Protocol. -----*/
-struct admin_connect_open_args {
+struct admin_daemon_open_args {
unsigned int flags;
};
-struct admin_connect_get_lib_version_ret {
+struct admin_daemon_get_version_ret {
unsigned hyper libVer;
};
@@ -70,15 +70,15 @@ enum admin_procedure {
/**
* @generate: none
*/
- ADMIN_PROC_CONNECT_OPEN = 1,
+ ADMIN_PROC_DAEMON_OPEN = 1,
/**
* @generate: none
*/
- ADMIN_PROC_CONNECT_CLOSE = 2,
+ ADMIN_PROC_DAEMON_CLOSE = 2,
/**
* @generate: both
*/
- ADMIN_PROC_CONNECT_GET_LIB_VERSION = 3
+ ADMIN_PROC_DAEMON_GET_VERSION = 3
};
diff --git a/src/admin/admin_remote.c b/src/admin/admin_remote.c
index 7b40ea16e1..8073b0827a 100644
--- a/src/admin/admin_remote.c
+++ b/src/admin/admin_remote.c
@@ -48,7 +48,7 @@ remoteAdminPrivDispose(void *opaque)
static int
-callFull(virAdmConnectPtr conn ATTRIBUTE_UNUSED,
+callFull(virAdmDaemonPtr dmn ATTRIBUTE_UNUSED,
remoteAdminPrivPtr priv,
int *fdin,
size_t fdinlen,
@@ -86,7 +86,7 @@ callFull(virAdmConnectPtr conn ATTRIBUTE_UNUSED,
}
static int
-call(virAdmConnectPtr conn,
+call(virAdmDaemonPtr dmn,
unsigned int flags,
int proc_nr,
xdrproc_t args_filter, char *args,
@@ -94,7 +94,7 @@ call(virAdmConnectPtr conn,
{
virCheckFlags(0, -1);
- return callFull(conn, conn->privateData,
+ return callFull(dmn, dmn->privateData,
NULL, 0, NULL, NULL, proc_nr,
args_filter, args, ret_filter, ret);
}
@@ -106,14 +106,14 @@ remoteAdminClientCloseFunc(virNetClientPtr client ATTRIBUTE_UNUSED,
int reason,
void *opaque)
{
- virAdmConnectCloseCallbackDataPtr cbdata = opaque;
+ virAdmDaemonCloseCallbackDataPtr cbdata = opaque;
virObjectLock(cbdata);
if (cbdata->callback) {
VIR_DEBUG("Triggering connection close callback %p reason=%d, opaque=%p",
cbdata->callback, reason, cbdata->opaque);
- cbdata->callback(cbdata->conn, reason, cbdata->opaque);
+ cbdata->callback(cbdata->dmn, reason, cbdata->opaque);
if (cbdata->freeCallback)
cbdata->freeCallback(cbdata->opaque);
@@ -124,11 +124,11 @@ remoteAdminClientCloseFunc(virNetClientPtr client ATTRIBUTE_UNUSED,
}
static int
-remoteAdminConnectOpen(virAdmConnectPtr conn, unsigned int flags)
+remoteAdminDaemonOpen(virAdmDaemonPtr dmn, unsigned int flags)
{
int rv = -1;
- remoteAdminPrivPtr priv = conn->privateData;
- admin_connect_open_args args;
+ remoteAdminPrivPtr priv = dmn->privateData;
+ admin_daemon_open_args args;
virObjectLock(priv);
@@ -140,13 +140,13 @@ remoteAdminConnectOpen(virAdmConnectPtr conn, unsigned int flags)
virResetLastError();
}
- virObjectRef(conn->closeCallback);
+ virObjectRef(dmn->closeCallback);
virNetClientSetCloseCallback(priv->client, remoteAdminClientCloseFunc,
- conn->closeCallback,
+ dmn->closeCallback,
virObjectFreeCallback);
- if (call(conn, 0, ADMIN_PROC_CONNECT_OPEN,
- (xdrproc_t)xdr_admin_connect_open_args, (char *)&args,
+ if (call(dmn, 0, ADMIN_PROC_DAEMON_OPEN,
+ (xdrproc_t)xdr_admin_daemon_open_args, (char *)&args,
(xdrproc_t)xdr_void, (char *)NULL) == -1) {
goto done;
}
@@ -159,14 +159,14 @@ remoteAdminConnectOpen(virAdmConnectPtr conn, unsigned int flags)
}
static int
-remoteAdminConnectClose(virAdmConnectPtr conn)
+remoteAdminDaemonClose(virAdmDaemonPtr dmn)
{
int rv = -1;
- remoteAdminPrivPtr priv = conn->privateData;
+ remoteAdminPrivPtr priv = dmn->privateData;
virObjectLock(priv);
- if (call(conn, 0, ADMIN_PROC_CONNECT_CLOSE,
+ if (call(dmn, 0, ADMIN_PROC_DAEMON_CLOSE,
(xdrproc_t)xdr_void, (char *)NULL,
(xdrproc_t)xdr_void, (char *)NULL) == -1) {
goto done;
@@ -184,10 +184,10 @@ remoteAdminConnectClose(virAdmConnectPtr conn)
static void
remoteAdminPrivFree(void *opaque)
{
- virAdmConnectPtr conn = opaque;
+ virAdmDaemonPtr dmn = opaque;
- remoteAdminConnectClose(conn);
- virObjectUnref(conn->privateData);
+ remoteAdminDaemonClose(dmn);
+ virObjectUnref(dmn->privateData);
}
static remoteAdminPrivPtr