* [PATCH] sched: Fix some spelling mistakes in the scheduler module
@ 2025-09-26 9:28 jianyun.gao
2025-09-26 11:00 ` Peter Zijlstra
0 siblings, 1 reply; 4+ messages in thread
From: jianyun.gao @ 2025-09-26 9:28 UTC (permalink / raw)
To: linux-kernel
Cc: mingo, peterz, juri.lelli, vincent.guittot, dietmar.eggemann,
rostedt, bsegall, mgorman, vschneid, jianyun.gao
The following are some spelling mistakes existing in the scheduler
module. Just fix it!
slection -> selection
achitectures -> architectures
excempt -> except
incorectly -> incorrectly
litle -> little
faireness -> fairness
borken -> broken
condtion -> condition
Signed-off-by: jianyun.gao <jianyungao89@gmail.com>
---
kernel/sched/core.c | 2 +-
kernel/sched/cputime.c | 2 +-
kernel/sched/fair.c | 8 ++++----
kernel/sched/topology.c | 2 +-
kernel/sched/wait_bit.c | 2 +-
5 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 7f1e5cb94c53..af5076e40567 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -6858,7 +6858,7 @@ static void __sched notrace __schedule(int sched_mode)
/*
* We pass task_is_blocked() as the should_block arg
* in order to keep mutex-blocked tasks on the runqueue
- * for slection with proxy-exec (without proxy-exec
+ * for selection with proxy-exec (without proxy-exec
* task_is_blocked() will always be false).
*/
try_to_block_task(rq, prev, &prev_state,
diff --git a/kernel/sched/cputime.c b/kernel/sched/cputime.c
index 7097de2c8cda..2429be5a5e40 100644
--- a/kernel/sched/cputime.c
+++ b/kernel/sched/cputime.c
@@ -585,7 +585,7 @@ void cputime_adjust(struct task_cputime *curr, struct prev_cputime *prev,
stime = mul_u64_u64_div_u64(stime, rtime, stime + utime);
/*
* Because mul_u64_u64_div_u64() can approximate on some
- * achitectures; enforce the constraint that: a*b/(b+c) <= a.
+ * architectures; enforce the constraint that: a*b/(b+c) <= a.
*/
if (unlikely(stime > rtime))
stime = rtime;
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 18a30ae35441..20fe5899b247 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -5381,7 +5381,7 @@ dequeue_entity(struct cfs_rq *cfs_rq, struct sched_entity *se, int flags)
bool delay = sleep;
/*
* DELAY_DEQUEUE relies on spurious wakeups, special task
- * states must not suffer spurious wakeups, excempt them.
+ * states must not suffer spurious wakeups, except them.
*/
if (flags & (DEQUEUE_SPECIAL | DEQUEUE_THROTTLE))
delay = false;
@@ -5842,7 +5842,7 @@ static bool enqueue_throttled_task(struct task_struct *p)
* target cfs_rq's limbo list.
*
* Do not do that when @p is current because the following race can
- * cause @p's group_node to be incorectly re-insterted in its rq's
+ * cause @p's group_node to be incorrectly re-insterted in its rq's
* cfs_tasks list, despite being throttled:
*
* cpuX cpuY
@@ -12161,7 +12161,7 @@ static inline bool update_newidle_cost(struct sched_domain *sd, u64 cost)
* sched_balance_newidle() bumps the cost whenever newidle
* balance fails, and we don't want things to grow out of
* control. Use the sysctl_sched_migration_cost as the upper
- * limit, plus a litle extra to avoid off by ones.
+ * limit, plus a little extra to avoid off by ones.
*/
sd->max_newidle_lb_cost =
min(cost, sysctl_sched_migration_cost + 200);
@@ -13176,7 +13176,7 @@ static void propagate_entity_cfs_rq(struct sched_entity *se)
* If a task gets attached to this cfs_rq and before being queued,
* it gets migrated to another CPU due to reasons like affinity
* change, make sure this cfs_rq stays on leaf cfs_rq list to have
- * that removed load decayed or it can cause faireness problem.
+ * that removed load decayed or it can cause fairness problem.
*/
if (!cfs_rq_pelt_clock_throttled(cfs_rq))
list_add_leaf_cfs_rq(cfs_rq);
diff --git a/kernel/sched/topology.c b/kernel/sched/topology.c
index 1104d931c015..796ece7989e6 100644
--- a/kernel/sched/topology.c
+++ b/kernel/sched/topology.c
@@ -2399,7 +2399,7 @@ static struct sched_domain *build_sched_domain(struct sched_domain_topology_leve
if (!cpumask_subset(sched_domain_span(child),
sched_domain_span(sd))) {
- pr_err("BUG: arch topology borken\n");
+ pr_err("BUG: arch topology broken\n");
pr_err(" the %s domain not a subset of the %s domain\n",
child->name, sd->name);
/* Fixup, ensure @sd has at least @child CPUs. */
diff --git a/kernel/sched/wait_bit.c b/kernel/sched/wait_bit.c
index 1088d3b7012c..47ab3bcd2ebc 100644
--- a/kernel/sched/wait_bit.c
+++ b/kernel/sched/wait_bit.c
@@ -207,7 +207,7 @@ EXPORT_SYMBOL(init_wait_var_entry);
* given variable to change. wait_var_event() can be waiting for an
* arbitrary condition to be true and associates that condition with an
* address. Calling wake_up_var() suggests that the condition has been
- * made true, but does not strictly require the condtion to use the
+ * made true, but does not strictly require the condition to use the
* address given.
*
* The wake-up is sent to tasks in a waitqueue selected by hash from a
--
2.34.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sched: Fix some spelling mistakes in the scheduler module
2025-09-26 9:28 [PATCH] sched: Fix some spelling mistakes in the scheduler module jianyun.gao
@ 2025-09-26 11:00 ` Peter Zijlstra
2025-09-26 16:18 ` 高建云
2025-09-26 16:25 ` [PATCH v2] " jianyun.gao
0 siblings, 2 replies; 4+ messages in thread
From: Peter Zijlstra @ 2025-09-26 11:00 UTC (permalink / raw)
To: jianyun.gao
Cc: linux-kernel, mingo, juri.lelli, vincent.guittot,
dietmar.eggemann, rostedt, bsegall, mgorman, vschneid
On Fri, Sep 26, 2025 at 05:28:32PM +0800, jianyun.gao wrote:
> The following are some spelling mistakes existing in the scheduler
> module. Just fix it!
>
> borken -> broken
That is not a typo. Also, in general I;m not really a fan of spelling
patches.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] sched: Fix some spelling mistakes in the scheduler module
2025-09-26 11:00 ` Peter Zijlstra
@ 2025-09-26 16:18 ` 高建云
2025-09-26 16:25 ` [PATCH v2] " jianyun.gao
1 sibling, 0 replies; 4+ messages in thread
From: 高建云 @ 2025-09-26 16:18 UTC (permalink / raw)
To: Peter Zijlstra
Cc: linux-kernel, mingo, juri.lelli, vincent.guittot,
dietmar.eggemann, rostedt, bsegall, mgorman, vschneid
On Fri, Sep 26, 2025 at 7:00 PM Peter Zijlstra <peterz@infradead.org> wrote:
>
> On Fri, Sep 26, 2025 at 05:28:32PM +0800, jianyun.gao wrote:
> > The following are some spelling mistakes existing in the scheduler
> > module. Just fix it!
> >
>
> > borken -> broken
>
> That is not a typo.
OK. I will drop it in the PATCH v2.
> Also, in general I;m not really a fan of spelling
> patches.
>
Me too. But this was my first Linux kernel patch. It was also the
beginning of my contributions to the kernel. :)
Best regards !
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] sched: Fix some spelling mistakes in the scheduler module
2025-09-26 11:00 ` Peter Zijlstra
2025-09-26 16:18 ` 高建云
@ 2025-09-26 16:25 ` jianyun.gao
1 sibling, 0 replies; 4+ messages in thread
From: jianyun.gao @ 2025-09-26 16:25 UTC (permalink / raw)
To: peterz
Cc: bsegall, dietmar.eggemann, jianyungao89, juri.lelli,
linux-kernel, mgorman, mingo, rostedt, vincent.guittot, vschneid
The following are some spelling mistakes existing in the scheduler
module. Just fix it!
slection -> selection
achitectures -> architectures
excempt -> except
incorectly -> incorrectly
litle -> little
faireness -> fairness
condtion -> condition
Signed-off-by: jianyun.gao <jianyungao89@gmail.com>
---
kernel/sched/core.c | 2 +-
kernel/sched/cputime.c | 2 +-
kernel/sched/fair.c | 8 ++++----
kernel/sched/wait_bit.c | 2 +-
4 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 7f1e5cb94c53..af5076e40567 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -6858,7 +6858,7 @@ static void __sched notrace __schedule(int sched_mode)
/*
* We pass task_is_blocked() as the should_block arg
* in order to keep mutex-blocked tasks on the runqueue
- * for slection with proxy-exec (without proxy-exec
+ * for selection with proxy-exec (without proxy-exec
* task_is_blocked() will always be false).
*/
try_to_block_task(rq, prev, &prev_state,
diff --git a/kernel/sched/cputime.c b/kernel/sched/cputime.c
index 7097de2c8cda..2429be5a5e40 100644
--- a/kernel/sched/cputime.c
+++ b/kernel/sched/cputime.c
@@ -585,7 +585,7 @@ void cputime_adjust(struct task_cputime *curr, struct prev_cputime *prev,
stime = mul_u64_u64_div_u64(stime, rtime, stime + utime);
/*
* Because mul_u64_u64_div_u64() can approximate on some
- * achitectures; enforce the constraint that: a*b/(b+c) <= a.
+ * architectures; enforce the constraint that: a*b/(b+c) <= a.
*/
if (unlikely(stime > rtime))
stime = rtime;
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 18a30ae35441..20fe5899b247 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -5381,7 +5381,7 @@ dequeue_entity(struct cfs_rq *cfs_rq, struct sched_entity *se, int flags)
bool delay = sleep;
/*
* DELAY_DEQUEUE relies on spurious wakeups, special task
- * states must not suffer spurious wakeups, excempt them.
+ * states must not suffer spurious wakeups, except them.
*/
if (flags & (DEQUEUE_SPECIAL | DEQUEUE_THROTTLE))
delay = false;
@@ -5842,7 +5842,7 @@ static bool enqueue_throttled_task(struct task_struct *p)
* target cfs_rq's limbo list.
*
* Do not do that when @p is current because the following race can
- * cause @p's group_node to be incorectly re-insterted in its rq's
+ * cause @p's group_node to be incorrectly re-insterted in its rq's
* cfs_tasks list, despite being throttled:
*
* cpuX cpuY
@@ -12161,7 +12161,7 @@ static inline bool update_newidle_cost(struct sched_domain *sd, u64 cost)
* sched_balance_newidle() bumps the cost whenever newidle
* balance fails, and we don't want things to grow out of
* control. Use the sysctl_sched_migration_cost as the upper
- * limit, plus a litle extra to avoid off by ones.
+ * limit, plus a little extra to avoid off by ones.
*/
sd->max_newidle_lb_cost =
min(cost, sysctl_sched_migration_cost + 200);
@@ -13176,7 +13176,7 @@ static void propagate_entity_cfs_rq(struct sched_entity *se)
* If a task gets attached to this cfs_rq and before being queued,
* it gets migrated to another CPU due to reasons like affinity
* change, make sure this cfs_rq stays on leaf cfs_rq list to have
- * that removed load decayed or it can cause faireness problem.
+ * that removed load decayed or it can cause fairness problem.
*/
if (!cfs_rq_pelt_clock_throttled(cfs_rq))
list_add_leaf_cfs_rq(cfs_rq);
diff --git a/kernel/sched/wait_bit.c b/kernel/sched/wait_bit.c
index 1088d3b7012c..47ab3bcd2ebc 100644
--- a/kernel/sched/wait_bit.c
+++ b/kernel/sched/wait_bit.c
@@ -207,7 +207,7 @@ EXPORT_SYMBOL(init_wait_var_entry);
* given variable to change. wait_var_event() can be waiting for an
* arbitrary condition to be true and associates that condition with an
* address. Calling wake_up_var() suggests that the condition has been
- * made true, but does not strictly require the condtion to use the
+ * made true, but does not strictly require the condition to use the
* address given.
*
* The wake-up is sent to tasks in a waitqueue selected by hash from a
--
2.34.1
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-09-26 16:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-26 9:28 [PATCH] sched: Fix some spelling mistakes in the scheduler module jianyun.gao
2025-09-26 11:00 ` Peter Zijlstra
2025-09-26 16:18 ` 高建云
2025-09-26 16:25 ` [PATCH v2] " jianyun.gao
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®