On Mon, Jan 19, 2009 at 06:56:12PM +0100, Jim Meyering wrote:
"Daniel P. Berrange" <berrange@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 :|