On Mon, Jan 19, 2009 at 06:56:12PM +0100, Jim Meyering wrote:
"Daniel P. Berrange" <berrange(a)redhat.com> wrote:
> This patch makes the various Xen drivers threadsafe by adding a
> mutex lock on the xenUnifiedPrivatePtr object. The XenD driver
> does not really need much locking, since it usually just calls
> out to XenD. Likewise the Xen driver just makes hypercalls. Locks
> are needed for libxenstore access, and the xm/inotify drivers
> since they have shared state
>
> src/proxy_internal.c | 173 ++++++++++++++++-----------------
> src/xen_inotify.c | 25 +++-
> src/xen_internal.c | 29 +++--
> src/xen_unified.c | 176 +++++++++++++++++++++-------------
> src/xen_unified.h | 36 +++++--
> src/xend_internal.c | 37 ++++++-
> src/xm_internal.c | 256 +++++++++++++++++++++++++++++++++-----------------
> src/xs_internal.c | 130 +++++++++++++++++++------
> src/xs_internal.h | 7 -
> tests/sexpr2xmltest.c | 19 +++
> 10 files changed, 575 insertions(+), 313 deletions(-)
Looks A-ok to me (yes, it was tedious).
ACK.
Committed now.
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 :|