[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 1/6] domain: stash xen_domctl_createdomain flags in struct domain
> -----Original Message----- > From: Jan Beulich <JBeulich@xxxxxxxx> > Sent: 30 July 2019 10:17 > To: Paul Durrant <Paul.Durrant@xxxxxxxxxx> > Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx; Julien Grall <julien.grall@xxxxxxx>; > Andrew Cooper > <Andrew.Cooper3@xxxxxxxxxx>; George Dunlap <George.Dunlap@xxxxxxxxxx>; Ian > Jackson > <Ian.Jackson@xxxxxxxxxx>; Stefano Stabellini <sstabellini@xxxxxxxxxx>; Konrad > Rzeszutek Wilk > <konrad.wilk@xxxxxxxxxx>; Tim (Xen.org) <tim@xxxxxxx>; Wei Liu <wl@xxxxxxx> > Subject: Re: [Xen-devel] [PATCH v2 1/6] domain: stash xen_domctl_createdomain > flags in struct domain > > On 25.07.2019 15:39, Paul Durrant wrote: > > --- a/xen/include/xen/sched.h > > +++ b/xen/include/xen/sched.h > > @@ -306,6 +306,7 @@ enum guest_type { > > > > struct domain > > { > > + unsigned int options; /* copy of createdomain flags */ > > domid_t domain_id; > > > > unsigned int max_vcpus; > > I was about to commit this when I noticed the placement here: > I think it would be pretty good to retain domain_id as the > first field. I'll be happy to make that adjustment while > committing, as long as you are fine with me doing so. Yes, sure. I guess put it just below domid instead. Paul > > Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |