[Xen-devel] PATCH: was Re**2: PATCH (and RFC): new dom0 hypercall: DOM0_DOMAIN_SETUP

[Tristan Gingold <Tristan.Gingold@xxxxxxxx>]

Le Lundi 19 Juin 2006 16:45, Keir Fraser a écrit :
> On 19 Jun 2006, at 15:43, Tristan Gingold wrote:
> >> I'd like the patch modified as follows:
> >>   * Change flag name to XEN_DOMAINSETUP_hvm_guest (explicitly in Xen
> >> namespace; more the style of other flags added recently; not VMX
> >> specific).
> >>   * Place the final two fields of the struct inside ifdef __ia64__.
> >
> > Why not defining an arch-dependant struct for adding more fields to
> > this
> > structure ?  (I prefer to do that early than breaking compatibility
> > later).
>
> How does that ease binary compatibility if you want to add extra common
> fields later? You mean to have a pointer to the arch-dep struct?
Err, this was a bad argument.  I was thinking about avoiding #if arch /#endif 
in this public header.

> PS. The flag name: DOM0_DOMAIN_SETUP_ would actually be a fine prefix
> for flags. Use whatever you prefer.
Here is the patch.  I hope it looks like you expect it :-)

Tristan.

Attachment: xen-ds1.diffs
Description: Text Data

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