Re: [libvirt] PATCH: 0/28: Thread safety for libvirtd daemon and drivers

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

 



On Wed, Dec 03, 2008 at 04:15:05PM -0800, Chris Wright wrote:
> * Daniel P. Berrange (berrange@xxxxxxxxxx) wrote:
> > (*
> >  * This is the list of all libvirt methods which return
> >  * pointers to locked objects
> >  *)
> > let lockedObjMethods = [
> >    "virDomainFindByID";
> >    "virDomainFindByUUID";
> >    "virDomainFindByName";
> >    "virDomainAssignDef";
> ...
> 
> All quite cool.  Anyway to annotate the source so these tables don't get
> out of date?

We'd have to invent some annotation for parameters and return types to
indicate whether they expect / return a locked object.

Although actually thinking about it, in domain_conf.c, every single func
which takes a virDomainObjPtr as a param expects it to be locked, and all
which return such an object give it to you pre-locked. So I could just
have it parse domain_conf.h and extract the list of APIs in that header.

Daniel
-- 
|: 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 :|

--
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]