diff options
| author | Keyon Jie <yang.jie@linux.intel.com> | 2019-08-07 16:50:30 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2019-08-08 21:38:53 +0200 |
| commit | 804cbf4bb063204ca6c2471baa694548aab02ce3 (patch) | |
| tree | f5d9076b955dd9086b533a0b0792032dd96acc2b /sound/soc/codecs/hdac_hda.c | |
| parent | ASoC: SOF: Intel: hda: Initialize HDA controller after i915 init (diff) | |
| download | linux-804cbf4bb063204ca6c2471baa694548aab02ce3.tar.xz linux-804cbf4bb063204ca6c2471baa694548aab02ce3.zip | |
ASoC: hdac_hda: fix page fault issue by removing race
There is a race between hda codec device removing and the
jack-detecting work, which will lead to a page fault issue as the
latter work is accessing codec device which could be already removed.
Here add the cancellation of jack-detecting work before codecs are actually
removed to avoid the race and fix the issue.
Bug: https://github.com/thesofproject/linux/issues/1067
Signed-off-by: Keyon Jie <yang.jie@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20190807145030.26117-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs/hdac_hda.c')
| -rw-r--r-- | sound/soc/codecs/hdac_hda.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sound/soc/codecs/hdac_hda.c b/sound/soc/codecs/hdac_hda.c index 7d4940256914..91242b6f8ea7 100644 --- a/sound/soc/codecs/hdac_hda.c +++ b/sound/soc/codecs/hdac_hda.c @@ -495,6 +495,10 @@ static int hdac_hda_dev_probe(struct hdac_device *hdev) static int hdac_hda_dev_remove(struct hdac_device *hdev) { + struct hdac_hda_priv *hda_pvt; + + hda_pvt = dev_get_drvdata(&hdev->dev); + cancel_delayed_work_sync(&hda_pvt->codec.jackpoll_work); return 0; } |