Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-04-05 Thread Takashi Iwai
On Thu, 04 Apr 2019 12:18:33 +0200, Timo Wischer wrote: > > >>> In principle, the PCM ops is supposed to be safe for operating a > >>> certain stuff. If a state change may happen during the operation, > >>> this should be called inside PCM stream lock. So the design of the > >>> new callback

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-04-04 Thread Timo Wischer
On 3/27/19 10:38, Takashi Iwai wrote: On Wed, 27 Mar 2019 10:26:56 +0100, Timo Wischer wrote: On 3/27/19 10:11, Takashi Iwai wrote: On Wed, 27 Mar 2019 09:34:40 +0100, Timo Wischer wrote: On 3/26/19 17:00, Takashi Iwai wrote: On Tue, 26 Mar 2019 16:16:54 +0100, Timo Wischer wrote: On

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-27 Thread Takashi Iwai
On Wed, 27 Mar 2019 10:26:56 +0100, Timo Wischer wrote: > > On 3/27/19 10:11, Takashi Iwai wrote: > > On Wed, 27 Mar 2019 09:34:40 +0100, > > Timo Wischer wrote: > >> On 3/26/19 17:00, Takashi Iwai wrote: > >>> On Tue, 26 Mar 2019 16:16:54 +0100, > >>> Timo Wischer wrote: > On 3/26/19 15:23,

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-27 Thread Timo Wischer
On 3/27/19 10:11, Takashi Iwai wrote: On Wed, 27 Mar 2019 09:34:40 +0100, Timo Wischer wrote: On 3/26/19 17:00, Takashi Iwai wrote: On Tue, 26 Mar 2019 16:16:54 +0100, Timo Wischer wrote: On 3/26/19 15:23, Takashi Iwai wrote: On Tue, 26 Mar 2019 12:25:37 +0100, Timo Wischer wrote: On

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-27 Thread Takashi Iwai
On Wed, 27 Mar 2019 09:34:40 +0100, Timo Wischer wrote: > > On 3/26/19 17:00, Takashi Iwai wrote: > > On Tue, 26 Mar 2019 16:16:54 +0100, > > Timo Wischer wrote: > >> On 3/26/19 15:23, Takashi Iwai wrote: > >>> On Tue, 26 Mar 2019 12:25:37 +0100, > >>> Timo Wischer wrote: > On 3/26/19 09:35,

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-27 Thread Timo Wischer
On 3/26/19 17:00, Takashi Iwai wrote: On Tue, 26 Mar 2019 16:16:54 +0100, Timo Wischer wrote: On 3/26/19 15:23, Takashi Iwai wrote: On Tue, 26 Mar 2019 12:25:37 +0100, Timo Wischer wrote: On 3/26/19 09:35, Takashi Iwai wrote: On Tue, 26 Mar 2019 08:49:33 +0100, wrote:

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-26 Thread Takashi Iwai
On Tue, 26 Mar 2019 16:16:54 +0100, Timo Wischer wrote: > > On 3/26/19 15:23, Takashi Iwai wrote: > > On Tue, 26 Mar 2019 12:25:37 +0100, > > Timo Wischer wrote: > >> On 3/26/19 09:35, Takashi Iwai wrote: > >> > >> On Tue, 26 Mar 2019 08:49:33 +0100, > >> wrote: > >>

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-26 Thread Timo Wischer
On 3/26/19 15:23, Takashi Iwai wrote: On Tue, 26 Mar 2019 12:25:37 +0100, Timo Wischer wrote: On 3/26/19 09:35, Takashi Iwai wrote: On Tue, 26 Mar 2019 08:49:33 +0100, wrote: From: Timo Wischer snd_pcm_link() can be called by the user as long as

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-26 Thread Takashi Iwai
On Tue, 26 Mar 2019 12:25:37 +0100, Timo Wischer wrote: > > On 3/26/19 09:35, Takashi Iwai wrote: > > On Tue, 26 Mar 2019 08:49:33 +0100, > wrote: > > From: Timo Wischer > > snd_pcm_link() can be called by the user as long as the device is not >

Re: [PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-26 Thread Takashi Iwai
On Tue, 26 Mar 2019 08:49:33 +0100, wrote: > > From: Timo Wischer > > snd_pcm_link() can be called by the user as long as the device is not > yet started. Therefore currently a driver which wants to iterate over > the linked substreams has to do this at the start trigger. But the start >

[PATCH 09/10] ALSA: pcm: Add snd_pcm_ops for snd_pcm_link()

2019-03-26 Thread twischer
From: Timo Wischer snd_pcm_link() can be called by the user as long as the device is not yet started. Therefore currently a driver which wants to iterate over the linked substreams has to do this at the start trigger. But the start trigger should not block for a long time. Therefore there is no