tree 7a82b787c530ecea7f1f08775eb81ef358d10179
parent 07c1b73e2a027ea9a52677beeb6a943a3e357139
author Kai Vehmanen <kai.vehmanen@linux.intel.com> 1572627995 -0500
committer Mark Brown <broonie@kernel.org> 1572873675 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFHBAABCgAxFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl3AJcsTHGJyb29uaWVA
 a2VybmVsLm9yZwAKCRAk1otyXVSH0CU6B/9Tu4khl+bOkYggwqnEBkrt1xuH6OLn
 s1et0acP14Z13quyqTJjMDSPvDQRiUpq9C8j+1NcvvttjfbWnMvgzMMtF0GgD7Lr
 nJcxaoM47nhR0a4phEQAn5GuWMkovOmsXujiCLYyl8IqF8qSoH6RwhQBO3PIimBF
 LuTUNl8fkGL5n9yc+NI1cDIoFcEEpeRdEgQjNxnqvf0X+0SR+jAQXM20+mR8s8H/
 j4QxEb0OVpuKTGk428bUxVcCsa7OxeJHhWqLtwrfL7tu6MyxuIxVfgegkTCOBWIX
 sAxydMCRBXlFZCjPD5SAz23y2bkv3gJg6fbV3fDmYi3C6v9begKdielH
 =9CjK
 -----END PGP SIGNATURE-----

ASoC: hdac_hda: fix race in device removal

When ASoC card instance is removed containing a HDA codec,
hdac_hda_codec_remove() may run in parallel with codec resume.
This will cause problems if the HDA link is freed with
snd_hdac_ext_bus_link_put() while the codec is still in
middle of its resume process.

To fix this, change the order such that pm_runtime_disable()
is called before the link is freed. This will ensure any
pending runtime PM action is completed before proceeding
to free the link.

This issue can be easily hit with e.g. SOF driver by loading and
unloading the drivers.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20191101170635.26389-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>