xenbus: don't look up transaction IDs for ordinary writes

PATCH Juergen Gross 2 participants 4 messages Aug 16, 2016 – Aug 18, 2016 1 message not in the archive

lore

Share & tools

lore

Juergen Gross

author Reviewed-by

2016-08-16 10:07 UTC #1 lore raw

In reply to a message not in the archive

Quoted 8 lines · Jan Beulich

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 <[email protected]>
Cc <[email protected]>
Signed-off-by Jan Beulich <[email protected]>
Tested-by Richard Schütz <[email protected]>

Reviewed-by Juergen Gross <[email protected]>

Thanks,

Juergen

Juergen Gross

author Reviewed-by

2016-08-16 10:07 UTC #4 lore raw

In reply to a message not in the archive

Quoted 8 lines · Jan Beulich

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 <[email protected]>
Cc <[email protected]>
Signed-off-by Jan Beulich <[email protected]>
Tested-by Richard Schütz <[email protected]>

Reviewed-by Juergen Gross <[email protected]>

Thanks,

Juergen

_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xen.org/xen-devel

Ed Swierk

2016-08-18 12:42 UTC #2 lore raw

In reply to Juergen Gross

Quoted 11 lines · Juergen Gross

Quoted 8 lines · Jan Beulich
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 <[email protected]>
Cc <[email protected]>
Signed-off-by Jan Beulich <[email protected]>
Tested-by Richard Schütz <[email protected]>
Reviewed-by Juergen Gross <[email protected]>

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