> -----Original Message----- > From: Jan Beulich <jbeul...@suse.com> > Sent: 03 February 2020 11:06 > To: Durrant, Paul <pdurr...@amazon.co.uk> > Cc: Wei Liu <w...@xen.org>; Xen Development List <xen- > de...@lists.xenproject.org>; Michael Kelley <mikel...@microsoft.com>; Wei > Liu <li...@microsoft.com>; Andrew Cooper <andrew.coop...@citrix.com>; > Roger Pau Monné <roger....@citrix.com> > Subject: Re: [PATCH v6 05/11] x86/hyperv: setup hypercall page > > On 03.02.2020 11:39, Durrant, Paul wrote: > >> -----Original Message----- > >> From: Wei Liu <w...@xen.org> > >> Sent: 31 January 2020 17:57 > >> To: Xen Development List <xen-devel@lists.xenproject.org> > >> Cc: Durrant, Paul <pdurr...@amazon.co.uk>; Michael Kelley > >> <mikel...@microsoft.com>; Wei Liu <li...@microsoft.com>; Wei Liu > >> <w...@xen.org>; Jan Beulich <jbeul...@suse.com>; Andrew Cooper > >> <andrew.coop...@citrix.com>; Roger Pau Monné <roger....@citrix.com> > >> Subject: Re: [PATCH v6 05/11] x86/hyperv: setup hypercall page > >> > >> (Note to self) > >> > >> On Fri, Jan 31, 2020 at 05:49:24PM +0000, Wei Liu wrote: > >> [...] > >>> +static uint64_t generate_guest_id(void) > >>> +{ > >>> + union hv_guest_os_id id; > >>> + > >> > >> id.raw = 0; > > > > Or just use a C99 initializer to set things up. A bit neater IMO. > > If you mean this for ... > > >>> + id.vendor = HV_XEN_VENDOR_ID; > >>> + id.major = xen_major_version(); > >>> + id.minor = xen_minor_version(); > > ... these three fields, then this won't work with rather old gcc > we still permit to be used. Using { .raw = 0 } would work afaict. >
Not even { .vendor = HV_XEN_VENDOR_ID } ? Paul > Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel