[PATCH 0/3] ALSA: add snd_card_disconnect_sync()

[PATCH 0/3] ALSA: add snd_card_disconnect_sync()

[Kuninori Morimoto]

Hi Takashi-san, Mark

Current ALSA (SoC) might be kernel panic if user
unbinds sound driver during playback.

	> aplay xxx.wav &
	> echo xxxx > /sys/bus/platform/drivers/xxxx/unbind 

Main issue is we *can't* skip unbind with return -Exxx,
because unbind operation doesn't checks return value from each driver.
Thus, we *must* stop driver immediately when unbind.

1) is for ALSA
2) is for ALSA SoC
3) is for Renesas sound driver.

I think almost all driver want to have 3) type patch on .remove
but this patch-set doesn't care at this point.
Please let me know if they should have it.

Takashi Iwai (1):
  1) ALSA: add snd_card_disconnect_sync()

Kuninori Morimoto (2):
  2) ASoC: add snd_soc_card_disconnect_sync()
  3) ASoC: rsnd: call snd_soc_card_disconnect_sync() when remove

 include/sound/core.h     |  2 ++
 include/sound/soc.h      |  2 ++
 sound/core/init.c        | 18 ++++++++++++++++++
 sound/core/pcm.c         |  4 ++++
 sound/soc/sh/rcar/core.c |  2 ++
 sound/soc/soc-core.c     | 10 ++++++++++
 6 files changed, 38 insertions(+)

-- 
1.9.1



Best regards
---
Kuninori Morimoto

[PATCH 0/3] ALSA: add snd_card_disconnect_sync()

[Kuninori Morimoto]

Hi Takashi-san, Mark

Current ALSA (SoC) might be kernel panic if user
unbinds sound driver during playback.

	> aplay xxx.wav &
	> echo xxxx > /sys/bus/platform/drivers/xxxx/unbind 

Main issue is we *can't* skip unbind with return -Exxx,
because unbind operation doesn't checks return value from each driver.
Thus, we *must* stop driver immediately when unbind.

1) is for ALSA
2) is for ALSA SoC
3) is for Renesas sound driver.

I think almost all driver want to have 3) type patch on .remove
but this patch-set doesn't care at this point.
Please let me know if they should have it.

Takashi Iwai (1):
  1) ALSA: add snd_card_disconnect_sync()

Kuninori Morimoto (2):
  2) ASoC: add snd_soc_card_disconnect_sync()
  3) ASoC: rsnd: call snd_soc_card_disconnect_sync() when remove

 include/sound/core.h     |  2 ++
 include/sound/soc.h      |  2 ++
 sound/core/init.c        | 18 ++++++++++++++++++
 sound/core/pcm.c         |  4 ++++
 sound/soc/sh/rcar/core.c |  2 ++
 sound/soc/soc-core.c     | 10 ++++++++++
 6 files changed, 38 insertions(+)

-- 
1.9.1



Best regards
---
Kuninori Morimoto

[PATCH 0/3] ALSA: add snd_card_disconnect_sync()

[Kuninori Morimoto]

Hi Takashi-san, Mark

Current ALSA (SoC) might be kernel panic if user
unbinds sound driver during playback.

	> aplay xxx.wav &
	> echo xxxx > /sys/bus/platform/drivers/xxxx/unbind 

Main issue is we *can't* skip unbind with return -Exxx,
because unbind operation doesn't checks return value from each driver.
Thus, we *must* stop driver immediately when unbind.

1) is for ALSA
2) is for ALSA SoC
3) is for Renesas sound driver.

I think almost all driver want to have 3) type patch on .remove
but this patch-set doesn't care at this point.
Please let me know if they should have it.

Takashi Iwai (1):
  1) ALSA: add snd_card_disconnect_sync()

Kuninori Morimoto (2):
  2) ASoC: add snd_soc_card_disconnect_sync()
  3) ASoC: rsnd: call snd_soc_card_disconnect_sync() when remove

 include/sound/core.h     |  2 ++
 include/sound/soc.h      |  2 ++
 sound/core/init.c        | 18 ++++++++++++++++++
 sound/core/pcm.c         |  4 ++++
 sound/soc/sh/rcar/core.c |  2 ++
 sound/soc/soc-core.c     | 10 ++++++++++
 6 files changed, 38 insertions(+)

-- 
1.9.1



Best regards
---
Kuninori Morimoto