From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Ian Campbell <Ian.Campbell@eu.citrix.com>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
"konrad.wilk@oracle.com" <konrad.wilk@oracle.com>
Subject: Re: [PATCH] xen: remove XEN_PLATFORM_PCI config option
Date: Tue, 27 Sep 2011 09:50:51 -0700 [thread overview]
Message-ID: <4E81FEEB.7040100@goop.org> (raw)
In-Reply-To: <alpine.DEB.2.00.1109271544470.3519@kaball-desktop>
On 09/27/2011 07:46 AM, Stefano Stabellini wrote:
> On Fri, 23 Sep 2011, Jeremy Fitzhardinge wrote:
>> On 09/23/2011 04:19 AM, stefano.stabellini@eu.citrix.com wrote:
>>> From: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>>>
>>> Xen PVHVM needs xen-platform-pci, on the other hand xen-platform-pci is
>>> useless in any other cases.
>>> Therefore remove the XEN_PLATFORM_PCI config option and compile
>>> xen-platform-pci built-in if XEN_PVHVM is selected.
>> What happens if you disable CONFIG_PCI?
>>
>> I think XEN_PLATFORM_PCI still needs to exist, but just not user-visible.
> What if we add CONFIG_PCI as a dependency of XEN_PVHVM?
>
> It is not like it is going to be useful to run a PV on HVM guest without
> PV drivers.
In principle you could have a domain with emulated ISA IDE and net but
with PV time, etc. But yeah, not very useful in practice. I think
making PVHVM depend on PCI is fine.
J
prev parent reply other threads:[~2011-09-27 16:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-23 11:19 [PATCH] xen: remove XEN_PLATFORM_PCI config option stefano.stabellini
2011-09-23 14:42 ` Jeremy Fitzhardinge
2011-09-27 14:46 ` Stefano Stabellini
2011-09-27 16:50 ` Jeremy Fitzhardinge [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=4E81FEEB.7040100@goop.org \
--to=jeremy@goop.org \
--cc=Ian.Campbell@eu.citrix.com \
--cc=konrad.wilk@oracle.com \
--cc=stefano.stabellini@eu.citrix.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.