Re: [PATCH v2 2/2] parallels: login to parallels SDK

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

 



On Thu, Sep 11, 2014 at 06:19:23PM +0400, Dmitry Guryanov wrote:
> On Thursday 11 September 2014 12:09:20 Daniel P. Berrange wrote:
> > On Sat, Sep 06, 2014 at 08:28:10PM +0400, 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.
> > 
> > As a general rule, even if we count the open/close calls
> > it isn't safe to run deinit functions in libvirt. eg consider
> > if libvirt is linked against another program or library that
> > also uses the paralllels SDK. Libvirt does not know if the
> > other code has stopped using the SDK. So either the reference
> > counting must be done in PrlApi_{InitEx,Deinit} functions
> > directly, or libvirt should simply not call PrlApi_Deinit
> > at all.  I'd probably just go with the latter, as I doubt there
> > is any real harm to skipping deinit.
> > 
> 
> I can move reference counting to parallels SDK, 

Ok cool. 

In the configure.ac pkg-config check be sure to add a version number
comparison, so that we guarantee a new enough library to include the
ref counting you add.


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]