On 27/11/15 08:24, Jan Beulich wrote: >>>> On 26.11.15 at 17:58, <andrew.coop...@citrix.com> wrote: >> The name is chosen to be consistent with Linux. Doing this allows >> early_intel_workaround() to be removed from common code. >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Jan Beulich <jbeul...@suse.com> > with one adjustment suggestion (which I'll do on commit unless you > object):
Absolutely fine. > >> --- a/xen/arch/x86/cpu/intel.c >> +++ b/xen/arch/x86/cpu/intel.c >> @@ -156,10 +156,8 @@ static void __devinit set_cpuidmask(const struct >> cpuinfo_x86 *c) >> } >> } >> >> -void __devinit early_intel_workaround(struct cpuinfo_x86 *c) >> +static void __devinit early_init_intel(struct cpuinfo_x86 *c) > The __devinit should be dropped here considering that we mean > to drop them gradually anyway. I shall dust off my series which drops them all. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
