Added missing return(-1) to cpu APIs. #ifdef out methods not used by proxy

diff --git a/ChangeLog b/ChangeLog
index 54df75baffed0db62a9178951c60b11b7f42363d..804259dc1d0293a07a5ac537a8b125aaf2ccbf64 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+Mon Sep  4 14:13:24 EDT 2006 Daniel Berrange <berrange@redhat.com>
+
+       * src/xen_intern$al.h: Added missing return(-1) statements
+       in xenHypervisorGetVcpus and xenHypervisorPinVcpu methods.
+       #ifndef out the pause/resume/etc methods from proxy code
+       
 Sun Sep  3 12:50:12 EDT 2006 Daniel Berrange <berrange@redhat.com>
 
        * src/conf.c: Allow '_' in variable names, and don't fail if there

diff --git a/src/xen_internal.c b/src/xen_internal.c
index 94082a5a1c0f470edab7dabde816f8647fe9c8af..f58b58b329e712888dccafe6e5f2226f241fe36f 100644 (file)
--- a/src/xen_internal.c
+++ b/src/xen_internal.c
@@ -448,6 +448,7 @@ xenHypervisorDoV2Sys(int handle, xen_op_v2_sys* op)
     return (0);
 }
 
+#ifndef PROXY
 /**
  * xenHypervisorDoV2Dom:
  * @handle: the handle to the Xen hypervisor
@@ -489,6 +490,7 @@ xenHypervisorDoV2Dom(int handle, xen_op_v2_dom* op)
 
     return (0);
 }
+#endif /* PROXY */
 
 /**
  * virXen_getdomaininfolist:
@@ -557,6 +559,7 @@ virXen_getdomaininfolist(int handle, int first_domain, int maxids,
     return(ret);
 }
 
+#ifndef PROXY
 /**
  * virXen_pausedomain:
  * @handle: the hypervisor handle
@@ -821,6 +824,7 @@ virXen_setvcpumap(int handle, int id, unsigned int vcpu,
     }
     return(ret);
 }
+#endif /* !PROXY*/
 
 /**
  * xenHypervisorInit:
@@ -1421,6 +1425,8 @@ xenHypervisorPinVcpu(virDomainPtr domain, unsigned int vcpu,
 
     if ((domain == NULL) || (domain->conn == NULL) ||
         (domain->conn->handle < 0) || (cpumap == NULL) || (maplen < 1))
+        return (-1);
+
     ret = virXen_setvcpumap(domain->conn->handle, domain->handle, vcpu,
                             cpumap, maplen);
     if (ret < 0)
@@ -1502,4 +1508,5 @@ xenHypervisorGetVcpus(virDomainPtr domain, virVcpuInfoPtr info, int maxinfo,
     }
     return nbinfo;
 #endif
+    return -1;
 }