Commit d79ddb06 authored by Chengming Zhou's avatar Chengming Zhou Committed by Peter Zijlstra
Browse files

sched/psi: Move private helpers to sched/stats.h 



This patch move psi_task_change/psi_task_switch declarations out of
PSI public header, since they are only needed for implementing the
PSI stats tracking in sched/stats.h

psi_task_switch is obvious, psi_task_change can't be public helper
since it doesn't check psi_disabled static key. And there is no
any user now, so put it in sched/stats.h too.

Signed-off-by: default avatarChengming Zhou <zhouchengming@bytedance.com>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: default avatarJohannes Weiner <hannes@cmpxchg.org>
Link: https://lore.kernel.org/r/20220825164111.29534-5-zhouchengming@bytedance.com
parent e2ad8ab0

include/linux/psi.h

+0 −4
Original line number Diff line number Diff line
@@ -18,10 +18,6 @@ extern struct psi_group psi_system; 


void psi_init(void);



void psi_task_change(struct task_struct *task, int clear, int set);

void psi_task_switch(struct task_struct *prev, struct task_struct *next,

		     bool sleep);



void psi_memstall_enter(unsigned long *flags);

void psi_memstall_leave(unsigned long *flags);

kernel/sched/stats.h

+4 −0
Original line number Diff line number Diff line
@@ -107,6 +107,10 @@ __schedstats_from_se(struct sched_entity *se) 
}



#ifdef CONFIG_PSI

void psi_task_change(struct task_struct *task, int clear, int set);

void psi_task_switch(struct task_struct *prev, struct task_struct *next,

		     bool sleep);



/*

 * PSI tracks state that persists across sleeps, such as iowaits and

 * memory stalls. As a result, it has to distinguish between sleeps,