]> git.baikalelectronics.ru Git - kernel.git/commit
ASoC: soc-pcm: protect BE dailink state changes in trigger
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Tue, 17 Aug 2021 16:40:53 +0000 (11:40 -0500)
committerMark Brown <broonie@kernel.org>
Thu, 26 Aug 2021 16:42:07 +0000 (17:42 +0100)
commit28368a047d96be348a4d9b0b44eaaccb51003553
treee0e09e874abaa7c6cff930ecc4631faf777a8a80
parentb3d6478c7b618da7acb16cdb30e7d47b568bffaf
ASoC: soc-pcm: protect BE dailink state changes in trigger

When more than one FE is connected to a BE, e.g. in a mixing use case,
the BE can be triggered multiple times when the FE are opened/started
concurrently. This race condition is problematic in the case of
SoundWire BE dailinks, and this is not desirable in a general
case. The code carefully checks when the BE can be stopped or
hw_free'ed, but the trigger code does not use any mutual exclusion.

Fix by using the same spinlock already used to check FE states, and
set the state before the trigger. In case of errors,  the initial
state will be restored.

This patch does not change how the triggers are handled, it only makes
sure the states are handled in critical sections.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Message-Id: <20210817164054.250028-2-pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-pcm.c