]> git.baikalelectronics.ru Git - kernel.git/commit
xen/xenbus: Avoid synchronous wait on XenBus stalling shutdown/restart.
authorKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Fri, 4 Apr 2014 18:53:40 +0000 (14:53 -0400)
committerDavid Vrabel <david.vrabel@citrix.com>
Tue, 15 Apr 2014 16:41:28 +0000 (17:41 +0100)
commitbca3614002d589c96a33b6f15f355f6de355a7ae
tree622c4232c082e9641024b6e2117c3bc5182362a9
parente1024a817c071dd96db2bd0185e257f51ab1a540
xen/xenbus: Avoid synchronous wait on XenBus stalling shutdown/restart.

The 'read_reply' works with 'process_msg' to read of a reply in XenBus.
'process_msg' is running from within the 'xenbus' thread. Whenever
a message shows up in XenBus it is put on a xs_state.reply_list list
and 'read_reply' picks it up.

The problem is if the backend domain or the xenstored process is killed.
In which case 'xenbus' is still awaiting - and 'read_reply' if called -
stuck forever waiting for the reply_list to have some contents.

This is normally not a problem - as the backend domain can come back
or the xenstored process can be restarted. However if the domain
is in process of being powered off/restarted/halted - there is no
point of waiting on it coming back - as we are effectively being
terminated and should not impede the progress.

This patch solves this problem by checking whether the guest is the
right domain. If it is an initial domain and hurtling towards death -
there is no point of continuing the wait. All other type of guests
continue with their behavior (as Xenstore is expected to still be
running in another domain).

Fixes-Bug: http://bugs.xenproject.org/xen/bug/8
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Reviewed-by: David Vrabel <david.vrabel@citrix.com>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
drivers/xen/xenbus/xenbus_xs.c