All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: xen-devel@lists.xen.org
Cc: anthony.perard@citrix.com, Wei Liu <wei.liu2@citrix.com>,
	ian.jackson@eu.citrix.com, ian.campbell@citrix.com,
	stefano.stabellini@eu.citrix.com
Subject: [PATCH 0/5] Fix QEMU startup protocol
Date: Fri, 13 Mar 2015 10:34:30 +0000	[thread overview]
Message-ID: <1426242875-24446-1-git-send-email-wei.liu2@citrix.com> (raw)

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
protocol.

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

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.

Wei Liu (5):
  libxl: remove DM path in libxl__device_model_destroy
  libxl: use LIBXL_TOOLSTACK_DOMID
  libxl: use new QEMU xenstore protocol
  libxl: wait for stubdom to be ready
  Revert "x86/hvm: wait for at least one ioreq server to be enabled"

 tools/libxl/libxl.c              |  7 +++--
 tools/libxl/libxl_device.c       |  5 +++-
 tools/libxl/libxl_dm.c           | 65 +++++++++++++++++++++++++++++++++-------
 tools/libxl/libxl_dom.c          | 51 ++++++++++++++++++++-----------
 tools/libxl/libxl_internal.h     |  2 ++
 tools/libxl/libxl_pci.c          | 29 +++++++++++-------
 xen/arch/x86/hvm/hvm.c           | 21 -------------
 xen/include/asm-x86/hvm/domain.h |  1 -
 8 files changed, 117 insertions(+), 64 deletions(-)

-- 
1.9.1

             reply	other threads:[~2015-03-13 10:34 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-13 10:34 Wei Liu [this message]
2015-03-13 10:34 ` [PATCH 1/5] libxl: remove DM path in libxl__device_model_destroy Wei Liu
2015-03-18 13:21   ` Ian Campbell
2015-03-18 13:23     ` Ian Campbell
2015-03-13 10:34 ` [PATCH 2/5] libxl: use LIBXL_TOOLSTACK_DOMID Wei Liu
2015-03-18 13:21   ` Ian Campbell
2015-03-13 10:34 ` [PATCH 3/5] libxl: use new QEMU xenstore protocol Wei Liu
2015-03-18 13:30   ` Ian Campbell
2015-03-19 10:36     ` Wei Liu
2015-03-19 10:55       ` Ian Campbell
2015-03-19 11:04         ` Wei Liu
2015-03-13 10:34 ` [PATCH 4/5] libxl: wait for stubdom to be ready Wei Liu
2015-03-16 17:55   ` Wei Liu
2015-03-18 13:35   ` Ian Campbell
2015-03-19 10:46     ` Wei Liu
2015-03-13 10:34 ` [PATCH 5/5] Revert "x86/hvm: wait for at least one ioreq server to be enabled" Wei Liu
2015-03-13 11:32   ` Paul Durrant
2015-03-13 10:38 ` [PATCH] xenstore: use relative path for device-model node Wei Liu
2015-03-13 10:46   ` Ian Campbell
2015-03-13 11:01     ` Wei Liu
2015-03-13 11:04   ` Stefano Stabellini
2015-03-18 13:24   ` Ian Campbell
2015-03-19 10:54     ` Wei Liu
2015-03-19 11:04       ` Ian Campbell
2015-03-24 17:16   ` Ian Jackson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1426242875-24446-1-git-send-email-wei.liu2@citrix.com \
    --to=wei.liu2@citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.