[PATCH] sched/fair: cleanup master/slave usage in wake_wide()

[PATCH] sched/fair: cleanup master/slave usage in wake_wide()

[trix]

From: Tom Rix <trix@redhat.com>

From Documentation/process/coding-style section for
inclusive terminology, make these variable name changes in
wake_wide()

master -> primary
slave -> secondary

Signed-off-by: Tom Rix <trix@redhat.com>
---
 kernel/sched/fair.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 161b92aa1c79..ade9f065c690 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -5754,13 +5754,13 @@ static void record_wakee(struct task_struct *p)
  */
 static int wake_wide(struct task_struct *p)
 {
-	unsigned int master = current->wakee_flips;
-	unsigned int slave = p->wakee_flips;
+	unsigned int primary = current->wakee_flips;
+	unsigned int secondary = p->wakee_flips;
 	int factor = __this_cpu_read(sd_llc_size);
 
-	if (master < slave)
-		swap(master, slave);
-	if (slave < factor || master < slave * factor)
+	if (primary < secondary)
+		swap(primary, secondary);
+	if (secondary < factor || primary < secondary * factor)
 		return 0;
 	return 1;
 }
-- 
2.26.3

Re: [PATCH] sched/fair: cleanup master/slave usage in wake_wide()

[Peter Zijlstra]

On Sun, May 30, 2021 at 12:30:39PM -0700, trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
> 
> From Documentation/process/coding-style section for
> inclusive terminology, make these variable name changes in
> wake_wide()
> 
> master -> primary
> slave -> secondary
> 
> Signed-off-by: Tom Rix <trix@redhat.com>

Please take your nonsense elsewhere. Unless there's a technical reason
to change this code, don't touch it.