diff options
| author | Michal Koutný <mkoutny@suse.com> | 2023-03-14 19:33:32 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2023-03-15 06:50:59 -0600 |
| commit | a5fc1441af7719e93dc7a638a960befb694ade89 (patch) | |
| tree | c9f57714001f8fda9d75fc699d1f0be00b75bbc5 | |
| parent | fa780334a8c392d959ae05eb19f2410b3a1e6cb0 (diff) | |
| download | linux-a5fc1441af7719e93dc7a638a960befb694ade89.tar.gz linux-a5fc1441af7719e93dc7a638a960befb694ade89.tar.bz2 linux-a5fc1441af7719e93dc7a638a960befb694ade89.zip | |
io_uring/sqpoll: Do not set PF_NO_SETAFFINITY on sqpoll threads
Users may specify a CPU where the sqpoll thread would run. This may
conflict with cpuset operations because of strict PF_NO_SETAFFINITY
requirement. That flag is unnecessary for polling "kernel" threads, see
the reasoning in commit 01e68ce08a30 ("io_uring/io-wq: stop setting
PF_NO_SETAFFINITY on io-wq workers"). Drop the flag on poll threads too.
Fixes: 01e68ce08a30 ("io_uring/io-wq: stop setting PF_NO_SETAFFINITY on io-wq workers")
Link: https://lore.kernel.org/all/20230314162559.pnyxdllzgw7jozgx@blackpad/
Signed-off-by: Michal Koutný <mkoutny@suse.com>
Link: https://lore.kernel.org/r/20230314183332.25834-1-mkoutny@suse.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
| -rw-r--r-- | io_uring/sqpoll.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/io_uring/sqpoll.c b/io_uring/sqpoll.c index 0119d3f1a556..9db4bc1f521a 100644 --- a/io_uring/sqpoll.c +++ b/io_uring/sqpoll.c @@ -233,7 +233,6 @@ static int io_sq_thread(void *data) set_cpus_allowed_ptr(current, cpumask_of(sqd->sq_cpu)); else set_cpus_allowed_ptr(current, cpu_online_mask); - current->flags |= PF_NO_SETAFFINITY; mutex_lock(&sqd->lock); while (1) { |