* Re: [PATCH] xenbus: don't look up transaction IDs for ordinary writes
[not found] <57B1F5AE0200007800106092@suse.com>
@ 2016-08-16 10:07 ` Juergen Gross
2016-08-18 12:41 ` Ed Swierk
0 siblings, 1 reply; 3+ messages in thread
From: Juergen Gross @ 2016-08-16 10:07 UTC (permalink / raw)
To: Jan Beulich, David Vrabel, Boris Ostrovsky; +Cc: xen-devel, lkml
On 15/08/16 17:02, Jan Beulich wrote:
> 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@uni-koblenz.de>
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> Tested-by: Richard Schütz <rschuetz@uni-koblenz.de>
Reviewed-by: Juergen Gross <jgross@suse.com>
Thanks,
Juergen
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] xenbus: don't look up transaction IDs for ordinary writes
2016-08-16 10:07 ` [PATCH] xenbus: don't look up transaction IDs for ordinary writes Juergen Gross
@ 2016-08-18 12:41 ` Ed Swierk
0 siblings, 0 replies; 3+ messages in thread
From: Ed Swierk @ 2016-08-18 12:41 UTC (permalink / raw)
To: Juergen Gross; +Cc: Jan Beulich, David Vrabel, Boris Ostrovsky, xen-devel, lkml
On Tue, Aug 16, 2016 at 3:07 AM, Juergen Gross <jgross@suse.com> wrote:
> On 15/08/16 17:02, Jan Beulich wrote:
>> 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@uni-koblenz.de>
>> Cc: <stable@vger.kernel.org>
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>> Tested-by: Richard Schütz <rschuetz@uni-koblenz.de>
>
> Reviewed-by: Juergen Gross <jgross@suse.com>
Confirmed that this patch fixes xenstore-read, xenstore-ls et al, on
stable kernel 4.4.17.
It should also go to 4.7, 4.6, 4.1, 3.18.
--Ed
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] xenbus: don't look up transaction IDs for ordinary writes
@ 2016-08-15 15:02 Jan Beulich
0 siblings, 0 replies; 3+ messages in thread
From: Jan Beulich @ 2016-08-15 15:02 UTC (permalink / raw)
To: david.vrabel, boris.ostrovsky, Juergen Gross; +Cc: xen-devel, linux-kernel
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@uni-koblenz.de>
Cc: <stable@vger.kernel.org>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Tested-by: Richard Schütz <rschuetz@uni-koblenz.de>
---
drivers/xen/xenbus/xenbus_dev_frontend.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- 4.8-rc2/drivers/xen/xenbus/xenbus_dev_frontend.c
+++ 4.8-rc2-xen-xenbus-dev-write-oridinary/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;
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-18 12:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <57B1F5AE0200007800106092@suse.com>
2016-08-16 10:07 ` [PATCH] xenbus: don't look up transaction IDs for ordinary writes Juergen Gross
2016-08-18 12:41 ` Ed Swierk
2016-08-15 15:02 Jan Beulich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®