mirror of
https://github.com/torvalds/linux.git
synced 2026-07-28 10:09:10 +02:00
sched_ext: Rename ops_cpu_valid() to scx_cpu_valid() and expose it
Rename the static ext.c helper and declare it in ext_internal.h so ext_idle.c and the upcoming cid code can call it directly instead of relying on build_policy.c textual inclusion. Pure rename and visibility change. Signed-off-by: Tejun Heo <tj@kernel.org> Reviewed-by: Cheng-Yang Chou <yphbchou0911@gmail.com> Reviewed-by: Changwoo Min <changwoo@igalia.com> Reviewed-by: Andrea Righi <arighi@nvidia.com>
This commit is contained in:
parent
0e781a7853
commit
93292808b2
|
|
@ -1067,7 +1067,7 @@ static inline bool __cpu_valid(s32 cpu)
|
|||
}
|
||||
|
||||
/**
|
||||
* ops_cpu_valid - Verify a cpu number, to be used on ops input args
|
||||
* scx_cpu_valid - Verify a cpu number, to be used on ops input args
|
||||
* @sch: scx_sched to abort on error
|
||||
* @cpu: cpu number which came from a BPF ops
|
||||
* @where: extra information reported on error
|
||||
|
|
@ -1076,7 +1076,7 @@ static inline bool __cpu_valid(s32 cpu)
|
|||
* Verify that it is in range and one of the possible cpus. If invalid, trigger
|
||||
* an ops error.
|
||||
*/
|
||||
static bool ops_cpu_valid(struct scx_sched *sch, s32 cpu, const char *where)
|
||||
bool scx_cpu_valid(struct scx_sched *sch, s32 cpu, const char *where)
|
||||
{
|
||||
if (__cpu_valid(cpu)) {
|
||||
return true;
|
||||
|
|
@ -1691,7 +1691,7 @@ static struct scx_dispatch_q *find_dsq_for_dispatch(struct scx_sched *sch,
|
|||
if ((dsq_id & SCX_DSQ_LOCAL_ON) == SCX_DSQ_LOCAL_ON) {
|
||||
s32 cpu = dsq_id & SCX_DSQ_LOCAL_CPU_MASK;
|
||||
|
||||
if (!ops_cpu_valid(sch, cpu, "in SCX_DSQ_LOCAL_ON dispatch verdict"))
|
||||
if (!scx_cpu_valid(sch, cpu, "in SCX_DSQ_LOCAL_ON dispatch verdict"))
|
||||
return find_global_dsq(sch, tcpu);
|
||||
|
||||
return &cpu_rq(cpu)->scx.local_dsq;
|
||||
|
|
@ -3274,7 +3274,7 @@ static int select_task_rq_scx(struct task_struct *p, int prev_cpu, int wake_flag
|
|||
this_rq()->scx.in_select_cpu = false;
|
||||
p->scx.selected_cpu = cpu;
|
||||
*ddsp_taskp = NULL;
|
||||
if (ops_cpu_valid(sch, cpu, "from ops.select_cpu()"))
|
||||
if (scx_cpu_valid(sch, cpu, "from ops.select_cpu()"))
|
||||
return cpu;
|
||||
else
|
||||
return prev_cpu;
|
||||
|
|
@ -8822,7 +8822,7 @@ static void scx_kick_cpu(struct scx_sched *sch, s32 cpu, u64 flags)
|
|||
struct rq *this_rq;
|
||||
unsigned long irq_flags;
|
||||
|
||||
if (!ops_cpu_valid(sch, cpu, NULL))
|
||||
if (!scx_cpu_valid(sch, cpu, NULL))
|
||||
return;
|
||||
|
||||
local_irq_save(irq_flags);
|
||||
|
|
@ -8919,7 +8919,7 @@ __bpf_kfunc s32 scx_bpf_dsq_nr_queued(u64 dsq_id, const struct bpf_prog_aux *aux
|
|||
} else if ((dsq_id & SCX_DSQ_LOCAL_ON) == SCX_DSQ_LOCAL_ON) {
|
||||
s32 cpu = dsq_id & SCX_DSQ_LOCAL_CPU_MASK;
|
||||
|
||||
if (ops_cpu_valid(sch, cpu, NULL)) {
|
||||
if (scx_cpu_valid(sch, cpu, NULL)) {
|
||||
ret = READ_ONCE(cpu_rq(cpu)->scx.local_dsq.nr);
|
||||
goto out;
|
||||
}
|
||||
|
|
@ -9308,7 +9308,7 @@ __bpf_kfunc u32 scx_bpf_cpuperf_cap(s32 cpu, const struct bpf_prog_aux *aux)
|
|||
guard(rcu)();
|
||||
|
||||
sch = scx_prog_sched(aux);
|
||||
if (likely(sch) && ops_cpu_valid(sch, cpu, NULL))
|
||||
if (likely(sch) && scx_cpu_valid(sch, cpu, NULL))
|
||||
return arch_scale_cpu_capacity(cpu);
|
||||
else
|
||||
return SCX_CPUPERF_ONE;
|
||||
|
|
@ -9336,7 +9336,7 @@ __bpf_kfunc u32 scx_bpf_cpuperf_cur(s32 cpu, const struct bpf_prog_aux *aux)
|
|||
guard(rcu)();
|
||||
|
||||
sch = scx_prog_sched(aux);
|
||||
if (likely(sch) && ops_cpu_valid(sch, cpu, NULL))
|
||||
if (likely(sch) && scx_cpu_valid(sch, cpu, NULL))
|
||||
return arch_scale_freq_capacity(cpu);
|
||||
else
|
||||
return SCX_CPUPERF_ONE;
|
||||
|
|
@ -9372,7 +9372,7 @@ __bpf_kfunc void scx_bpf_cpuperf_set(s32 cpu, u32 perf, const struct bpf_prog_au
|
|||
return;
|
||||
}
|
||||
|
||||
if (ops_cpu_valid(sch, cpu, NULL)) {
|
||||
if (scx_cpu_valid(sch, cpu, NULL)) {
|
||||
struct rq *rq = cpu_rq(cpu), *locked_rq = scx_locked_rq();
|
||||
struct rq_flags rf;
|
||||
|
||||
|
|
@ -9485,7 +9485,7 @@ __bpf_kfunc struct rq *scx_bpf_cpu_rq(s32 cpu, const struct bpf_prog_aux *aux)
|
|||
if (unlikely(!sch))
|
||||
return NULL;
|
||||
|
||||
if (!ops_cpu_valid(sch, cpu, NULL))
|
||||
if (!scx_cpu_valid(sch, cpu, NULL))
|
||||
return NULL;
|
||||
|
||||
if (!sch->warned_deprecated_rq) {
|
||||
|
|
@ -9542,7 +9542,7 @@ __bpf_kfunc struct task_struct *scx_bpf_cpu_curr(s32 cpu, const struct bpf_prog_
|
|||
if (unlikely(!sch))
|
||||
return NULL;
|
||||
|
||||
if (!ops_cpu_valid(sch, cpu, NULL))
|
||||
if (!scx_cpu_valid(sch, cpu, NULL))
|
||||
return NULL;
|
||||
|
||||
return rcu_dereference(cpu_rq(cpu)->curr);
|
||||
|
|
|
|||
|
|
@ -916,7 +916,7 @@ static s32 select_cpu_from_kfunc(struct scx_sched *sch, struct task_struct *p,
|
|||
bool we_locked = false;
|
||||
s32 cpu;
|
||||
|
||||
if (!ops_cpu_valid(sch, prev_cpu, NULL))
|
||||
if (!scx_cpu_valid(sch, prev_cpu, NULL))
|
||||
return -EINVAL;
|
||||
|
||||
if (!check_builtin_idle_enabled(sch))
|
||||
|
|
@ -989,7 +989,7 @@ __bpf_kfunc s32 scx_bpf_cpu_node(s32 cpu, const struct bpf_prog_aux *aux)
|
|||
guard(rcu)();
|
||||
|
||||
sch = scx_prog_sched(aux);
|
||||
if (unlikely(!sch) || !ops_cpu_valid(sch, cpu, NULL))
|
||||
if (unlikely(!sch) || !scx_cpu_valid(sch, cpu, NULL))
|
||||
return NUMA_NO_NODE;
|
||||
return cpu_to_node(cpu);
|
||||
}
|
||||
|
|
@ -1271,7 +1271,7 @@ __bpf_kfunc bool scx_bpf_test_and_clear_cpu_idle(s32 cpu, const struct bpf_prog_
|
|||
if (!check_builtin_idle_enabled(sch))
|
||||
return false;
|
||||
|
||||
if (!ops_cpu_valid(sch, cpu, NULL))
|
||||
if (!scx_cpu_valid(sch, cpu, NULL))
|
||||
return false;
|
||||
|
||||
return scx_idle_test_and_clear_cpu(cpu);
|
||||
|
|
|
|||
|
|
@ -1358,6 +1358,8 @@ DECLARE_PER_CPU(struct rq *, scx_locked_rq_state);
|
|||
|
||||
int scx_kfunc_context_filter(const struct bpf_prog *prog, u32 kfunc_id);
|
||||
|
||||
bool scx_cpu_valid(struct scx_sched *sch, s32 cpu, const char *where);
|
||||
|
||||
/*
|
||||
* Return the rq currently locked from an scx callback, or NULL if no rq is
|
||||
* locked.
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user