>>> On 03.12.15 at 02:58, <huaitong....@intel.com> wrote: > On Wed, 2015-12-02 at 04:31 -0700, Jan Beulich wrote: >> > > > On 02.12.15 at 08:20, <huaitong....@intel.com> wrote: >> > > Does this even compile? There is already >> > > >> > > static void *get_xsave_addr(void *xsave, unsigned int >> > > xfeature_idx) >> > > >> > > higher in the same file. >> > > >> > > That function should be augmented to take a struct xsave_struct >> > > *xsave, >> > > look at whether the representation is compressed or not, and use >> > > the >> > > appropriate offset array. >> > > >> > Just because I have pulled staging branch when "static void >> > *get_xsave_addr(void *xsave, unsigned int xfeature_idx)" is not >> > added. >> >> Yet you use the function on patch 7 already... > The function used by patch 7 has same name but is added by me.
Seems like you don't get the point: You can't use in patch 7 a function that you only add in patch 8. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel