[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V3 0/6] firmware changes as part ofMU/Xen merge.



On Fri, 29 Oct 2010, Pasi Kärkkäinen wrote:

> On Fri, Oct 29, 2010 at 02:18:29PM +0100, Keir Fraser wrote:
> > On 29/10/2010 14:16, "Keir Fraser" <keir@xxxxxxx> wrote:
> >
> > >> Should it be called something else than HVM_PARAM_ACPI_NEW_IOPORT ?
> > >> After a couple of years "NEW" might not make much sense anymore..
> > >>
> > >> IOPORT_V2 ? Or something..
> > >
> > > I've asked for a nice big explanatory comment to be added beside the
> > > parameter's definition. Also we should treat this field as a version 
> > > number,
> > > so baking a fixed version into its name is not a good idea. With the
> > > explanatory comment, I think its existing name is good enough. To do much
> > > better the name would have to become cumbersomely long.
> >
> > That said, HVM_PARAM_ACPI_IOPORTS_LOCATION might be better... I'm not
> > personally that fussed either way however.
> >
>
> Yeah, that sounds good..

This seam better, I will use this name with this value:
  - 0 : for the xen/old addresses.
  - 1 : for the qemu/new addresses.

-- 
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.