[libvirt] PATCH: 13/25: Remove global state from XM driver
Richard W.M. Jones
rjones at redhat.com
Thu Jan 15 17:23:33 UTC 2009
On Tue, Jan 13, 2009 at 05:44:00PM +0000, Daniel P. Berrange wrote:
> The 'xm' driver currently keeps all its state in a global static
> variables. Not cool, since we access this from all sorts of places
> and its hard to guarentee thread safety. So we move the state into
> the virConnect object. This will increase memory usage if a single
> process has multiple Xen connections open though.
>
> Undecided whether this is a big problem or not. If so, I'll can
> try and redo the next thread locking patch to use a lock over the
> existing global state.
>
> xen_inotify.c | 4
> xen_unified.c | 1
> xen_unified.h | 14 ++-
> xm_internal.c | 262 +++++++++++++++++++++++++++++-----------------------------
> xm_internal.h | 3
> 5 files changed, 149 insertions(+), 135 deletions(-)
The patch appears straightforward enough, and for the XM driver
(ie. ancient history Xen) I'm sure we don't care much about a tiny bit
of extra memory.
Rich.
--
Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones
virt-p2v converts physical machines to virtual machines. Boot with a
live CD or over the network (PXE) and turn machines into Xen guests.
http://et.redhat.com/~rjones/virt-p2v
More information about the libvir-list
mailing list