[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <lsq.1479082461.360027432@decadent.org.uk>
Date: Mon, 14 Nov 2016 00:14:21 +0000
From: Ben Hutchings <ben@...adent.org.uk>
To: linux-kernel@...r.kernel.org, stable@...r.kernel.org
CC: akpm@...ux-foundation.org,
"Ed Swierk" <eswierk@...portsystems.com>,
"David Vrabel" <david.vrabel@...rix.com>,
"Jan Beulich" <jbeulich@...e.com>,
"Richard Schütz" <rschuetz@...-koblenz.de>,
"Jan Beulich" <JBeulich@...e.com>
Subject: [PATCH 3.16 343/346] xenbus: don't look up transaction IDs for
ordinary writes
3.16.39-rc1 review patch. If anyone has any objections, please let me know.
------------------
From: Jan Beulich <JBeulich@...e.com>
commit 9a035a40f7f3f6708b79224b86c5777a3334f7ea upstream.
This should really only be done for XS_TRANSACTION_END messages, or
else at least some of the xenstore-* tools don't work anymore.
Fixes: 0beef634b8 ("xenbus: don't BUG() on user mode induced condition")
Reported-by: Richard Schütz <rschuetz@...-koblenz.de>
Signed-off-by: Jan Beulich <jbeulich@...e.com>
Tested-by: Richard Schütz <rschuetz@...-koblenz.de>
Signed-off-by: David Vrabel <david.vrabel@...rix.com>
Cc: Ed Swierk <eswierk@...portsystems.com>
Signed-off-by: Ben Hutchings <ben@...adent.org.uk>
---
drivers/xen/xenbus/xenbus_dev_frontend.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/xen/xenbus/xenbus_dev_frontend.c
+++ b/drivers/xen/xenbus/xenbus_dev_frontend.c
@@ -316,7 +316,7 @@ static int xenbus_write_transaction(unsi
rc = -ENOMEM;
goto out;
}
- } else {
+ } else if (msg_type == XS_TRANSACTION_END) {
list_for_each_entry(trans, &u->transactions, list)
if (trans->handle.id == u->u.msg.tx_id)
break;
Powered by blists - more mailing lists