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

Re: [libvirt] PATCH: 3/25: Remove use of macros from remote driver



THis patch removes use of macros for accessing the privateDtaa
on a connection because they obscure data access making it harder
to visually validate correct thread locking

 remote_internal.c |  272 ++++++++++++++++++++++--------------------------------
 1 file changed, 114 insertions(+), 158 deletions(-)

Daniel

diff --git a/src/remote_internal.c b/src/remote_internal.c
--- a/src/remote_internal.c
+++ b/src/remote_internal.c
@@ -86,14 +86,9 @@
 #include "util.h"
 #include "event.h"
 
-/* Per-connection private data. */
-#define MAGIC 999               /* private_data->magic if OK */
-#define DEAD 998                /* private_data->magic if dead/closed */
-
 static int inside_daemon = 0;
 
 struct private_data {
-    int magic;                  /* Should be MAGIC or DEAD. */
     int sock;                   /* Socket. */
     int watch;                  /* File handle watch */
     pid_t pid;                  /* PID of tunnel process */
@@ -118,39 +113,6 @@ struct private_data {
     /* Timer for flushing domainEvents queue */
     int eventFlushTimer;
 };
-
-#define GET_PRIVATE(conn,retcode)                                       \
-    struct private_data *priv = (struct private_data *) (conn)->privateData; \
-    if (!priv || priv->magic != MAGIC) {                                \
-        error (conn, VIR_ERR_INVALID_ARG,                               \
-               _("tried to use a closed or uninitialised handle"));     \
-        return (retcode);                                               \
-    }
-
-#define GET_NETWORK_PRIVATE(conn,retcode)                               \
-    struct private_data *priv = (struct private_data *) (conn)->networkPrivateData; \
-    if (!priv || priv->magic != MAGIC) {                                \
-        error (conn, VIR_ERR_INVALID_ARG,                               \
-               _("tried to use a closed or uninitialised handle"));     \
-        return (retcode);                                               \
-    }
-
-#define GET_STORAGE_PRIVATE(conn,retcode)                               \
-    struct private_data *priv = (struct private_data *) (conn)->storagePrivateData; \
-    if (!priv || priv->magic != MAGIC) {                                \
-        error (conn, VIR_ERR_INVALID_ARG,                               \
-               "tried to use a closed or uninitialised handle");        \
-        return (retcode);                                               \
-    }
-
-#define GET_DEVMON_PRIVATE(conn,retcode)                               \
-    struct private_data *priv = (struct private_data *) (conn)->devMonPrivateData; \
-    if (!priv || priv->magic != MAGIC) {                                \
-        error (conn, VIR_ERR_INVALID_ARG,                               \
-               _("tried to use a closed or uninitialised handle"));     \
-        return (retcode);                                               \
-    }
-
 
 enum {
     REMOTE_CALL_IN_OPEN = 1,
@@ -917,14 +879,12 @@ remoteOpen (virConnectPtr conn,
         }
     }
 
-    priv->magic = DEAD;
     priv->sock = -1;
     ret = doRemoteOpen(conn, priv, auth, rflags);
     if (ret != VIR_DRV_OPEN_SUCCESS) {
         conn->privateData = NULL;
         VIR_FREE(priv);
     } else {
-        priv->magic = MAGIC;
         conn->privateData = priv;
     }
     return ret;
@@ -1268,9 +1228,6 @@ doRemoteClose (virConnectPtr conn, struc
     /* See comment for remoteType. */
     free (priv->type);
 
-    /* Free private data. */
-    priv->magic = DEAD;
-
     /* Free callback list */
     virDomainEventCallbackListFree(priv->callbackList);
 
@@ -1284,10 +1241,10 @@ remoteClose (virConnectPtr conn)
 remoteClose (virConnectPtr conn)
 {
     int ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     ret = doRemoteClose(conn, priv);
-    free (priv);
+    VIR_FREE (priv);
     conn->privateData = NULL;
 
     return ret;
@@ -1298,7 +1255,7 @@ remoteSupportsFeature (virConnectPtr con
 {
     remote_supports_feature_args args;
     remote_supports_feature_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     /* VIR_DRV_FEATURE_REMOTE* features are handled directly. */
     if (feature == VIR_DRV_FEATURE_REMOTE) return 1;
@@ -1326,7 +1283,7 @@ remoteType (virConnectPtr conn)
 remoteType (virConnectPtr conn)
 {
     remote_get_type_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     /* Cached? */
     if (priv->type) return priv->type;
@@ -1345,7 +1302,7 @@ remoteGetVersion (virConnectPtr conn, un
 remoteGetVersion (virConnectPtr conn, unsigned long *hvVer)
 {
     remote_get_version_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_GET_VERSION,
@@ -1361,7 +1318,7 @@ remoteGetHostname (virConnectPtr conn)
 remoteGetHostname (virConnectPtr conn)
 {
     remote_get_hostname_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_GET_HOSTNAME,
@@ -1378,7 +1335,7 @@ remoteGetMaxVcpus (virConnectPtr conn, c
 {
     remote_get_max_vcpus_args args;
     remote_get_max_vcpus_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     args.type = type == NULL ? NULL : (char **) &type;
@@ -1394,7 +1351,7 @@ remoteNodeGetInfo (virConnectPtr conn, v
 remoteNodeGetInfo (virConnectPtr conn, virNodeInfoPtr info)
 {
     remote_node_get_info_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NODE_GET_INFO,
@@ -1418,7 +1375,7 @@ remoteGetCapabilities (virConnectPtr con
 remoteGetCapabilities (virConnectPtr conn)
 {
     remote_get_capabilities_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_GET_CAPABILITIES,
@@ -1439,7 +1396,7 @@ remoteNodeGetCellsFreeMemory(virConnectP
     remote_node_get_cells_free_memory_args args;
     remote_node_get_cells_free_memory_ret ret;
     int i;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     if (maxCells > REMOTE_NODE_MAX_CELLS) {
         errorf (conn, VIR_ERR_RPC,
@@ -1470,7 +1427,7 @@ remoteNodeGetFreeMemory (virConnectPtr c
 remoteNodeGetFreeMemory (virConnectPtr conn)
 {
     remote_node_get_free_memory_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NODE_GET_FREE_MEMORY,
@@ -1488,7 +1445,7 @@ remoteListDomains (virConnectPtr conn, i
     int i;
     remote_list_domains_args args;
     remote_list_domains_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     if (maxids > REMOTE_DOMAIN_ID_LIST_MAX) {
         errorf (conn, VIR_ERR_RPC,
@@ -1524,7 +1481,7 @@ remoteNumOfDomains (virConnectPtr conn)
 remoteNumOfDomains (virConnectPtr conn)
 {
     remote_num_of_domains_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_DOMAINS,
@@ -1543,7 +1500,7 @@ remoteDomainCreateXML (virConnectPtr con
     virDomainPtr dom;
     remote_domain_create_xml_args args;
     remote_domain_create_xml_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     args.xml_desc = (char *) xmlDesc;
     args.flags = flags;
@@ -1566,7 +1523,7 @@ remoteDomainLookupByID (virConnectPtr co
     virDomainPtr dom;
     remote_domain_lookup_by_id_args args;
     remote_domain_lookup_by_id_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     args.id = id;
 
@@ -1588,7 +1545,7 @@ remoteDomainLookupByUUID (virConnectPtr 
     virDomainPtr dom;
     remote_domain_lookup_by_uuid_args args;
     remote_domain_lookup_by_uuid_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     memcpy (args.uuid, uuid, VIR_UUID_BUFLEN);
 
@@ -1609,7 +1566,7 @@ remoteDomainLookupByName (virConnectPtr 
     virDomainPtr dom;
     remote_domain_lookup_by_name_args args;
     remote_domain_lookup_by_name_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     args.name = (char *) name;
 
@@ -1629,7 +1586,7 @@ remoteDomainSuspend (virDomainPtr domain
 remoteDomainSuspend (virDomainPtr domain)
 {
     remote_domain_suspend_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1645,7 +1602,7 @@ remoteDomainResume (virDomainPtr domain)
 remoteDomainResume (virDomainPtr domain)
 {
     remote_domain_resume_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1661,7 +1618,7 @@ remoteDomainShutdown (virDomainPtr domai
 remoteDomainShutdown (virDomainPtr domain)
 {
     remote_domain_shutdown_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1677,7 +1634,7 @@ remoteDomainReboot (virDomainPtr domain,
 remoteDomainReboot (virDomainPtr domain, unsigned int flags)
 {
     remote_domain_reboot_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.flags = flags;
@@ -1694,7 +1651,7 @@ remoteDomainDestroy (virDomainPtr domain
 remoteDomainDestroy (virDomainPtr domain)
 {
     remote_domain_destroy_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1711,7 +1668,7 @@ remoteDomainGetOSType (virDomainPtr doma
 {
     remote_domain_get_os_type_args args;
     remote_domain_get_os_type_ret ret;
-    GET_PRIVATE (domain->conn, NULL);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1730,7 +1687,7 @@ remoteDomainGetMaxMemory (virDomainPtr d
 {
     remote_domain_get_max_memory_args args;
     remote_domain_get_max_memory_ret ret;
-    GET_PRIVATE (domain->conn, 0);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1747,7 +1704,7 @@ remoteDomainSetMaxMemory (virDomainPtr d
 remoteDomainSetMaxMemory (virDomainPtr domain, unsigned long memory)
 {
     remote_domain_set_max_memory_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.memory = memory;
@@ -1764,7 +1721,7 @@ remoteDomainSetMemory (virDomainPtr doma
 remoteDomainSetMemory (virDomainPtr domain, unsigned long memory)
 {
     remote_domain_set_memory_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.memory = memory;
@@ -1782,7 +1739,7 @@ remoteDomainGetInfo (virDomainPtr domain
 {
     remote_domain_get_info_args args;
     remote_domain_get_info_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1805,7 +1762,7 @@ remoteDomainSave (virDomainPtr domain, c
 remoteDomainSave (virDomainPtr domain, const char *to)
 {
     remote_domain_save_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.to = (char *) to;
@@ -1822,7 +1779,7 @@ remoteDomainRestore (virConnectPtr conn,
 remoteDomainRestore (virConnectPtr conn, const char *from)
 {
     remote_domain_restore_args args;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     args.from = (char *) from;
 
@@ -1838,7 +1795,7 @@ remoteDomainCoreDump (virDomainPtr domai
 remoteDomainCoreDump (virDomainPtr domain, const char *to, int flags)
 {
     remote_domain_core_dump_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.to = (char *) to;
@@ -1856,7 +1813,7 @@ remoteDomainSetVcpus (virDomainPtr domai
 remoteDomainSetVcpus (virDomainPtr domain, unsigned int nvcpus)
 {
     remote_domain_set_vcpus_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.nvcpus = nvcpus;
@@ -1876,7 +1833,7 @@ remoteDomainPinVcpu (virDomainPtr domain
                      int maplen)
 {
     remote_domain_pin_vcpu_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     if (maplen > REMOTE_CPUMAP_MAX) {
         errorf (domain->conn, VIR_ERR_RPC,
@@ -1908,7 +1865,7 @@ remoteDomainGetVcpus (virDomainPtr domai
     int i;
     remote_domain_get_vcpus_args args;
     remote_domain_get_vcpus_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     if (maxinfo > REMOTE_VCPUINFO_MAX) {
         errorf (domain->conn, VIR_ERR_RPC,
@@ -1970,7 +1927,7 @@ remoteDomainGetMaxVcpus (virDomainPtr do
 {
     remote_domain_get_max_vcpus_args args;
     remote_domain_get_max_vcpus_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -1988,7 +1945,7 @@ remoteDomainDumpXML (virDomainPtr domain
 {
     remote_domain_dump_xml_args args;
     remote_domain_dump_xml_ret ret;
-    GET_PRIVATE (domain->conn, NULL);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.flags = flags;
@@ -2012,7 +1969,7 @@ remoteDomainMigratePrepare (virConnectPt
 {
     remote_domain_migrate_prepare_args args;
     remote_domain_migrate_prepare_ret ret;
-    GET_PRIVATE (dconn, -1);
+    struct private_data *priv = dconn->privateData;
 
     args.uri_in = uri_in == NULL ? NULL : (char **) &uri_in;
     args.flags = flags;
@@ -2045,7 +2002,7 @@ remoteDomainMigratePerform (virDomainPtr
                             unsigned long resource)
 {
     remote_domain_migrate_perform_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.cookie.cookie_len = cookielen;
@@ -2074,7 +2031,7 @@ remoteDomainMigrateFinish (virConnectPtr
     virDomainPtr ddom;
     remote_domain_migrate_finish_args args;
     remote_domain_migrate_finish_ret ret;
-    GET_PRIVATE (dconn, NULL);
+    struct private_data *priv = dconn->privateData;
 
     args.dname = (char *) dname;
     args.cookie.cookie_len = cookielen;
@@ -2104,7 +2061,7 @@ remoteDomainMigratePrepare2 (virConnectP
 {
     remote_domain_migrate_prepare2_args args;
     remote_domain_migrate_prepare2_ret ret;
-    GET_PRIVATE (dconn, -1);
+    struct private_data *priv = dconn->privateData;
 
     args.uri_in = uri_in == NULL ? NULL : (char **) &uri_in;
     args.flags = flags;
@@ -2140,7 +2097,7 @@ remoteDomainMigrateFinish2 (virConnectPt
     virDomainPtr ddom;
     remote_domain_migrate_finish2_args args;
     remote_domain_migrate_finish2_ret ret;
-    GET_PRIVATE (dconn, NULL);
+    struct private_data *priv = dconn->privateData;
 
     args.dname = (char *) dname;
     args.cookie.cookie_len = cookielen;
@@ -2167,7 +2124,7 @@ remoteListDefinedDomains (virConnectPtr 
     int i;
     remote_list_defined_domains_args args;
     remote_list_defined_domains_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     if (maxnames > REMOTE_DOMAIN_NAME_LIST_MAX) {
         errorf (conn, VIR_ERR_RPC,
@@ -2208,7 +2165,7 @@ remoteNumOfDefinedDomains (virConnectPtr
 remoteNumOfDefinedDomains (virConnectPtr conn)
 {
     remote_num_of_defined_domains_ret ret;
-    GET_PRIVATE (conn, -1);
+    struct private_data *priv = conn->privateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_DEFINED_DOMAINS,
@@ -2223,7 +2180,7 @@ remoteDomainCreate (virDomainPtr domain)
 remoteDomainCreate (virDomainPtr domain)
 {
     remote_domain_create_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -2241,7 +2198,7 @@ remoteDomainDefineXML (virConnectPtr con
     virDomainPtr dom;
     remote_domain_define_xml_args args;
     remote_domain_define_xml_ret ret;
-    GET_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->privateData;
 
     args.xml = (char *) xml;
 
@@ -2261,7 +2218,7 @@ remoteDomainUndefine (virDomainPtr domai
 remoteDomainUndefine (virDomainPtr domain)
 {
     remote_domain_undefine_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -2277,7 +2234,7 @@ remoteDomainAttachDevice (virDomainPtr d
 remoteDomainAttachDevice (virDomainPtr domain, const char *xml)
 {
     remote_domain_attach_device_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.xml = (char *) xml;
@@ -2294,7 +2251,7 @@ remoteDomainDetachDevice (virDomainPtr d
 remoteDomainDetachDevice (virDomainPtr domain, const char *xml)
 {
     remote_domain_detach_device_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.xml = (char *) xml;
@@ -2312,7 +2269,7 @@ remoteDomainGetAutostart (virDomainPtr d
 {
     remote_domain_get_autostart_args args;
     remote_domain_get_autostart_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -2330,7 +2287,7 @@ remoteDomainSetAutostart (virDomainPtr d
 remoteDomainSetAutostart (virDomainPtr domain, int autostart)
 {
     remote_domain_set_autostart_args args;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.autostart = autostart;
@@ -2348,7 +2305,7 @@ remoteDomainGetSchedulerType (virDomainP
 {
     remote_domain_get_scheduler_type_args args;
     remote_domain_get_scheduler_type_ret ret;
-    GET_PRIVATE (domain->conn, NULL);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -2371,7 +2328,7 @@ remoteDomainGetSchedulerParameters (virD
     remote_domain_get_scheduler_parameters_args args;
     remote_domain_get_scheduler_parameters_ret ret;
     int i;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.nparams = *nparams;
@@ -2431,7 +2388,7 @@ remoteDomainSetSchedulerParameters (virD
 {
     remote_domain_set_scheduler_parameters_args args;
     int i, do_error;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
 
@@ -2489,7 +2446,7 @@ remoteDomainBlockStats (virDomainPtr dom
 {
     remote_domain_block_stats_args args;
     remote_domain_block_stats_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.path = (char *) path;
@@ -2516,7 +2473,7 @@ remoteDomainInterfaceStats (virDomainPtr
 {
     remote_domain_interface_stats_args args;
     remote_domain_interface_stats_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     make_nonnull_domain (&args.dom, domain);
     args.path = (char *) path;
@@ -2551,7 +2508,7 @@ remoteDomainBlockPeek (virDomainPtr doma
 {
     remote_domain_block_peek_args args;
     remote_domain_block_peek_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     if (size > REMOTE_DOMAIN_BLOCK_PEEK_BUFFER_MAX) {
         errorf (domain->conn, VIR_ERR_RPC,
@@ -2596,7 +2553,7 @@ remoteDomainMemoryPeek (virDomainPtr dom
 {
     remote_domain_memory_peek_args args;
     remote_domain_memory_peek_ret ret;
-    GET_PRIVATE (domain->conn, -1);
+    struct private_data *priv = domain->conn->privateData;
 
     if (size > REMOTE_DOMAIN_MEMORY_PEEK_BUFFER_MAX) {
         errorf (domain->conn, VIR_ERR_RPC,
@@ -2666,14 +2623,12 @@ remoteNetworkOpen (virConnectPtr conn,
             rflags |= VIR_DRV_OPEN_REMOTE_RO;
         rflags |= VIR_DRV_OPEN_REMOTE_UNIX;
 
-        priv->magic = DEAD;
         priv->sock = -1;
         ret = doRemoteOpen(conn, priv, auth, rflags);
         if (ret != VIR_DRV_OPEN_SUCCESS) {
             conn->networkPrivateData = NULL;
             VIR_FREE(priv);
         } else {
-            priv->magic = MAGIC;
             priv->localUses = 1;
             conn->networkPrivateData = priv;
         }
@@ -2685,7 +2640,8 @@ remoteNetworkClose (virConnectPtr conn)
 remoteNetworkClose (virConnectPtr conn)
 {
     int ret = 0;
-    GET_NETWORK_PRIVATE (conn, -1);
+    struct private_data *priv = conn->networkPrivateData;
+
     if (priv->localUses) {
         priv->localUses--;
         if (!priv->localUses) {
@@ -2701,7 +2657,7 @@ remoteNumOfNetworks (virConnectPtr conn)
 remoteNumOfNetworks (virConnectPtr conn)
 {
     remote_num_of_networks_ret ret;
-    GET_NETWORK_PRIVATE (conn, -1);
+    struct private_data *priv = conn->networkPrivateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_NETWORKS,
@@ -2718,7 +2674,7 @@ remoteListNetworks (virConnectPtr conn, 
     int i;
     remote_list_networks_args args;
     remote_list_networks_ret ret;
-    GET_NETWORK_PRIVATE (conn, -1);
+    struct private_data *priv = conn->networkPrivateData;
 
     if (maxnames > REMOTE_NETWORK_NAME_LIST_MAX) {
         errorf (conn, VIR_ERR_RPC,
@@ -2759,7 +2715,7 @@ remoteNumOfDefinedNetworks (virConnectPt
 remoteNumOfDefinedNetworks (virConnectPtr conn)
 {
     remote_num_of_defined_networks_ret ret;
-    GET_NETWORK_PRIVATE (conn, -1);
+    struct private_data *priv = conn->networkPrivateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_DEFINED_NETWORKS,
@@ -2777,7 +2733,7 @@ remoteListDefinedNetworks (virConnectPtr
     int i;
     remote_list_defined_networks_args args;
     remote_list_defined_networks_ret ret;
-    GET_NETWORK_PRIVATE (conn, -1);
+    struct private_data *priv = conn->networkPrivateData;
 
     if (maxnames > REMOTE_NETWORK_NAME_LIST_MAX) {
         errorf (conn, VIR_ERR_RPC,
@@ -2821,7 +2777,7 @@ remoteNetworkLookupByUUID (virConnectPtr
     virNetworkPtr net;
     remote_network_lookup_by_uuid_args args;
     remote_network_lookup_by_uuid_ret ret;
-    GET_NETWORK_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->networkPrivateData;
 
     memcpy (args.uuid, uuid, VIR_UUID_BUFLEN);
 
@@ -2844,7 +2800,7 @@ remoteNetworkLookupByName (virConnectPtr
     virNetworkPtr net;
     remote_network_lookup_by_name_args args;
     remote_network_lookup_by_name_ret ret;
-    GET_NETWORK_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->networkPrivateData;
 
     args.name = (char *) name;
 
@@ -2866,7 +2822,7 @@ remoteNetworkCreateXML (virConnectPtr co
     virNetworkPtr net;
     remote_network_create_xml_args args;
     remote_network_create_xml_ret ret;
-    GET_NETWORK_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->networkPrivateData;
 
     args.xml = (char *) xmlDesc;
 
@@ -2888,7 +2844,7 @@ remoteNetworkDefineXML (virConnectPtr co
     virNetworkPtr net;
     remote_network_define_xml_args args;
     remote_network_define_xml_ret ret;
-    GET_NETWORK_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->networkPrivateData;
 
     args.xml = (char *) xml;
 
@@ -2908,7 +2864,7 @@ remoteNetworkUndefine (virNetworkPtr net
 remoteNetworkUndefine (virNetworkPtr network)
 {
     remote_network_undefine_args args;
-    GET_NETWORK_PRIVATE (network->conn, -1);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
 
@@ -2924,7 +2880,7 @@ remoteNetworkCreate (virNetworkPtr netwo
 remoteNetworkCreate (virNetworkPtr network)
 {
     remote_network_create_args args;
-    GET_NETWORK_PRIVATE (network->conn, -1);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
 
@@ -2940,7 +2896,7 @@ remoteNetworkDestroy (virNetworkPtr netw
 remoteNetworkDestroy (virNetworkPtr network)
 {
     remote_network_destroy_args args;
-    GET_NETWORK_PRIVATE (network->conn, -1);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
 
@@ -2957,7 +2913,7 @@ remoteNetworkDumpXML (virNetworkPtr netw
 {
     remote_network_dump_xml_args args;
     remote_network_dump_xml_ret ret;
-    GET_NETWORK_PRIVATE (network->conn, NULL);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
     args.flags = flags;
@@ -2977,7 +2933,7 @@ remoteNetworkGetBridgeName (virNetworkPt
 {
     remote_network_get_bridge_name_args args;
     remote_network_get_bridge_name_ret ret;
-    GET_NETWORK_PRIVATE (network->conn, NULL);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
 
@@ -2996,7 +2952,7 @@ remoteNetworkGetAutostart (virNetworkPtr
 {
     remote_network_get_autostart_args args;
     remote_network_get_autostart_ret ret;
-    GET_NETWORK_PRIVATE (network->conn, -1);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
 
@@ -3015,7 +2971,7 @@ remoteNetworkSetAutostart (virNetworkPtr
 remoteNetworkSetAutostart (virNetworkPtr network, int autostart)
 {
     remote_network_set_autostart_args args;
-    GET_NETWORK_PRIVATE (network->conn, -1);
+    struct private_data *priv = network->conn->networkPrivateData;
 
     make_nonnull_network (&args.net, network);
     args.autostart = autostart;
@@ -3071,14 +3027,12 @@ remoteStorageOpen (virConnectPtr conn,
             rflags |= VIR_DRV_OPEN_REMOTE_RO;
         rflags |= VIR_DRV_OPEN_REMOTE_UNIX;
 
-        priv->magic = DEAD;
         priv->sock = -1;
         ret = doRemoteOpen(conn, priv, auth, rflags);
         if (ret != VIR_DRV_OPEN_SUCCESS) {
             conn->storagePrivateData = NULL;
             VIR_FREE(priv);
         } else {
-            priv->magic = MAGIC;
             priv->localUses = 1;
             conn->storagePrivateData = priv;
         }
@@ -3090,7 +3044,8 @@ remoteStorageClose (virConnectPtr conn)
 remoteStorageClose (virConnectPtr conn)
 {
     int ret = 0;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->storagePrivateData;
+
     if (priv->localUses) {
         priv->localUses--;
         if (!priv->localUses) {
@@ -3106,7 +3061,7 @@ remoteNumOfStoragePools (virConnectPtr c
 remoteNumOfStoragePools (virConnectPtr conn)
 {
     remote_num_of_storage_pools_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->storagePrivateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_STORAGE_POOLS,
@@ -3123,7 +3078,7 @@ remoteListStoragePools (virConnectPtr co
     int i;
     remote_list_storage_pools_args args;
     remote_list_storage_pools_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->storagePrivateData;
 
     if (maxnames > REMOTE_STORAGE_POOL_NAME_LIST_MAX) {
         error (conn, VIR_ERR_RPC, _("too many storage pools requested"));
@@ -3160,7 +3115,7 @@ remoteNumOfDefinedStoragePools (virConne
 remoteNumOfDefinedStoragePools (virConnectPtr conn)
 {
     remote_num_of_defined_storage_pools_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->storagePrivateData;
 
     memset (&ret, 0, sizeof ret);
     if (call (conn, priv, 0, REMOTE_PROC_NUM_OF_DEFINED_STORAGE_POOLS,
@@ -3178,7 +3133,7 @@ remoteListDefinedStoragePools (virConnec
     int i;
     remote_list_defined_storage_pools_args args;
     remote_list_defined_storage_pools_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->storagePrivateData;
 
     if (maxnames > REMOTE_STORAGE_POOL_NAME_LIST_MAX) {
         error (conn, VIR_ERR_RPC, _("too many storage pools requested"));
@@ -3219,7 +3174,7 @@ remoteFindStoragePoolSources (virConnect
 {
     remote_find_storage_pool_sources_args args;
     remote_find_storage_pool_sources_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
     const char *emptyString = "";
     char *retval;
 
@@ -3259,7 +3214,7 @@ remoteStoragePoolLookupByUUID (virConnec
     virStoragePoolPtr pool;
     remote_storage_pool_lookup_by_uuid_args args;
     remote_storage_pool_lookup_by_uuid_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     memcpy (args.uuid, uuid, VIR_UUID_BUFLEN);
 
@@ -3282,7 +3237,7 @@ remoteStoragePoolLookupByName (virConnec
     virStoragePoolPtr pool;
     remote_storage_pool_lookup_by_name_args args;
     remote_storage_pool_lookup_by_name_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     args.name = (char *) name;
 
@@ -3304,7 +3259,7 @@ remoteStoragePoolLookupByVolume (virStor
     virStoragePoolPtr pool;
     remote_storage_pool_lookup_by_volume_args args;
     remote_storage_pool_lookup_by_volume_ret ret;
-    GET_STORAGE_PRIVATE (vol->conn, NULL);
+    struct private_data *priv = vol->conn->storagePrivateData;
 
     make_nonnull_storage_vol (&args.vol, vol);
 
@@ -3327,7 +3282,7 @@ remoteStoragePoolCreateXML (virConnectPt
     virStoragePoolPtr pool;
     remote_storage_pool_create_xml_args args;
     remote_storage_pool_create_xml_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     args.xml = (char *) xmlDesc;
     args.flags = flags;
@@ -3350,7 +3305,7 @@ remoteStoragePoolDefineXML (virConnectPt
     virStoragePoolPtr pool;
     remote_storage_pool_define_xml_args args;
     remote_storage_pool_define_xml_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     args.xml = (char *) xml;
     args.flags = flags;
@@ -3371,7 +3326,7 @@ remoteStoragePoolUndefine (virStoragePoo
 remoteStoragePoolUndefine (virStoragePoolPtr pool)
 {
     remote_storage_pool_undefine_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
 
@@ -3387,7 +3342,7 @@ remoteStoragePoolCreate (virStoragePoolP
 remoteStoragePoolCreate (virStoragePoolPtr pool, unsigned int flags)
 {
     remote_storage_pool_create_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.flags = flags;
@@ -3405,7 +3360,7 @@ remoteStoragePoolBuild (virStoragePoolPt
                         unsigned int flags)
 {
     remote_storage_pool_build_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.flags = flags;
@@ -3422,7 +3377,7 @@ remoteStoragePoolDestroy (virStoragePool
 remoteStoragePoolDestroy (virStoragePoolPtr pool)
 {
     remote_storage_pool_destroy_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
 
@@ -3439,7 +3394,7 @@ remoteStoragePoolDelete (virStoragePoolP
                          unsigned int flags)
 {
     remote_storage_pool_delete_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.flags = flags;
@@ -3457,7 +3412,7 @@ remoteStoragePoolRefresh (virStoragePool
                           unsigned int flags)
 {
     remote_storage_pool_refresh_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.flags = flags;
@@ -3475,7 +3430,7 @@ remoteStoragePoolGetInfo (virStoragePool
 {
     remote_storage_pool_get_info_args args;
     remote_storage_pool_get_info_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
 
@@ -3499,7 +3454,7 @@ remoteStoragePoolDumpXML (virStoragePool
 {
     remote_storage_pool_dump_xml_args args;
     remote_storage_pool_dump_xml_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, NULL);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.flags = flags;
@@ -3519,7 +3474,7 @@ remoteStoragePoolGetAutostart (virStorag
 {
     remote_storage_pool_get_autostart_args args;
     remote_storage_pool_get_autostart_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
 
@@ -3538,7 +3493,7 @@ remoteStoragePoolSetAutostart (virStorag
 remoteStoragePoolSetAutostart (virStoragePoolPtr pool, int autostart)
 {
     remote_storage_pool_set_autostart_args args;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.autostart = autostart;
@@ -3557,7 +3512,7 @@ remoteStoragePoolNumOfVolumes (virStorag
 {
     remote_storage_pool_num_of_volumes_args args;
     remote_storage_pool_num_of_volumes_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool(&args.pool, pool);
 
@@ -3576,7 +3531,7 @@ remoteStoragePoolListVolumes (virStorage
     int i;
     remote_storage_pool_list_volumes_args args;
     remote_storage_pool_list_volumes_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, -1);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     if (maxnames > REMOTE_STORAGE_VOL_NAME_LIST_MAX) {
         error (pool->conn, VIR_ERR_RPC, _("too many storage volumes requested"));
@@ -3619,7 +3574,7 @@ remoteStorageVolLookupByName (virStorage
     virStorageVolPtr vol;
     remote_storage_vol_lookup_by_name_args args;
     remote_storage_vol_lookup_by_name_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, NULL);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool(&args.pool, pool);
     args.name = (char *) name;
@@ -3643,7 +3598,7 @@ remoteStorageVolLookupByKey (virConnectP
     virStorageVolPtr vol;
     remote_storage_vol_lookup_by_key_args args;
     remote_storage_vol_lookup_by_key_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     args.key = (char *) key;
 
@@ -3666,7 +3621,7 @@ remoteStorageVolLookupByPath (virConnect
     virStorageVolPtr vol;
     remote_storage_vol_lookup_by_path_args args;
     remote_storage_vol_lookup_by_path_ret ret;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->storagePrivateData;
 
     args.path = (char *) path;
 
@@ -3689,7 +3644,7 @@ remoteStorageVolCreateXML (virStoragePoo
     virStorageVolPtr vol;
     remote_storage_vol_create_xml_args args;
     remote_storage_vol_create_xml_ret ret;
-    GET_STORAGE_PRIVATE (pool->conn, NULL);
+    struct private_data *priv = pool->conn->storagePrivateData;
 
     make_nonnull_storage_pool (&args.pool, pool);
     args.xml = (char *) xmlDesc;
@@ -3712,7 +3667,7 @@ remoteStorageVolDelete (virStorageVolPtr
                         unsigned int flags)
 {
     remote_storage_vol_delete_args args;
-    GET_STORAGE_PRIVATE (vol->conn, -1);
+    struct private_data *priv = vol->conn->storagePrivateData;
 
     make_nonnull_storage_vol (&args.vol, vol);
     args.flags = flags;
@@ -3730,7 +3685,7 @@ remoteStorageVolGetInfo (virStorageVolPt
 {
     remote_storage_vol_get_info_args args;
     remote_storage_vol_get_info_ret ret;
-    GET_STORAGE_PRIVATE (vol->conn, -1);
+    struct private_data *priv = vol->conn->storagePrivateData;
 
     make_nonnull_storage_vol (&args.vol, vol);
 
@@ -3753,7 +3708,7 @@ remoteStorageVolDumpXML (virStorageVolPt
 {
     remote_storage_vol_dump_xml_args args;
     remote_storage_vol_dump_xml_ret ret;
-    GET_STORAGE_PRIVATE (vol->conn, NULL);
+    struct private_data *priv = vol->conn->storagePrivateData;
 
     make_nonnull_storage_vol (&args.vol, vol);
     args.flags = flags;
@@ -3773,7 +3728,7 @@ remoteStorageVolGetPath (virStorageVolPt
 {
     remote_storage_vol_get_path_args args;
     remote_storage_vol_get_path_ret ret;
-    GET_NETWORK_PRIVATE (vol->conn, NULL);
+    struct private_data *priv = vol->conn->storagePrivateData;
 
     make_nonnull_storage_vol (&args.vol, vol);
 
@@ -3813,7 +3768,8 @@ static int remoteDevMonClose(virConnectP
 static int remoteDevMonClose(virConnectPtr conn)
 {
     int ret = 0;
-    GET_DEVMON_PRIVATE (conn, -1);
+    struct private_data *priv = conn->devMonPrivateData;
+
     if (priv->localUses) {
         priv->localUses--;
         if (!priv->localUses) {
@@ -3831,7 +3787,7 @@ static int remoteNodeNumOfDevices(virCon
 {
     remote_node_num_of_devices_args args;
     remote_node_num_of_devices_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->devMonPrivateData;
 
     args.cap = cap ? (char **)&cap : NULL;
     args.flags = flags;
@@ -3855,7 +3811,7 @@ static int remoteNodeListDevices(virConn
     int i;
     remote_node_list_devices_args args;
     remote_node_list_devices_ret ret;
-    GET_STORAGE_PRIVATE (conn, -1);
+    struct private_data *priv = conn->devMonPrivateData;
 
     if (maxnames > REMOTE_NODE_DEVICE_NAME_LIST_MAX) {
         error (conn, VIR_ERR_RPC, _("too many device names requested"));
@@ -3897,7 +3853,7 @@ static virNodeDevicePtr remoteNodeDevice
     remote_node_device_lookup_by_name_args args;
     remote_node_device_lookup_by_name_ret ret;
     virNodeDevicePtr dev;
-    GET_STORAGE_PRIVATE (conn, NULL);
+    struct private_data *priv = conn->devMonPrivateData;
 
     args.name = (char *)name;
 
@@ -3919,7 +3875,7 @@ static char *remoteNodeDeviceDumpXML(vir
 {
     remote_node_device_dump_xml_args args;
     remote_node_device_dump_xml_ret ret;
-    GET_STORAGE_PRIVATE (dev->conn, NULL);
+    struct private_data *priv = dev->conn->devMonPrivateData;
 
     args.name = dev->name;
     args.flags = flags;
@@ -3938,7 +3894,7 @@ static char *remoteNodeDeviceGetParent(v
 {
     remote_node_device_get_parent_args args;
     remote_node_device_get_parent_ret ret;
-    GET_STORAGE_PRIVATE (dev->conn, NULL);
+    struct private_data *priv = dev->conn->devMonPrivateData;
 
     args.name = dev->name;
 
@@ -3956,7 +3912,7 @@ static int remoteNodeDeviceNumOfCaps(vir
 {
     remote_node_device_num_of_caps_args args;
     remote_node_device_num_of_caps_ret ret;
-    GET_STORAGE_PRIVATE (dev->conn, -1);
+    struct private_data *priv = dev->conn->devMonPrivateData;
 
     args.name = dev->name;
 
@@ -3976,7 +3932,7 @@ static int remoteNodeDeviceListCaps(virN
     int i;
     remote_node_device_list_caps_args args;
     remote_node_device_list_caps_ret ret;
-    GET_STORAGE_PRIVATE (dev->conn, -1);
+    struct private_data *priv = dev->conn->devMonPrivateData;
 
     if (maxnames > REMOTE_NODE_DEVICE_CAPS_LIST_MAX) {
         error (dev->conn, VIR_ERR_RPC, _("too many capability names requested"));

-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|


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