tools/xenstored: Stash the correct request in lu_status->in
authorJulien Grall <jgrall@amazon.com>
Thu, 1 Jul 2021 14:03:07 +0000 (15:03 +0100)
committerJulien Grall <jgrall@amazon.com>
Tue, 6 Jul 2021 08:50:57 +0000 (09:50 +0100)
commit09e2cd69956fe529bfc3d53dae477b44eff217d1
tree38127eee213cc5e51e3a1ec7c6a40fff48d6010f
parent33e48312cf28f6fb78c03b14514fa7718de97ef5
tools/xenstored: Stash the correct request in lu_status->in

When Live-Updating with some load, Xenstored may hit the assert
req->in == lu_status->in in do_lu_start().

This is happening because the request is stashed when Live-Update
begins. This happens in a different request (see call lu_begin()
when select the new binary) from the one performing Live-Update.

To avoid the problem, stash the request in lu_start().

Fixes: 65f19ed62aa1 ("tools/xenstore: Don't assume conn->in points to the LU request")
Reported-by: Michael Kurth <mku@amazon.com>
Signed-off-by: Julien Grall <jgrall@amazon.com>
Reviewed-by: luca.fancellu@arm.com
Reviewed-by: Juergen Gross <jgross@suse.com>
tools/xenstore/xenstored_control.c