Unverified Commit 0b0eb770 authored Sep 29, 2025 by Marco Crivellari Committed by Mark Brown Oct 15, 2025

ASoC: replace use of system_wq with system_dfl_wq



Currently if a user enqueue a work item using schedule_delayed_work() the
used wq is "system_wq" (per-cpu wq) while queue_delayed_work() use
WORK_CPU_UNBOUND (used when a cpu is not specified). The same applies to
schedule_work() that is using system_wq and queue_work(), that makes use
again of WORK_CPU_UNBOUND.

This lack of consistentcy cannot be addressed without refactoring the API.

system_wq is a per-CPU worqueue, replaced by system_percpu_wq. Despite that,
system_wq in this change has been replaced by system_dfl_wq, because there
aren't per-cpu variables.

The old wq will be kept for a few release cylces.

Suggested-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Marco Crivellari <marco.crivellari@suse.com>
Link: https://patch.msgid.link/20250929155053.400342-2-marco.crivellari@suse.com


Signed-off-by: Mark Brown <broonie@kernel.org>

parent 5e537031

sound/soc/codecs/aw88081.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -779,7 +779,7 @@ static void aw88081_start(struct aw88081 *aw88081, bool sync_start)
		if (sync_start == AW88081_SYNC_START)
		aw88081_start_pa(aw88081);
		else
		queue_delayed_work(system_wq,
		queue_delayed_work(system_dfl_wq,
		&aw88081->start_work,
		AW88081_START_WORK_DELAY_MS);
		}

sound/soc/codecs/aw88166.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1310,7 +1310,7 @@ static void aw88166_start(struct aw88166 *aw88166, bool sync_start)
		if (sync_start == AW88166_SYNC_START)
		aw88166_start_pa(aw88166);
		else
		queue_delayed_work(system_wq,
		queue_delayed_work(system_dfl_wq,
		&aw88166->start_work,
		AW88166_START_WORK_DELAY_MS);
		}

sound/soc/codecs/aw88261.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -705,7 +705,7 @@ static void aw88261_start(struct aw88261 *aw88261, bool sync_start)
		if (sync_start == AW88261_SYNC_START)
		aw88261_start_pa(aw88261);
		else
		queue_delayed_work(system_wq,
		queue_delayed_work(system_dfl_wq,
		&aw88261->start_work,
		AW88261_START_WORK_DELAY_MS);
		}

sound/soc/codecs/aw88395/aw88395.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -75,7 +75,7 @@ static void aw88395_start(struct aw88395 *aw88395, bool sync_start)
		if (sync_start == AW88395_SYNC_START)
		aw88395_start_pa(aw88395);
		else
		queue_delayed_work(system_wq,
		queue_delayed_work(system_dfl_wq,
		&aw88395->start_work,
		AW88395_START_WORK_DELAY_MS);
		}

sound/soc/codecs/aw88399.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1340,7 +1340,7 @@ static void aw88399_start(struct aw88399 *aw88399, bool sync_start)
		if (sync_start == AW88399_SYNC_START)
		aw88399_start_pa(aw88399);
		else
		queue_delayed_work(system_wq,
		queue_delayed_work(system_dfl_wq,
		&aw88399->start_work,
		AW88399_START_WORK_DELAY_MS);
		}