]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
pds_core: yield the CPU while waiting for the adminq to drain
authorNikhil P. Rao <nikhil.rao@amd.com>
Tue, 14 Jul 2026 20:14:56 +0000 (20:14 +0000)
committerJakub Kicinski <kuba@kernel.org>
Tue, 21 Jul 2026 19:43:23 +0000 (12:43 -0700)
commita11f0b8a204296fe7db9eaec53441012222cb004
treef2f5d82353dc45188715db9ae51eb69d3fadf141
parentdf282a9db8ac96bb6a30ed8534f77856b8dfbd8f
pds_core: yield the CPU while waiting for the adminq to drain

pdsc_adminq_wait_and_dec_once_unused() busy-waits for adminq_refcnt to
drop to one:

while (!refcount_dec_if_one(&pdsc->adminq_refcnt))
cpu_relax();

The refcount is held by pdsc_adminq_post() for the duration of an
in-flight command, which can wait up to devcmd_timeout seconds
(PDS_CORE_DEVCMD_TIMEOUT is 5) for the hardware to complete. cpu_relax()
is not a reschedule point, so on a non-preemptible kernel this loop can
spin on the CPU for several seconds, starving other tasks on that core.

Add cond_resched() to the loop so the waiter yields to other runnable
tasks while it polls, keeping cpu_relax() as the busy-wait hint between
checks.

Fixes: 7e82a8745b95 ("pds_core: Prevent race issues involving the adminq")
Reported-by: sashiko-bot <sashiko-bot@kernel.org>
Closes: https://sashiko.dev/#/patchset/20260629200358.2626129-1-nikhil.rao%40amd.com?part=2
Signed-off-by: Nikhil P. Rao <nikhil.rao@amd.com>
Reviewed-by: Eric Joyner <eric.joyner@amd.com>
Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Link: https://patch.msgid.link/20260714201456.1776153-1-nikhil.rao@amd.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/amd/pds_core/core.c