3:36 p.m.
This patchset begins reworking of parallels driver. We have
published Opensource version of parallels SDK (under LGPL license),
so libvirt can link with it.
Dmitry Guryanov (2):
parallels: build with parallels SDK
parallels: login to parallels SDK
configure.ac | 11 +++++-----
po/POTFILES.in | 1 +
src/Makefile.am | 8 +++++--
src/parallels/parallels_driver.c | 46 +++++++++++++++++++++++++++++++++++++++-
src/parallels/parallels_utils.h | 3 +++
5 files changed, 60 insertions(+), 9 deletions(-)
--
1.9.3
3:36 p.m.
New subject: [libvirt] [PATCH 1/2] parallels: build with parallels SDK
Executing prlctl command is not an optimal way to interact with
Parallels Cloud Server (PCS), it's better to use parallels SDK,
which is a remote API to paralles dispatcher service.
We prepared opensource version of this SDK and published it on
github, it's distributed under LGPL license. Here is a git repo:
https://github.com/Parallels/parallels-sdk.
To build with parallels SDK user should get compiler and linker
options from pkg-config 'parallels-sdk' file. So fix checks in
configure script and build with parallels SDK, if that pkg-config
file exists and add gcc options to makefile.
Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
---
configure.ac | 11 +++++------
src/Makefile.am | 4 +++-
2 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/configure.ac b/configure.ac
index f93c6c2..4318d24 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1046,15 +1046,14 @@ dnl
dnl Checks for the Parallels driver
dnl
+PKG_CHECK_MODULES([PARALLELS_SDK], [parallels-sdk], [PARALLELS_SDK_FOUND=yes],
[PARALLELS_SDK_FOUND=no])
+
if test "$with_parallels" = "check"; then
- with_parallels=$with_linux
- if test ! $host_cpu = 'x86_64'; then
- with_parallels=no
- fi
+ with_parallels=$PARALLELS_SDK_FOUND
fi
-if test "$with_parallels" = "yes" && test
"$with_linux" = "no"; then
- AC_MSG_ERROR([The Parallels driver can be enabled on Linux only.])
+if test "$with_parallels" = "yes" && test
"$PARALLELS_SDK_FOUND" = "no"; then
+ AC_MSG_ERROR([Parallels Virtualization SDK is needed to build the Parallels
driver.])
fi
if test "$with_parallels" = "yes"; then
diff --git a/src/Makefile.am b/src/Makefile.am
index 538530e..dad7c7f 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1352,7 +1352,9 @@ if WITH_PARALLELS
noinst_LTLIBRARIES += libvirt_driver_parallels.la
libvirt_la_BUILT_LIBADD += libvirt_driver_parallels.la
libvirt_driver_parallels_la_CFLAGS = \
- -I$(top_srcdir)/src/conf $(AM_CFLAGS)
+ -I$(top_srcdir)/src/conf $(AM_CFLAGS) \
+ $(PARALLELS_SDK_CFLAGS)
+libvirt_driver_parallels_la_LIBADD = $(PARALLELS_SDK_LIBS)
libvirt_driver_parallels_la_SOURCES = $(PARALLELS_DRIVER_SOURCES)
endif WITH_PARALLELS
--
1.9.3
9:38 a.m.
New subject: [libvirt] [PATCH 1/2] parallels: build with parallels SDK
On 21.08.2014 22:36, Dmitry Guryanov wrote:
Executing prlctl command is not an optimal way to interact with
Parallels Cloud Server (PCS), it's better to use parallels SDK,
which is a remote API to paralles dispatcher service.
We prepared opensource version of this SDK and published it on
github, it's distributed under LGPL license. Here is a git repo:
https://github.com/Parallels/parallels-sdk.
To build with parallels SDK user should get compiler and linker
options from pkg-config 'parallels-sdk' file. So fix checks in
configure script and build with parallels SDK, if that pkg-config
file exists and add gcc options to makefile.
Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
---
configure.ac | 11 +++++------
src/Makefile.am | 4 +++-
2 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/configure.ac b/configure.ac
index f93c6c2..4318d24 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1046,15 +1046,14 @@ dnl
dnl Checks for the Parallels driver
dnl
+PKG_CHECK_MODULES([PARALLELS_SDK], [parallels-sdk], [PARALLELS_SDK_FOUND=yes],
[PARALLELS_SDK_FOUND=no])
+
if test "$with_parallels" = "check"; then
- with_parallels=$with_linux
- if test ! $host_cpu = 'x86_64'; then
- with_parallels=no
- fi
+ with_parallels=$PARALLELS_SDK_FOUND
fi
-if test "$with_parallels" = "yes" && test
"$with_linux" = "no"; then
- AC_MSG_ERROR([The Parallels driver can be enabled on Linux only.])
+if test "$with_parallels" = "yes" && test
"$PARALLELS_SDK_FOUND" = "no"; then
+ AC_MSG_ERROR([Parallels Virtualization SDK is needed to build the Parallels
driver.])
fi
if test "$with_parallels" = "yes"; then
diff --git a/src/Makefile.am b/src/Makefile.am
index 538530e..dad7c7f 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1352,7 +1352,9 @@ if WITH_PARALLELS
noinst_LTLIBRARIES += libvirt_driver_parallels.la
libvirt_la_BUILT_LIBADD += libvirt_driver_parallels.la
libvirt_driver_parallels_la_CFLAGS = \
- -I$(top_srcdir)/src/conf $(AM_CFLAGS)
+ -I$(top_srcdir)/src/conf $(AM_CFLAGS) \
+ $(PARALLELS_SDK_CFLAGS)
+libvirt_driver_parallels_la_LIBADD = $(PARALLELS_SDK_LIBS)
libvirt_driver_parallels_la_SOURCES = $(PARALLELS_DRIVER_SOURCES)
endif WITH_PARALLELS
So previously, this was enabled for x86_64 linux hosts, now you need to
have SDK. I understand why you are doing that but perhaps do we want to
enable backward compatibility and let users use prlctl command?
Michal
10:50 a.m.
New subject: [libvirt] [PATCH 1/2] parallels: build with parallels SDK
On Friday 22 August 2014 16:38:17 Michal Privoznik wrote:
On 21.08.2014 22:36, Dmitry Guryanov wrote:
> Executing prlctl command is not an optimal way to interact with
> Parallels Cloud Server (PCS), it's better to use parallels SDK,
> which is a remote API to paralles dispatcher service.
>
> We prepared opensource version of this SDK and published it on
> github, it's distributed under LGPL license. Here is a git repo:
> https://github.com/Parallels/parallels-sdk.
>
> To build with parallels SDK user should get compiler and linker
> options from pkg-config 'parallels-sdk' file. So fix checks in
> configure script and build with parallels SDK, if that pkg-config
> file exists and add gcc options to makefile.
>
> Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
> ---
>
> configure.ac | 11 +++++------
> src/Makefile.am | 4 +++-
> 2 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index f93c6c2..4318d24 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1046,15 +1046,14 @@ dnl
>
> dnl Checks for the Parallels driver
> dnl
>
> +PKG_CHECK_MODULES([PARALLELS_SDK], [parallels-sdk],
> [PARALLELS_SDK_FOUND=yes], [PARALLELS_SDK_FOUND=no]) +
>
> if test "$with_parallels" = "check"; then
>
> - with_parallels=$with_linux
> - if test ! $host_cpu = 'x86_64'; then
> - with_parallels=no
> - fi
> + with_parallels=$PARALLELS_SDK_FOUND
>
> fi
>
> -if test "$with_parallels" = "yes" && test
"$with_linux" = "no"; then
> - AC_MSG_ERROR([The Parallels driver can be enabled on Linux only.])
> +if test "$with_parallels" = "yes" && test
"$PARALLELS_SDK_FOUND" = "no";
> then + AC_MSG_ERROR([Parallels Virtualization SDK is needed to build
> the Parallels driver.])>
> fi
>
> if test "$with_parallels" = "yes"; then
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 538530e..dad7c7f 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -1352,7 +1352,9 @@ if WITH_PARALLELS
>
> noinst_LTLIBRARIES += libvirt_driver_parallels.la
> libvirt_la_BUILT_LIBADD += libvirt_driver_parallels.la
> libvirt_driver_parallels_la_CFLAGS = \
>
> - -I$(top_srcdir)/src/conf $(AM_CFLAGS)
> + -I$(top_srcdir)/src/conf $(AM_CFLAGS) \
> + $(PARALLELS_SDK_CFLAGS)
> +libvirt_driver_parallels_la_LIBADD = $(PARALLELS_SDK_LIBS)
>
> libvirt_driver_parallels_la_SOURCES = $(PARALLELS_DRIVER_SOURCES)
> endif WITH_PARALLELS
So previously, this was enabled for x86_64 linux hosts, now you need to
have SDK. I understand why you are doing that but perhaps do we want to
enable backward compatibility and let users use prlctl command?
Actually backward compatibility is not needed, because there was only several users
inside company. Also PCS is a full distribution to be installed on baremetal hosts, so
starting with the next update we will provide this new version of SDK and libvirt
package, built with parallels support. Users, which connect to PCS from their desktops
will use parallels+ssh scheme and still will be able to use this driver.
Michal
--
Dmitry Guryanov
3:36 p.m.
New subject: [libvirt] [PATCH 2/2] parallels: login to parallels SDK
Add files parallels_sdk.c and parallels_sdk.h for code
which works with SDK, so libvirt's code will not mix with
dealing with parallels SDK.
To use Parallels SDK you must first call PrlApi_InitEx function,
and then you will be able to connect to a server with
PrlSrv_LoginLocalEx function. When you've done you must call
PrlApi_Deinit. So let's call PrlApi_InitEx on first .connectOpen,
count number of connections and deinitialize, when this counter
becomes zero.
Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
---
po/POTFILES.in | 1 +
src/Makefile.am | 4 +++-
src/parallels/parallels_driver.c | 46 +++++++++++++++++++++++++++++++++++++++-
src/parallels/parallels_utils.h | 3 +++
4 files changed, 52 insertions(+), 2 deletions(-)
diff --git a/po/POTFILES.in b/po/POTFILES.in
index f17b35f..4c1302d 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -96,6 +96,7 @@ src/openvz/openvz_driver.c
src/openvz/openvz_util.c
src/parallels/parallels_driver.c
src/parallels/parallels_network.c
+src/parallels/parallels_sdk.c
src/parallels/parallels_utils.c
src/parallels/parallels_utils.h
src/parallels/parallels_storage.c
diff --git a/src/Makefile.am b/src/Makefile.am
index dad7c7f..d4c6465 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -787,7 +787,9 @@ PARALLELS_DRIVER_SOURCES = \
parallels/parallels_utils.c \
parallels/parallels_utils.h \
parallels/parallels_storage.c \
- parallels/parallels_network.c
+ parallels/parallels_network.c \
+ parallels/parallels_sdk.h \
+ parallels/parallels_sdk.c
BHYVE_DRIVER_SOURCES = \
bhyve/bhyve_capabilities.c \
diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c
index bb9538f..7dc9963 100644
--- a/src/parallels/parallels_driver.c
+++ b/src/parallels/parallels_driver.c
@@ -55,6 +55,7 @@
#include "parallels_driver.h"
#include "parallels_utils.h"
+#include "parallels_sdk.h"
#define VIR_FROM_THIS VIR_FROM_PARALLELS
@@ -73,6 +74,9 @@ VIR_LOG_INIT("parallels.parallels_driver");
#define IS_CT(def) (STREQ_NULLABLE(def->os.type, "exe"))
+unsigned int numConns = 0;
+virMutex numConnsLock;
+
static int parallelsConnectClose(virConnectPtr conn);
static const char * parallelsGetDiskBusName(int bus) {
@@ -929,9 +933,25 @@ parallelsOpenDefault(virConnectPtr conn)
if (virMutexInit(&privconn->lock) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("cannot initialize mutex"));
- goto error;
+ goto err_free;
}
+ virMutexLock(&numConnsLock);
+ numConns++;
+
+ if (numConns == 1) {
+ if (prlsdkInit()) {
+ VIR_DEBUG("%s", _("Can't initialize Parallels
SDK"));
+ virMutexUnlock(&numConnsLock);
+ goto err_free;
+ }
+ }
+
+ virMutexUnlock(&numConnsLock);
+
+ if (prlsdkConnect(privconn) < 0)
+ goto err_free;
+
if (!(privconn->caps = parallelsBuildCapabilities()))
goto error;
@@ -953,6 +973,9 @@ parallelsOpenDefault(virConnectPtr conn)
virObjectUnref(privconn->domains);
virObjectUnref(privconn->caps);
virStoragePoolObjListFree(&privconn->pools);
+ prlsdkDisconnect(privconn);
+ prlsdkDeinit();
+ err_free:
VIR_FREE(privconn);
return VIR_DRV_OPEN_ERROR;
}
@@ -999,8 +1022,17 @@ parallelsConnectClose(virConnectPtr conn)
virObjectUnref(privconn->caps);
virObjectUnref(privconn->xmlopt);
virObjectUnref(privconn->domains);
+ prlsdkDisconnect(privconn);
conn->privateData = NULL;
+ virMutexLock(&numConnsLock);
+ numConns--;
+
+ if (numConns == 0)
+ prlsdkDeinit();
+
+ virMutexUnlock(&numConnsLock);
+
parallelsDriverUnlock(privconn);
virMutexDestroy(&privconn->lock);
@@ -2453,6 +2485,12 @@ static virDriver parallelsDriver = {
.connectIsAlive = parallelsConnectIsAlive, /* 1.2.5 */
};
+static virStateDriver parallelsStateDriver = {
+ .name = "parallels",
+ .stateInitialize = parallelsStateInitialize,
+ .stateCleanup = parallelsStateCleanup,
+};
+
/**
* parallelsRegister:
*
@@ -2471,6 +2509,12 @@ parallelsRegister(void)
VIR_FREE(prlctl_path);
+ if (virMutexInit(&numConnsLock) < 0) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ _("cannot initialize mutex"));
+ return 0;
+ }
+
if (virRegisterDriver(¶llelsDriver) < 0)
return -1;
if (parallelsStorageRegister())
diff --git a/src/parallels/parallels_utils.h b/src/parallels/parallels_utils.h
index 599e2c5..095c104 100644
--- a/src/parallels/parallels_utils.h
+++ b/src/parallels/parallels_utils.h
@@ -23,6 +23,8 @@
#ifndef PARALLELS_UTILS_H
# define PARALLELS_UTILS_H
+# include <Parallels.h>
+
# include "driver.h"
# include "conf/domain_conf.h"
# include "conf/storage_conf.h"
@@ -40,6 +42,7 @@
struct _parallelsConn {
virMutex lock;
virDomainObjListPtr domains;
+ PRL_HANDLE server;
virStoragePoolObjList pools;
virNetworkObjList networks;
virCapsPtr caps;
--
1.9.3
9:38 a.m.
New subject: [libvirt] [PATCH 2/2] parallels: login to parallels SDK
On 21.08.2014 22:36, Dmitry Guryanov wrote:
Add files parallels_sdk.c and parallels_sdk.h for code
which works with SDK, so libvirt's code will not mix with
dealing with parallels SDK.
To use Parallels SDK you must first call PrlApi_InitEx function,
and then you will be able to connect to a server with
PrlSrv_LoginLocalEx function. When you've done you must call
PrlApi_Deinit. So let's call PrlApi_InitEx on first .connectOpen,
count number of connections and deinitialize, when this counter
becomes zero.
Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
---
po/POTFILES.in | 1 +
src/Makefile.am | 4 +++-
src/parallels/parallels_driver.c | 46 +++++++++++++++++++++++++++++++++++++++-
src/parallels/parallels_utils.h | 3 +++
4 files changed, 52 insertions(+), 2 deletions(-)
This fails 'make syntax-check' for me.
diff --git a/po/POTFILES.in b/po/POTFILES.in
index f17b35f..4c1302d 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -96,6 +96,7 @@ src/openvz/openvz_driver.c
src/openvz/openvz_util.c
src/parallels/parallels_driver.c
src/parallels/parallels_network.c
+src/parallels/parallels_sdk.c
src/parallels/parallels_utils.c
src/parallels/parallels_utils.h
src/parallels/parallels_storage.c
diff --git a/src/Makefile.am b/src/Makefile.am
index dad7c7f..d4c6465 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -787,7 +787,9 @@ PARALLELS_DRIVER_SOURCES = \
parallels/parallels_utils.c \
parallels/parallels_utils.h \
parallels/parallels_storage.c \
- parallels/parallels_network.c
+ parallels/parallels_network.c \
+ parallels/parallels_sdk.h \
+ parallels/parallels_sdk.c
BHYVE_DRIVER_SOURCES = \
bhyve/bhyve_capabilities.c \
diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c
index bb9538f..7dc9963 100644
--- a/src/parallels/parallels_driver.c
+++ b/src/parallels/parallels_driver.c
@@ -55,6 +55,7 @@
#include "parallels_driver.h"
#include "parallels_utils.h"
+#include "parallels_sdk.h"
#define VIR_FROM_THIS VIR_FROM_PARALLELS
@@ -73,6 +74,9 @@ VIR_LOG_INIT("parallels.parallels_driver");
#define IS_CT(def) (STREQ_NULLABLE(def->os.type, "exe"))
+unsigned int numConns = 0;
+virMutex numConnsLock;
1: ^^^
+
static int parallelsConnectClose(virConnectPtr conn);
static const char * parallelsGetDiskBusName(int bus) {
@@ -929,9 +933,25 @@ parallelsOpenDefault(virConnectPtr conn)
if (virMutexInit(&privconn->lock) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("cannot initialize mutex"));
- goto error;
+ goto err_free;
}
+ virMutexLock(&numConnsLock);
+ numConns++;
+
+ if (numConns == 1) {
+ if (prlsdkInit()) {
+ VIR_DEBUG("%s", _("Can't initialize Parallels
SDK"));
+ virMutexUnlock(&numConnsLock);
+ goto err_free;
+ }
+ }
+
+ virMutexUnlock(&numConnsLock);
+
+ if (prlsdkConnect(privconn) < 0)
+ goto err_free;
+
if (!(privconn->caps = parallelsBuildCapabilities()))
goto error;
@@ -953,6 +973,9 @@ parallelsOpenDefault(virConnectPtr conn)
virObjectUnref(privconn->domains);
virObjectUnref(privconn->caps);
virStoragePoolObjListFree(&privconn->pools);
+ prlsdkDisconnect(privconn);
+ prlsdkDeinit();
+ err_free:
VIR_FREE(privconn);
return VIR_DRV_OPEN_ERROR;
}
@@ -999,8 +1022,17 @@ parallelsConnectClose(virConnectPtr conn)
virObjectUnref(privconn->caps);
virObjectUnref(privconn->xmlopt);
virObjectUnref(privconn->domains);
+ prlsdkDisconnect(privconn);
conn->privateData = NULL;
+ virMutexLock(&numConnsLock);
+ numConns--;
+
+ if (numConns == 0)
+ prlsdkDeinit();
+
+ virMutexUnlock(&numConnsLock);
+
parallelsDriverUnlock(privconn);
virMutexDestroy(&privconn->lock);
@@ -2453,6 +2485,12 @@ static virDriver parallelsDriver = {
.connectIsAlive = parallelsConnectIsAlive, /* 1.2.5 */
};
+static virStateDriver parallelsStateDriver = {
+ .name = "parallels",
+ .stateInitialize = parallelsStateInitialize,
+ .stateCleanup = parallelsStateCleanup,
+};
+
This is not called (read registered) anywhere so the
state{Initialize,Cleanup} members are not gonna be called at all.
Moreover, if you intent to turn this into state driver, then the [1] are
good candidates to become members of some privateData struct of the
state driver.
/**
* parallelsRegister:
*
@@ -2471,6 +2509,12 @@ parallelsRegister(void)
VIR_FREE(prlctl_path);
+ if (virMutexInit(&numConnsLock) < 0) {
+ virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+ _("cannot initialize mutex"));
+ return 0;
+ }
+
if (virRegisterDriver(¶llelsDriver) < 0)
return -1;
if (parallelsStorageRegister())
diff --git a/src/parallels/parallels_utils.h b/src/parallels/parallels_utils.h
index 599e2c5..095c104 100644
--- a/src/parallels/parallels_utils.h
+++ b/src/parallels/parallels_utils.h
@@ -23,6 +23,8 @@
#ifndef PARALLELS_UTILS_H
# define PARALLELS_UTILS_H
+# include <Parallels.h>
+
# include "driver.h"
# include "conf/domain_conf.h"
# include "conf/storage_conf.h"
@@ -40,6 +42,7 @@
struct _parallelsConn {
virMutex lock;
virDomainObjListPtr domains;
+ PRL_HANDLE server;
virStoragePoolObjList pools;
virNetworkObjList networks;
virCapsPtr caps;
Michal
11:06 a.m.
New subject: [libvirt] [PATCH 2/2] parallels: login to parallels SDK
On Friday 22 August 2014 16:38:11 Michal Privoznik wrote:
On 21.08.2014 22:36, Dmitry Guryanov wrote:
> Add files parallels_sdk.c and parallels_sdk.h for code
> which works with SDK, so libvirt's code will not mix with
> dealing with parallels SDK.
>
> To use Parallels SDK you must first call PrlApi_InitEx function,
> and then you will be able to connect to a server with
> PrlSrv_LoginLocalEx function. When you've done you must call
> PrlApi_Deinit. So let's call PrlApi_InitEx on first .connectOpen,
> count number of connections and deinitialize, when this counter
> becomes zero.
>
> Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
> ---
>
> po/POTFILES.in | 1 +
> src/Makefile.am | 4 +++-
> src/parallels/parallels_driver.c | 46
> +++++++++++++++++++++++++++++++++++++++-
> src/parallels/parallels_utils.h | 3 +++
> 4 files changed, 52 insertions(+), 2 deletions(-)
This fails 'make syntax-check' for me.
> diff --git a/po/POTFILES.in b/po/POTFILES.in
> index f17b35f..4c1302d 100644
> --- a/po/POTFILES.in
> +++ b/po/POTFILES.in
> @@ -96,6 +96,7 @@ src/openvz/openvz_driver.c
>
> src/openvz/openvz_util.c
> src/parallels/parallels_driver.c
> src/parallels/parallels_network.c
>
> +src/parallels/parallels_sdk.c
>
> src/parallels/parallels_utils.c
> src/parallels/parallels_utils.h
> src/parallels/parallels_storage.c
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index dad7c7f..d4c6465 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -787,7 +787,9 @@ PARALLELS_DRIVER_SOURCES =
\
>
> parallels/parallels_utils.c \
> parallels/parallels_utils.h \
> parallels/parallels_storage.c \
>
> - parallels/parallels_network.c
> + parallels/parallels_network.c \
> + parallels/parallels_sdk.h \
> + parallels/parallels_sdk.c
>
> BHYVE_DRIVER_SOURCES =
\
>
> bhyve/bhyve_capabilities.c
\
>
> diff --git a/src/parallels/parallels_driver.c
> b/src/parallels/parallels_driver.c index bb9538f..7dc9963 100644
> --- a/src/parallels/parallels_driver.c
> +++ b/src/parallels/parallels_driver.c
> @@ -55,6 +55,7 @@
>
> #include "parallels_driver.h"
> #include "parallels_utils.h"
>
> +#include "parallels_sdk.h"
>
> #define VIR_FROM_THIS VIR_FROM_PARALLELS
>
> @@ -73,6 +74,9 @@ VIR_LOG_INIT("parallels.parallels_driver");
>
> #define IS_CT(def) (STREQ_NULLABLE(def->os.type, "exe"))
>
> +unsigned int numConns = 0;
> +virMutex numConnsLock;
1: ^^^
> +
>
> static int parallelsConnectClose(virConnectPtr conn);
>
> static const char * parallelsGetDiskBusName(int bus) {
>
> @@ -929,9 +933,25 @@ parallelsOpenDefault(virConnectPtr conn)
>
> if (virMutexInit(&privconn->lock) < 0) {
>
> virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
>
> _("cannot initialize mutex"));
>
> - goto error;
> + goto err_free;
>
> }
>
> + virMutexLock(&numConnsLock);
> + numConns++;
> +
> + if (numConns == 1) {
> + if (prlsdkInit()) {
> + VIR_DEBUG("%s", _("Can't initialize Parallels
SDK"));
> + virMutexUnlock(&numConnsLock);
> + goto err_free;
> + }
> + }
> +
> + virMutexUnlock(&numConnsLock);
> +
> + if (prlsdkConnect(privconn) < 0)
> + goto err_free;
> +
>
> if (!(privconn->caps = parallelsBuildCapabilities()))
>
> goto error;
>
> @@ -953,6 +973,9 @@ parallelsOpenDefault(virConnectPtr conn)
>
> virObjectUnref(privconn->domains);
> virObjectUnref(privconn->caps);
> virStoragePoolObjListFree(&privconn->pools);
>
> + prlsdkDisconnect(privconn);
> + prlsdkDeinit();
>
11:44 a.m.
New subject: [libvirt] [PATCH 2/2] parallels: login to parallels SDK
On Friday 22 August 2014 16:38:11 Michal Privoznik wrote:
On 21.08.2014 22:36, Dmitry Guryanov wrote:
> Add files parallels_sdk.c and parallels_sdk.h for code
> which works with SDK, so libvirt's code will not mix with
> dealing with parallels SDK.
>
> To use Parallels SDK you must first call PrlApi_InitEx function,
> and then you will be able to connect to a server with
> PrlSrv_LoginLocalEx function. When you've done you must call
> PrlApi_Deinit. So let's call PrlApi_InitEx on first .connectOpen,
> count number of connections and deinitialize, when this counter
> becomes zero.
>
> Signed-off-by: Dmitry Guryanov <dguryanov(a)parallels.com>
> ---
>
> po/POTFILES.in | 1 +
> src/Makefile.am | 4 +++-
> src/parallels/parallels_driver.c | 46
> +++++++++++++++++++++++++++++++++++++++-
> src/parallels/parallels_utils.h | 3 +++
> 4 files changed, 52 insertions(+), 2 deletions(-)
This fails 'make syntax-check' for me.
It seems I've sent completely wrong patch :( I should add 2 more files in this patch.
> diff --git a/po/POTFILES.in b/po/POTFILES.in
> index f17b35f..4c1302d 100644
> --- a/po/POTFILES.in
> +++ b/po/POTFILES.in
> @@ -96,6 +96,7 @@ src/openvz/openvz_driver.c
>
> src/openvz/openvz_util.c
> src/parallels/parallels_driver.c
> src/parallels/parallels_network.c
>
> +src/parallels/parallels_sdk.c
>
> src/parallels/parallels_utils.c
> src/parallels/parallels_utils.h
> src/parallels/parallels_storage.c
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index dad7c7f..d4c6465 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -787,7 +787,9 @@ PARALLELS_DRIVER_SOURCES =
\
>
> parallels/parallels_utils.c \
> parallels/parallels_utils.h \
> parallels/parallels_storage.c \
>
> - parallels/parallels_network.c
> + parallels/parallels_network.c \
> + parallels/parallels_sdk.h \
> + parallels/parallels_sdk.c
>
> BHYVE_DRIVER_SOURCES =
\
>
> bhyve/bhyve_capabilities.c
\
>
> diff --git a/src/parallels/parallels_driver.c
> b/src/parallels/parallels_driver.c index bb9538f..7dc9963 100644
> --- a/src/parallels/parallels_driver.c
> +++ b/src/parallels/parallels_driver.c
> @@ -55,6 +55,7 @@
>
> #include "parallels_driver.h"
> #include "parallels_utils.h"
>
> +#include "parallels_sdk.h"
>
> #define VIR_FROM_THIS VIR_FROM_PARALLELS
>
> @@ -73,6 +74,9 @@ VIR_LOG_INIT("parallels.parallels_driver");
>
> #define IS_CT(def) (STREQ_NULLABLE(def->os.type, "exe"))
>
> +unsigned int numConns = 0;
> +virMutex numConnsLock;
1: ^^^
> +
>
> static int parallelsConnectClose(virConnectPtr conn);
>
> static const char * parallelsGetDiskBusName(int bus) {
>
> @@ -929,9 +933,25 @@ parallelsOpenDefault(virConnectPtr conn)
>
> if (virMutexInit(&privconn->lock) < 0) {
>
> virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
>
> _("cannot initialize mutex"));
>
> - goto error;
> + goto err_free;
>
> }
>
> + virMutexLock(&numConnsLock);
> + numConns++;
> +
> + if (numConns == 1) {
> + if (prlsdkInit()) {
> + VIR_DEBUG("%s", _("Can't initialize Parallels
SDK"));
> + virMutexUnlock(&numConnsLock);
> + goto err_free;
> + }
> + }
> +
> + virMutexUnlock(&numConnsLock);
> +
> + if (prlsdkConnect(privconn) < 0)
> + goto err_free;
> +
>
> if (!(privconn->caps = parallelsBuildCapabilities()))
>
> goto error;
>
> @@ -953,6 +973,9 @@ parallelsOpenDefault(virConnectPtr conn)
>
> virObjectUnref(privconn->domains);
> virObjectUnref(privconn->caps);
> virStoragePoolObjListFree(&privconn->pools);
>
3748
days inactive
3749
days old
7 comments
2 participants
participants (2)
-
Dmitry Guryanov -
Michal Privoznik