Re: [Xen-devel] [PATCH 0/2] Handle xen_platform_pci=0 case

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2013-11-22 at 17:18 +0000, Anthony PERARD wrote:
> On Fri, Nov 22, 2013 at 03:56:13PM +0000, Ian Campbell wrote:
> > On Fri, 2013-11-22 at 15:13 +0000, Anthony PERARD wrote:
> > > There are two patches, the first one adds an option to specifies the QEMU
> > > machine that a user wants and we handle the xen_platform_pci=0 case using 
> > > the
> > > new option.
> > 
> > I'm thinking we should make this new option an enum of actually
> > supported machine types, rather than just a free reign. Makes more sense
> > from a "what do we support" PoV.
> 
> And start with the following list ?

Uh, that's quite long.
> pc
> pc-i440fx-1.7
> pc-i440fx-1.6
> pc-i440fx-1.5
> pc-i440fx-1.4
> pc-1.3
> pc-1.2
> pc-1.1
> pc-1.0
> pc-0.15
> pc-0.14
> pc-0.13
> pc-0.12
> pc-0.11
> pc-0.10
> xenfv
> 
> I wonder how many of this will work. I suspect most of them.

Work != Warantied by Xen.org (whatever that means)

>  The only
> down side is that it will be necessary to patch libxl if one want to
> try, let say q35.
> 
> If we only keep xenfv and pc in the list, it might be better to just not
> expose the option to the config file, for now.
> 
> But yes, having a list for the supported feature point of view will be
> better.

I can just about see having a non "_override" enum *and* and _override
string. But ick.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel