All of lore.kernel.org
 help / color / mirror / Atom feed
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

WARNING: multiple messages have this Message-ID (diff)
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: [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

  reply	other threads:[~2014-07-15 12:25 UTC|newest]

Thread overview: 18+ 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 14:33 ` Stefano Stabellini
2014-06-12 17:07 ` [Qemu-devel] [Xen-devel] " Fabio Fantoni
2014-06-12 17:07   ` Fabio Fantoni
2014-07-10 17:03 ` [Qemu-devel] " Stefano Stabellini
2014-07-10 17:03   ` Stefano Stabellini
2014-07-10 17:22   ` [Qemu-devel] " Ian Jackson
2014-07-10 17:22     ` Ian Jackson
2014-07-11 11:14     ` [Qemu-devel] " Stefano Stabellini
2014-07-11 11:14       ` Stefano Stabellini
2014-07-11 14:19       ` [Qemu-devel] " Ian Campbell
2014-07-11 14:19         ` Ian Campbell
2014-07-11 14:54         ` [Qemu-devel] [Xen-devel] " Stefano Stabellini
2014-07-11 14:54           ` Stefano Stabellini
2014-07-11 15:08           ` [Qemu-devel] " Ian Campbell
2014-07-11 15:08             ` Ian Campbell
2014-07-15 12:25             ` Don Slutz [this message]
2014-07-15 12:25               ` Don Slutz

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