]> git.baikalelectronics.ru Git - kernel.git/commit
xen/pv-on-hvm kexec: add xs_reset_watches to shutdown watches from old kernel
authorOlaf Hering <olaf@aepfle.de>
Thu, 22 Sep 2011 14:14:49 +0000 (16:14 +0200)
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Thu, 22 Sep 2011 20:32:24 +0000 (16:32 -0400)
commit8fa68d08e33759755e7f44c64a6fcff309f33d78
tree621aaaa29cd7b549854365d0d942af81acc9aaeb
parent887977a4a7baa55252fc14c89c4e18885f4b0617
xen/pv-on-hvm kexec: add xs_reset_watches to shutdown watches from old kernel

Add new xs_reset_watches function to shutdown watches from old kernel after
kexec boot.  The old kernel does not unregister all watches in the
shutdown path.  They are still active, the double registration can not
be detected by the new kernel.  When the watches fire, unexpected events
will arrive and the xenwatch thread will crash (jumps to NULL).  An
orderly reboot of a hvm guest will destroy the entire guest with all its
resources (including the watches) before it is rebuilt from scratch, so
the missing unregister is not an issue in that case.

With this change the xenstored is instructed to wipe all active watches
for the guest.  However, a patch for xenstored is required so that it
accepts the XS_RESET_WATCHES request from a client (see changeset
23839:42a45baf037d in xen-unstable.hg). Without the patch for xenstored
the registration of watches will fail and some features of a PVonHVM
guest are not available. The guest is still able to boot, but repeated
kexec boots will fail.

[v5: use xs_single instead of passing a dummy string to xs_talkv]
[v4: ignore -EEXIST in xs_reset_watches]
[v3: use XS_RESET_WATCHES instead of XS_INTRODUCE]
[v2: move all code which deals with XS_INTRODUCE into xs_introduce()
    (based on feedback from Ian Campbell); remove casts from kvec assignment]
Signed-off-by: Olaf Hering <olaf@aepfle.de>
[v1: Redid the git description a bit]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
drivers/xen/xenbus/xenbus_xs.c
include/xen/interface/io/xs_wire.h