Re: [Xen-devel] Re: [PATCH 0/2] [RFC] xl: add cpuid config file option

[Andre Przywara <andre.przywara@xxxxxxx>]

Ian Jackson wrote:
> Stefano Stabellini writes ("[Xen-devel] Re: [PATCH 0/2] [RFC] xl: add cpuid config 
> file option"):
> > On Fri, 27 Aug 2010, Andre Przywara wrote:
> > > This first version works for me, I'd like to hear your comments.
> > I like your approach. Ian, what do you think?
> I think this is a nice approach.  But there didn't seem to be a patch
> attached to your mail ? :-)
>
> > We would still need another parser for backward compatibility
> > though.
> Is it possible to distinguish the old xend syntax and so have libxl
> handle either ?  I'm not familiar with the xend cpuid syntax.
xm uses a Python list syntax, so this looks different. It is possible to 
tell them apart: Currently you can check the return value of 
xlu_cfg_get_list() to try the new approach otherwise, but this prints an 
ugly warning to the console. So I created a (silent) get_type function 
to check it before calling the respective functions.
The prototype works already, I will test and polish and then send out a 
second version later.
> > Also I think that parse_cpuid belongs to libxl instead of xl.
> Yes.
You are right, I have already moved it (in v2).

Regards,
Andre.

--
Andre Przywara
AMD-Operating System Research Center (OSRC), Dresden, Germany
Tel: +49 351 448-3567-12


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