[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH v3 0/5] Fix QEMU startup protocol

This patch series tries to reason about various hardcoded "/local/domain/0" in
libxl and 1) replace them with something sensible, 2) fix QEMU startup

Please apply a patch to QEMU traditional first, squash the update to Config.mk
into "libxl: use new QEMU xenstore protocol" to keep the tree bisectable.

The new protocol is introduced in "libxl: use new QEMU xenstore protocol".
Basically it replaces hardcoded "/local/domain/0" with

For both QEMU upstream and traditional running in Dom0, the new protocol is
compatible with the old one, because in those cases $dm_domid is 0.

For QEMU traditional running in stubdom, this protocol is incompatible with
old one. However this is acceptable because we always ships QEMU traditional
with Xen. For Xen 4.5 we should backport Paul's workaround.

For QEMU upstream running in stubdom, there is no compatibility issue because
there is no QEMU upstream stubdom yet. This patch series is also a good start
to avoid making the same mistake in QEMU upstream stubdom.


Wei Liu (5):
  libxl: introduce libxl__device_model_xs_path
  libxl: remove device model path in libxl__device_model_destroy
  libxl: use new QEMU xenstore protocol
  Revert "x86/hvm: wait for at least one ioreq server to be enabled"

 tools/libxl/libxl.c              |  6 ++---
 tools/libxl/libxl_device.c       |  4 ++-
 tools/libxl/libxl_dm.c           | 55 ++++++++++++++++++++++++++++++++++++---
 tools/libxl/libxl_dom.c          | 56 +++++++++++++++++++++++++---------------
 tools/libxl/libxl_internal.c     | 40 +++++++++++++++++++++++-----
 tools/libxl/libxl_internal.h     |  6 +++++
 tools/libxl/libxl_pci.c          | 22 +++++++++-------
 xen/arch/x86/hvm/hvm.c           | 21 ---------------
 xen/include/asm-x86/hvm/domain.h |  1 -
 9 files changed, 144 insertions(+), 67 deletions(-)


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.