On Wed, Mar 05, 2008 at 03:15:19PM -0700, Ky Srinivasan wrote:
> I am attaching updated versions of the patches that I posted a couple 
> of weeks ago. These have been merged up to the current unstable tip: 
> changeset 17186:854b0704962b
>
> These patches have been tested on the unstable tip.
I'm not expert enough to comment on the HV extension implementation itself,
but in terms of the userspace side, the user visible configuration file 
option 'extid=1' is pretty unpleasant. It is akin to a 'magic constant'
in C code - no understandable meaning at all.
I'd like to see it accept a named extension type - if its possible to
have multiple extensions per guest, then using a list instead of a scalar
would be better. So how about something closer to
 extensions = [ "win2k8" ]
> +gopts.var('extid', val='EXTID',
> + fn=set_int, default=0,
> + use="Specify extention ID for a HVM domain.")
> +
And this help message could list the valid extension names
Regards,
Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=|
|=- Perl modules: http://search.cpan.org/~danberr/ -=|
|=- Projects: http://freshmeat.net/~danielpb/ -=|
|=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=| 
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel