From: Don Slutz <dslutz@verizon.com>
To: Ian Campbell <Ian.Campbell@citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Anthony.Perard@citrix.com, tiejun.chen@intel.com,
xen-devel@lists.xensource.com,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v2] libxl: change default QEMU machine to pc-i440fx-1.6
Date: Tue, 15 Jul 2014 08:25:36 -0400 [thread overview]
Message-ID: <53C51DC0.3000401@terremark.com> (raw)
In-Reply-To: <1405091336.32197.1.camel@kazak.uk.xensource.com>
On 07/11/14 11:08, Ian Campbell wrote:
> On Fri, 2014-07-11 at 15:54 +0100, Stefano Stabellini wrote:
>> On Fri, 11 Jul 2014, Ian Campbell wrote:
>>> On Fri, 2014-07-11 at 12:14 +0100, Stefano Stabellini wrote:
>>>> On Thu, 10 Jul 2014, Ian Jackson wrote:
>>>>> Stefano Stabellini writes ("Re: [PATCH v2] libxl: change default QEMU machine to pc-i440fx-1.6"):
>>>
>> If we use -M pc or -M pc-i440fx-1.6, the xen-platform-pci device is
>> created slightly later in the QEMU boot sequence. The Intel HDA device
>> is able to "steal" its slot on the PCI bus.
>>
>> On the other hand it is worth pointing out that currently libxl switches
>> from -M xenfv to -M pc silently depending on whether the
>> xen_platform_pci option is enabled or disabled. If it is disabled we get
>> pc, that is a moving target and not guaranteed to be compatible with
>> anything.
> Changing that option is a guest reboot scenario, which is completely
> different to a migration. I think it is (just about) OK for the PCI id
> to change on a reboot, but it certainly isn't OK on migrate.
>
> If you can't get the platform device created earlier then perhaps you
> can find a way to influence the slot used by the hda device?
>
> Ian.
I see 2 options:
1) change only the -M pc to -M pc-i440fx-1.6 (I.E. continue to use xenfv)
2) Add ",addr=0x1.0x0" to the xen platform device. (Which will cause
QEMU to abort in this case.) Also fix soundhw handling to stop using
"-soundhw", and switch to "--device intel-hda,addr=0x12.0x0 -device
hda-duplex" etc.
As a side note on #1, I do have a hack to allow adjusting the placement
of the xen platform device (adding a new xl.cfg option for it), but is not
ready to be a patch. It would allow still using "-soundhw" "hda" but would
require a config change.
-Don Slutz
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
prev parent reply other threads:[~2014-07-15 12:25 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-12 14:33 [Qemu-devel] [PATCH v2] libxl: change default QEMU machine to pc-i440fx-1.6 Stefano Stabellini
2014-06-12 17:07 ` [Qemu-devel] [Xen-devel] " Fabio Fantoni
2014-07-10 17:03 ` [Qemu-devel] " Stefano Stabellini
2014-07-10 17:22 ` Ian Jackson
2014-07-11 11:14 ` Stefano Stabellini
2014-07-11 14:19 ` Ian Campbell
2014-07-11 14:54 ` [Qemu-devel] [Xen-devel] " Stefano Stabellini
2014-07-11 15:08 ` Ian Campbell
2014-07-15 12:25 ` Don Slutz [this message]
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=53C51DC0.3000401@terremark.com \
--to=dslutz@verizon.com \
--cc=Anthony.Perard@citrix.com \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=tiejun.chen@intel.com \
--cc=xen-devel@lists.xensource.com \
/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).