xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Ian Jackson <ian.jackson@eu.citrix.com>, xen-devel@lists.xenproject.org
Subject: Re: [OSSTEST PATCH 22/26] ts-debian-hvm-install: Use /dev/sda for i386, not /dev/xvda
Date: Mon, 21 Sep 2015 10:40:21 +0100	[thread overview]
Message-ID: <1442828421.10338.30.camel@citrix.com> (raw)
In-Reply-To: <1442598616-2884-9-git-send-email-ian.jackson@eu.citrix.com>

On Fri, 2015-09-18 at 18:50 +0100, Ian Jackson wrote:
> Empirically, the device shows up as /dev/sda.

This is because the plain i386 kernel used by the installer does not
support Xen (which requires PAE) and this affects even the options needed
for PVHVM. amd64 on the other hand doesn't have this restriction and PVHVM
works with the installer kernel.

Because of this the i386 install ends up using the emulated sda device
instead of switching to the pv devices as amd64 does.

I think it's probably worth explaining some of this in the commit log
rather than "Empirically".

IIRC someone was working on enabling PVHVM even for non-PAE i386 kernels,
not sure what the status of that is, but clearly it isn't going to change
things for Wheezy.

If you wanted instead to enable PV support for these installs then the
multi-arch (i386+amd64) isoimages contain the Xen enabled kernels for i386
(as /install.386/xen/*, n.b. /install.amd/xen/* are suitable symlinks, so
you can aoid special cases.

e.g. this one:

http://cdimage.debian.org/cdimage/archive/7.9.0/multi-arch/iso-cd/

(sorry for forgetting about this when we spoke IRL last week).

Anyway, that's clearly an "if you can be bothered thing" so with an
expanded commit message:
    Acked-by: Ian Campbell <    ian.campbell@citrix.com    >

  reply	other threads:[~2015-09-21  9:40 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-18 17:50 [OSSTEST PATCH v3 00/15-26] xen.git#staging smoke tests Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 15/26] cr*: Support xen-unstable-smoke Ian Jackson
2015-09-21  9:17   ` Ian Campbell
2015-09-21 10:21     ` Ian Jackson
2015-09-21 10:32       ` Ian Campbell
2015-09-21 10:53         ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 16/26] ap-*: Be able to fetch and push xen.git#smoke Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 17/26] crontab: Enable xen-unstable-smoke branch every 3h at <1mod3>:51 UTC Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 18/26] ts-debian-hvm-install, etc.: Do not hardcode in-iso path Ian Jackson
2015-09-21  9:19   ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 19/26] ts-debian-hvm-install: Cope with images containing only isolinux Ian Jackson
2015-09-21  9:28   ` Ian Campbell
2015-09-21 10:35     ` Ian Jackson
2015-09-21 14:30       ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 20/26] ts-debian-hvm-install: Set $gsuite after $gho Ian Jackson
2015-09-21  9:29   ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 21/26] ts-debian-hvm-install: Do not create EFI partition if EFI not in use Ian Jackson
2015-09-21  9:32   ` Ian Campbell
2015-09-21 10:36     ` Ian Jackson
2015-09-21 10:41     ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 22/26] ts-debian-hvm-install: Use /dev/sda for i386, not /dev/xvda Ian Jackson
2015-09-21  9:40   ` Ian Campbell [this message]
2015-09-21 10:38     ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 23/26] Timeouts: Introduce target_adjust_timeout Ian Jackson
2015-09-21  9:41   ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 24/26] Timeouts: Honour guest-related timeout-adjustment runvars Ian Jackson
2015-09-21  9:43   ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 25/26] Debian i386 HVM tests: Increase installation timeout Ian Jackson
2015-09-21  9:43   ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 26/26] cri-common: Add a missing semicolon Ian Jackson
2015-09-21  9:44   ` Ian Campbell

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=1442828421.10338.30.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).