5:07 p.m.
Currently the VMware version check code only supports two types of
VMware backends, Workstation and Player. But in the near future we will
have an additional one so we need to support more. Additionally, we
discover and cache the path to the vmrun binary so we should use that
path when using the cooresponding binary from the VMware VIX SDK.
---
change from v1:
* Added default case so we don't potentially pass NULL to virCommand
---
src/vmware/vmware_conf.c | 28 +++++++++++++++++++++++++---
src/vmware/vmware_driver.c | 20 ++++++++++++++++++--
2 files changed, 43 insertions(+), 5 deletions(-)
diff --git a/src/vmware/vmware_conf.c b/src/vmware/vmware_conf.c
index 261a4f6..c11c92d 100644
--- a/src/vmware/vmware_conf.c
+++ b/src/vmware/vmware_conf.c
@@ -257,10 +257,30 @@ vmwareExtractVersion(struct vmware_driver *driver)
{
unsigned long version = 0;
int ret = -1;
- virCommandPtr cmd;
+ virCommandPtr cmd = NULL;
char * outbuf = NULL;
- const char * bin = (driver->type == VMWARE_DRIVER_PLAYER) ?
- "vmplayer" : "vmware";
+ char *bin = NULL;
+ char *vmwarePath = NULL;
+
+ if ((vmwarePath = mdir_name(driver->vmrun)) == NULL)
+ goto cleanup;
+
+ switch (driver->type) {
+ case VMWARE_DRIVER_PLAYER:
+ if (virAsprintf(&bin, "%s/%s", vmwarePath,
"vmplayer"))
+ goto cleanup;
+ break;
+
+ case VMWARE_DRIVER_WORKSTATION:
+ if (virAsprintf(&bin, "%s/%s", vmwarePath,
"vmware"))
+ goto cleanup;
+ break;
+
+ default:
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ _("invalid driver type for version detection"));
+ goto cleanup;
+ }
cmd = virCommandNewArgList(bin, "-v", NULL);
virCommandSetOutputBuffer(cmd, &outbuf);
@@ -276,6 +296,8 @@ vmwareExtractVersion(struct vmware_driver *driver)
cleanup:
virCommandFree(cmd);
VIR_FREE(outbuf);
+ VIR_FREE(bin);
+ VIR_FREE(vmwarePath);
return ret;
}
diff --git a/src/vmware/vmware_driver.c b/src/vmware/vmware_driver.c
index 4e56971..01d65ed 100644
--- a/src/vmware/vmware_driver.c
+++ b/src/vmware/vmware_driver.c
@@ -93,6 +93,7 @@ vmwareConnectOpen(virConnectPtr conn,
{
struct vmware_driver *driver;
size_t i;
+ char *tmp;
virCheckFlags(VIR_CONNECT_RO, VIR_DRV_OPEN_ERROR);
@@ -144,8 +145,23 @@ vmwareConnectOpen(virConnectPtr conn,
if (virMutexInit(&driver->lock) < 0)
goto cleanup;
- driver->type = STRNEQ(conn->uri->scheme, "vmwareplayer") ?
- VMWARE_DRIVER_WORKSTATION : VMWARE_DRIVER_PLAYER;
+ if ((tmp = STRSKIP(conn->uri->scheme, "vmware")) == NULL) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, _("unable to parse URI "
+ "scheme '%s'"), conn->uri->scheme);
+ goto cleanup;
+ }
+
+ driver->type = -1;
+ for (i = 0; i < VMWARE_DRIVER_LAST; i++) {
+ if (STREQ(tmp, vmwareDriverTypeToString(i)))
+ driver->type = i;
+ }
+
+ if (driver->type == -1) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, _("unable to find valid "
+ "requested VMware backend '%s'"), tmp);
+ goto cleanup;
+ }
if (!(driver->domains = virDomainObjListNew()))
goto cleanup;
--
1.8.1.5
5:07 p.m.
New subject: [libvirt] [PATCHv2 2/2] VMware: Initial VMware Fusion support
Add support for VMware Fusion in the existing VMware driver. Connect
via the URI vmwarefusion:///session
---
change from v1:
* ifdef PATH for Apple machines only
---
docs/drvvmware.html.in | 15 +++++++++++----
src/vmware/vmware_conf.c | 8 +++++++-
src/vmware/vmware_conf.h | 1 +
src/vmware/vmware_driver.c | 5 +++++
4 files changed, 24 insertions(+), 5 deletions(-)
diff --git a/docs/drvvmware.html.in b/docs/drvvmware.html.in
index 9ee0ebd..240afd0 100644
--- a/docs/drvvmware.html.in
+++ b/docs/drvvmware.html.in
@@ -2,10 +2,11 @@
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<body>
- <h1>VMware Workstation / Player hypervisors driver</h1>
+ <h1>VMware Workstation / Player / Fusion hypervisors driver</h1>
<p>
- The libvirt VMware Workstation driver should be able to manage any Workstation
and
- Player version supported by the VMware VIX API. See the compatibility list
+ The libvirt VMware driver should be able to manage any Workstation,
+ Player, Fusion version supported by the VMware VIX API. See the
+ compatibility list
<a
href="http://www.vmware.com/support/developer/vix-api/vix110_referen...;.
</p>
<p>
@@ -21,17 +22,22 @@
The <a href="http://www.vmware.com/">VMware Workstation and
Player</a> hypervisors
</li>
+ <li>
+ The <a href="http://www.vmware.com/fusion">VMware
Fusion</a>
+ hypervisor
+ </li>
</ul>
<h2>Connections to VMware driver</h2>
<p>
The libvirt VMware driver provides per-user drivers (the "session"
instance).
- Two uris are available:
+ Three uris are available:
</p>
<ul>
<li>"vmwareplayer" for VMware Player</li>
<li>"vmwarews" for VMware Workstation</li>
+ <li>"vmwarefusion" for VMware Fusion</li>
</ul>
<p>
Some example connection URIs for the driver are:
@@ -40,6 +46,7 @@
<pre>
vmwareplayer:///session (local access to VMware Player per-user
instance)
vmwarews:///session (local access to VMware Workstation per-user
instance)
+vmwarefusion:///session (local access to VMware Fusion per-user
instance)
vmwarews+tcp://user@example.com/session (remote access to VMware Workstation,
SASl/Kerberos)
vmwarews+ssh://user@example.com/session (remote access to VMware Workstation, SSH
tunnelled)
</pre>
diff --git a/src/vmware/vmware_conf.c b/src/vmware/vmware_conf.c
index c11c92d..9197381 100644
--- a/src/vmware/vmware_conf.c
+++ b/src/vmware/vmware_conf.c
@@ -37,7 +37,8 @@
VIR_ENUM_IMPL(vmwareDriver, VMWARE_DRIVER_LAST,
"player",
- "ws");
+ "ws",
+ "fusion");
/* Free all memory associated with a vmware_driver structure */
void
@@ -276,6 +277,11 @@ vmwareExtractVersion(struct vmware_driver *driver)
goto cleanup;
break;
+ case VMWARE_DRIVER_FUSION:
+ if (virAsprintf(&bin, "%s/%s", vmwarePath,
"vmware-vmx"))
+ goto cleanup;
+ break;
+
default:
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("invalid driver type for version detection"));
diff --git a/src/vmware/vmware_conf.h b/src/vmware/vmware_conf.h
index a4fd855..b9fca6c 100644
--- a/src/vmware/vmware_conf.h
+++ b/src/vmware/vmware_conf.h
@@ -33,6 +33,7 @@
enum vmwareDriverType {
VMWARE_DRIVER_PLAYER = 0, /* VMware Player */
VMWARE_DRIVER_WORKSTATION = 1, /* VMware Workstation */
+ VMWARE_DRIVER_FUSION = 2, /* VMware Fusion */
VMWARE_DRIVER_LAST, /* required last item */
};
diff --git a/src/vmware/vmware_driver.c b/src/vmware/vmware_driver.c
index 01d65ed..4db2ad2 100644
--- a/src/vmware/vmware_driver.c
+++ b/src/vmware/vmware_driver.c
@@ -40,6 +40,10 @@
*/
static const char * const vmrun_candidates[] = {
"vmrun",
+#ifdef __APPLE__
+ "/Applications/VMware Fusion.app/Contents/Library/vmrun",
+ "/Library/Application Support/VMware Fusion/vmrun",
+#endif /* __APPLE__ */
};
static void
@@ -103,6 +107,7 @@ vmwareConnectOpen(virConnectPtr conn,
} else {
if (conn->uri->scheme == NULL ||
(STRNEQ(conn->uri->scheme, "vmwareplayer") &&
+ STRNEQ(conn->uri->scheme, "vmwarefusion") &&
STRNEQ(conn->uri->scheme, "vmwarews")))
return VIR_DRV_OPEN_DECLINED;
--
1.8.1.5
4080
days inactive
4080
days old
1 comments
1 participants
participants (1)
-
Doug Goldstein