diff options
author | Takashi Iwai <tiwai@suse.de> | 2019-11-05 08:43:16 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2019-11-05 08:44:12 -0500 |
commit | 15c2b3cc09a31620914955cb2a89c277c18ee999 (patch) | |
tree | 73ad01a51339a6d91f827e454fe3148b041f279e | |
parent | 706ad6746a66546daf96d4e4a95e46faf6cf689a (diff) |
ALSA: hda/ca0132 - Fix possible workqueue stall
The unsolicited event handler for the headphone jack on CA0132 codec
driver tries to reschedule the another delayed work with
cancel_delayed_work_sync(). It's no good idea, unfortunately,
especially after we changed the work queue to the standard global
one; this may lead to a stall because both works are using the same
global queue.
Fix it by dropping the _sync but does call cancel_delayed_work()
instead.
Fixes: 993884f6a26c ("ALSA: hda/ca0132 - Delay HP amp turnon.")
BugLink: https://bugzilla.suse.com/show_bug.cgi?id=1155836
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20191105134316.19294-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/pci/hda/patch_ca0132.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c index 6d1fb7c11f17..b7a1abb3e231 100644 --- a/sound/pci/hda/patch_ca0132.c +++ b/sound/pci/hda/patch_ca0132.c | |||
@@ -7604,7 +7604,7 @@ static void hp_callback(struct hda_codec *codec, struct hda_jack_callback *cb) | |||
7604 | /* Delay enabling the HP amp, to let the mic-detection | 7604 | /* Delay enabling the HP amp, to let the mic-detection |
7605 | * state machine run. | 7605 | * state machine run. |
7606 | */ | 7606 | */ |
7607 | cancel_delayed_work_sync(&spec->unsol_hp_work); | 7607 | cancel_delayed_work(&spec->unsol_hp_work); |
7608 | schedule_delayed_work(&spec->unsol_hp_work, msecs_to_jiffies(500)); | 7608 | schedule_delayed_work(&spec->unsol_hp_work, msecs_to_jiffies(500)); |
7609 | tbl = snd_hda_jack_tbl_get(codec, cb->nid); | 7609 | tbl = snd_hda_jack_tbl_get(codec, cb->nid); |
7610 | if (tbl) | 7610 | if (tbl) |