diff options
| author |   Cezary Rojewski <cezary.rojewski@intel.com> | 2025-05-30 16:10:18 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2025-06-02 12:26:45 +0100 |
| commit | 9ad1f3cd0d60444c69948854c7e50d2a61b63755 (patch) | |
| tree | e40ff9fe5e8701a7ee3cb9f0fb9e07d733e5bb1e /sound | |
| parent | ASoC: codecs: hda: Fix RPM usage count underflow (diff) | |
| download | linux-rng-9ad1f3cd0d60444c69948854c7e50d2a61b63755.tar.xz linux-rng-9ad1f3cd0d60444c69948854c7e50d2a61b63755.zip | |
ASoC: Intel: avs: Fix deadlock when the failing IPC is SET_D0IX
The procedure handling IPC timeouts and EXCEPTION_CAUGHT notification
shall cancel any D0IX work before proceeding with DSP recovery. If
SET_D0IX called from delayed_work is the failing IPC the procedure will
deadlock. Conditionally skip cancelling the work to fix that.
Fixes: 335c4cbd201d ("ASoC: Intel: avs: D0ix power state support")
Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://patch.msgid.link/20250530141025.2942936-3-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/intel/avs/ipc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sound/soc/intel/avs/ipc.c b/sound/soc/intel/avs/ipc.c index 08ed9d96738a..0314f9d4ea5f 100644 --- a/sound/soc/intel/avs/ipc.c +++ b/sound/soc/intel/avs/ipc.c @@ -169,7 +169,9 @@ static void avs_dsp_exception_caught(struct avs_dev *adev, union avs_notify_msg dev_crit(adev->dev, "communication severed, rebooting dsp..\n"); - cancel_delayed_work_sync(&ipc->d0ix_work); + /* Avoid deadlock as the exception may be the response to SET_D0IX. */ + if (current_work() != &ipc->d0ix_work.work) + cancel_delayed_work_sync(&ipc->d0ix_work); ipc->in_d0ix = false; /* Re-enabled on recovery completion. */ pm_runtime_disable(adev->dev); |