Commit d13508f9 authored by Frederic Weisbecker's avatar Frederic Weisbecker
Browse files

nohz: Optimize full dynticks's sched hooks with static keys 



Scheduler IPIs and task context switches are serious fast path.
Let's try to hide as much as we can the impact of full
dynticks APIs' off case that are called on these sites
through the use of static keys.

Signed-off-by: default avatarFrederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Li Zhong <zhong@linux.vnet.ibm.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Kevin Hilman <khilman@linaro.org>
parent 460775df

include/linux/tick.h

+16 −4
Original line number Diff line number Diff line
@@ -180,20 +180,32 @@ static inline bool tick_nohz_full_cpu(int cpu) 
}



extern void tick_nohz_init(void);

extern void tick_nohz_full_check(void);

extern void __tick_nohz_full_check(void);

extern void tick_nohz_full_kick(void);

extern void tick_nohz_full_kick_all(void);

extern void tick_nohz_task_switch(struct task_struct *tsk);

extern void __tick_nohz_task_switch(struct task_struct *tsk);

#else

static inline void tick_nohz_init(void) { }

static inline bool tick_nohz_full_enabled(void) { return false; }

static inline bool tick_nohz_full_cpu(int cpu) { return false; }

static inline void tick_nohz_full_check(void) { }

static inline void __tick_nohz_full_check(void) { }

static inline void tick_nohz_full_kick(void) { }

static inline void tick_nohz_full_kick_all(void) { }

static inline void tick_nohz_task_switch(struct task_struct *tsk) { }

static inline void __tick_nohz_task_switch(struct task_struct *tsk) { }

#endif



static inline void tick_nohz_full_check(void)

{

	if (tick_nohz_full_enabled())

		__tick_nohz_full_check();

}



static inline void tick_nohz_task_switch(struct task_struct *tsk)

{

	if (tick_nohz_full_enabled())

		__tick_nohz_task_switch(tsk);

}





# ifdef CONFIG_CPU_IDLE_GOV_MENU

extern void menu_hrtimer_cancel(void);

kernel/time/tick-sched.c

+4 −4
Original line number Diff line number Diff line
@@ -198,7 +198,7 @@ static void tick_nohz_restart_sched_tick(struct tick_sched *ts, ktime_t now); 
 * Re-evaluate the need for the tick on the current CPU

 * and restart it if necessary.

 */

void tick_nohz_full_check(void)

void __tick_nohz_full_check(void)

{

	struct tick_sched *ts = &__get_cpu_var(tick_cpu_sched);
@@ -212,7 +212,7 @@ void tick_nohz_full_check(void) 


static void nohz_full_kick_work_func(struct irq_work *work)

{

	tick_nohz_full_check();

	__tick_nohz_full_check();

}



static DEFINE_PER_CPU(struct irq_work, nohz_full_kick_work) = {
@@ -231,7 +231,7 @@ void tick_nohz_full_kick(void) 


static void nohz_full_kick_ipi(void *info)

{

	tick_nohz_full_check();

	__tick_nohz_full_check();

}



/*
@@ -254,7 +254,7 @@ void tick_nohz_full_kick_all(void) 
 * It might need the tick due to per task/process properties:

 * perf events, posix cpu timers, ...

 */

void tick_nohz_task_switch(struct task_struct *tsk)

void __tick_nohz_task_switch(struct task_struct *tsk)

{

	unsigned long flags;

CRA Git | Maintained and supported by SUSTech CRA and CCSE