Re: [PATCH -v2 1/2] x86-32: Fix crashes with CPU hotplug on AMDmachines

From: Borislav Petkov
Date: Wed Aug 18 2010 - 15:09:45 EST


From: "H. Peter Anvin" <hpa@xxxxxxxxx>
Date: Wed, Aug 18, 2010 at 02:41:17PM -0400

> On 08/16/2010 05:38 AM, Borislav Petkov wrote:
> > #else
> > static inline void reserve_trampoline_memory(void) {};
> > +extern void __init setup_trampoline_page_table(void) {};
> > #endif /* CONFIG_X86_TRAMPOLINE */
>
> This breaks compilation in the !CONFIG_X86_TRAMPOLINE case; I presume
> you meant:
>
> static inline void setup_trampoline_page_table(void) {}
>
> ... here too?
>
> (No semicolon after a function body!)

Yeah, thanks for catching that - this should be a static inline stub in
the !CONFIG_X86_TRAMPOLINE case.

--
Regards/Gruss,
Boris.

Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/