From: Keir Fraser <keir.fraser@citrix.com>
Date: Wed, 12 May 2010 07:49:13 +0000 (+0100)
Subject: xs: avoid pthread_join deadlock in xs_daemon_close
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~12188
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=990e331218dba6b671854cec24f639ff45763d7d;p=xen.git

xs: avoid pthread_join deadlock in xs_daemon_close

Doing a pthread_cancel and join on the reader thread while holding all
the request/reply/watch mutexes can deadlock if the thread needs to
take any of those mutexes to exit.  Kill off the reader thread before
taking any mutexes (which should be redundant if we're
single-threaded at that point).

Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
---

diff --git a/tools/xenstore/xs.c b/tools/xenstore/xs.c
index 6a87ccf708..53e3d44345 100644
--- a/tools/xenstore/xs.c
+++ b/tools/xenstore/xs.c
@@ -260,10 +260,6 @@ void xs_daemon_destroy_postfork(struct xs_handle *h)
 
 void xs_daemon_close(struct xs_handle *h)
 {
-	mutex_lock(&h->request_mutex);
-	mutex_lock(&h->reply_mutex);
-	mutex_lock(&h->watch_mutex);
-
 #ifdef USE_PTHREAD
 	if (h->read_thr_exists) {
 		pthread_cancel(h->read_thr);
@@ -271,6 +267,10 @@ void xs_daemon_close(struct xs_handle *h)
 	}
 #endif
 
+	mutex_lock(&h->request_mutex);
+	mutex_lock(&h->reply_mutex);
+	mutex_lock(&h->watch_mutex);
+
         close_free_msgs(h);
 
 	mutex_unlock(&h->request_mutex);