On Tue, 20 Nov 2018, Tim Chen wrote: > #if defined(CONFIG_SMP) && defined(CONFIG_HOTPLUG_SMT) > -extern enum cpuhp_smt_control cpu_smt_control; > +DECLARE_STATIC_KEY_TRUE(cpu_smt_enabled); > +#define cpu_use_smt_and_hotplug (static_branch_likely(&cpu_smt_enabled)) Errm. No. This looks like a variable and I really got confused by the code change. What's wrong with inline functions? Nothing, really. And no, you don't need that thing in the context switch code at all. Thanks, tglx
