linux-kernel - [PATCH 0/4] workqueue: cleanups for schedule callbacks part2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20211223123140.3789-1-jiangshanlai@gmail.com>
Date:   Thu, 23 Dec 2021 20:31:36 +0800
From:   Lai Jiangshan <jiangshanlai@...il.com>
To:     linux-kernel@...r.kernel.org, Tejun Heo <tj@...nel.org>
Cc:     Lai Jiangshan <laijs@...ux.alibaba.com>
Subject: [PATCH 0/4] workqueue: cleanups for schedule callbacks part2

From: Lai Jiangshan <laijs@...ux.alibaba.com>

The commit 6d25be5782e4 ("sched/core, workqueues: Distangle worker
accounting from rq lock") changed the schedule callbacks for workqueue.

It simplified the connection between scheduler and workqueue.  But it
caused some code useless and some comments outdated in workqueue which
needs to be cleaned up.

Patch1-3 are the cleanups based on the fact that 6d25be5782e4 changed
to use pool lock in wq_worker_sleeping().

Patch4 is based on the fact that schedule callbacks were changed to be
only called from schedule() which means all modification to nr_running
is on local CPU when the worker is concurrent managed.

Part1:
https://lore.kernel.org/lkml/20211207073543.61092-1-jiangshanlai@gmail.com/

Most patches of part1 are queued.

Lai Jiangshan (4):
  workqueue: Remove the mb() pair between wq_worker_sleeping() and
    insert_work()
  workqueue: Change the comments of the synchronization about the
    idle_list
  workqueue: Use wake_up_worker() in wq_worker_sleeping() instead of
    open code
  workqueue: Convert the type of pool->nr_running to int

 kernel/workqueue.c | 58 ++++++++++++++++------------------------------
 1 file changed, 20 insertions(+), 38 deletions(-)

-- 
2.19.1.6.gb485710b