Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-22 Thread Georg Chini
On 22.02.2018 09:49, Tanu Kaskinen wrote: On Thu, 2018-02-22 at 09:26 +0100, Georg Chini wrote: On 21.02.2018 21:02, Georg Chini wrote: On 21.02.2018 20:15, Raman Shishniou wrote: On 02/21/2018 09:41 PM, Raman Shuishniou wrote: 21.02.2018 20:07, Georg Chini пишет: Maybe you misunderstood

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-22 Thread Tanu Kaskinen
On Thu, 2018-02-22 at 09:26 +0100, Georg Chini wrote: > On 21.02.2018 21:02, Georg Chini wrote: > > On 21.02.2018 20:15, Raman Shishniou wrote: > > > On 02/21/2018 09:41 PM, Raman Shuishniou wrote: > > > > 21.02.2018 20:07, Georg Chini пишет: > > > > > Maybe you misunderstood me. What I mean, is

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-22 Thread Georg Chini
On 21.02.2018 21:02, Georg Chini wrote: On 21.02.2018 20:15, Raman Shishniou wrote: On 02/21/2018 09:41 PM, Raman Shuishniou wrote: 21.02.2018 20:07, Georg Chini пишет: Maybe you misunderstood me. What I mean, is that the pipe can be opened for writing as long as we are suspended. So it open

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 09:41 PM, Raman Shuishniou wrote: > 21.02.2018 20:07, Georg Chini пишет: >> Maybe you misunderstood me. What I mean, is that the pipe can be >> opened for writing as long as we are suspended. So it open when >> we see that the source is suspended or when we auto suspend. Close >> it

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 20:15, Raman Shishniou wrote: On 02/21/2018 09:41 PM, Raman Shuishniou wrote: 21.02.2018 20:07, Georg Chini пишет: Maybe you misunderstood me. What I mean, is that the pipe can be opened for writing as long as we are suspended. So it open when we see that the source is suspended

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shuishniou
21.02.2018 20:07, Georg Chini пишет: Maybe you misunderstood me. What I mean, is that the pipe can be opened for writing as long as we are suspended. So it open when we see that the source is suspended or when we auto suspend. Close it as soon as the source gets unsuspended. This will avoid

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 17:09, Raman Shishniou wrote: On 02/21/2018 06:43 PM, Georg Chini wrote: On 21.02.2018 16:15, Raman Shishniou wrote: On 02/21/2018 05:59 PM, Georg Chini wrote: On 21.02.2018 15:33, Raman Shishniou wrote: On 02/21/2018 05:00 PM, Georg Chini wrote: On 21.02.2018 12:50, Raman

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 06:43 PM, Georg Chini wrote: > On 21.02.2018 16:15, Raman Shishniou wrote: >> On 02/21/2018 05:59 PM, Georg Chini wrote: >>> On 21.02.2018 15:33, Raman Shishniou wrote: On 02/21/2018 05:00 PM, Georg Chini wrote: > On 21.02.2018 12:50, Raman Shishniou wrote: >> On

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 16:15, Raman Shishniou wrote: On 02/21/2018 05:59 PM, Georg Chini wrote: On 21.02.2018 15:33, Raman Shishniou wrote: On 02/21/2018 05:00 PM, Georg Chini wrote: On 21.02.2018 12:50, Raman Shishniou wrote: On 02/21/2018 02:24 PM, Georg Chini wrote: On 21.02.2018 12:22, Raman

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 05:59 PM, Georg Chini wrote: > On 21.02.2018 15:33, Raman Shishniou wrote: >> On 02/21/2018 05:00 PM, Georg Chini wrote: >>> On 21.02.2018 12:50, Raman Shishniou wrote: On 02/21/2018 02:24 PM, Georg Chini wrote: > On 21.02.2018 12:22, Raman Shishniou wrote: >> On

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 15:33, Raman Shishniou wrote: On 02/21/2018 05:00 PM, Georg Chini wrote: On 21.02.2018 12:50, Raman Shishniou wrote: On 02/21/2018 02:24 PM, Georg Chini wrote: On 21.02.2018 12:22, Raman Shishniou wrote: On 02/21/2018 12:13 PM, Raman Shishniou wrote: On 02/21/2018 09:39 AM,

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 05:00 PM, Georg Chini wrote: > On 21.02.2018 12:50, Raman Shishniou wrote: >> On 02/21/2018 02:24 PM, Georg Chini wrote: >>> On 21.02.2018 12:22, Raman Shishniou wrote: On 02/21/2018 12:13 PM, Raman Shishniou wrote: > On 02/21/2018 09:39 AM, Georg Chini wrote: >> On

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 12:50, Raman Shishniou wrote: On 02/21/2018 02:24 PM, Georg Chini wrote: On 21.02.2018 12:22, Raman Shishniou wrote: On 02/21/2018 12:13 PM, Raman Shishniou wrote: On 02/21/2018 09:39 AM, Georg Chini wrote: On 21.02.2018 06:05, Georg Chini wrote: On 21.02.2018 05:55, Georg

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 02:24 PM, Georg Chini wrote: > On 21.02.2018 12:22, Raman Shishniou wrote: >> On 02/21/2018 12:13 PM, Raman Shishniou wrote: >>> On 02/21/2018 09:39 AM, Georg Chini wrote: On 21.02.2018 06:05, Georg Chini wrote: > On 21.02.2018 05:55, Georg Chini wrote: >> On 20.02.2018

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Georg Chini
On 21.02.2018 12:22, Raman Shishniou wrote: On 02/21/2018 12:13 PM, Raman Shishniou wrote: On 02/21/2018 09:39 AM, Georg Chini wrote: On 21.02.2018 06:05, Georg Chini wrote: On 21.02.2018 05:55, Georg Chini wrote: On 20.02.2018 22:34, Raman Shishniou wrote: On 02/20/2018 11:04 PM, Georg

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 12:13 PM, Raman Shishniou wrote: > On 02/21/2018 09:39 AM, Georg Chini wrote: >> On 21.02.2018 06:05, Georg Chini wrote: >>> On 21.02.2018 05:55, Georg Chini wrote: On 20.02.2018 22:34, Raman Shishniou wrote: > On 02/20/2018 11:04 PM, Georg Chini wrote: >> On 20.02.2018

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-21 Thread Raman Shishniou
On 02/21/2018 09:39 AM, Georg Chini wrote: > On 21.02.2018 06:05, Georg Chini wrote: >> On 21.02.2018 05:55, Georg Chini wrote: >>> On 20.02.2018 22:34, Raman Shishniou wrote: On 02/20/2018 11:04 PM, Georg Chini wrote: > On 20.02.2018 19:49, Raman Shishniou wrote: >> On 02/20/2018

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Georg Chini
On 21.02.2018 06:05, Georg Chini wrote: On 21.02.2018 05:55, Georg Chini wrote: On 20.02.2018 22:34, Raman Shishniou wrote: On 02/20/2018 11:04 PM, Georg Chini wrote: On 20.02.2018 19:49, Raman Shishniou wrote: On 02/20/2018 07:02 PM, Georg Chini wrote: On 20.02.2018 16:38, Raman Shyshniou

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Georg Chini
On 21.02.2018 05:55, Georg Chini wrote: On 20.02.2018 22:34, Raman Shishniou wrote: On 02/20/2018 11:04 PM, Georg Chini wrote: On 20.02.2018 19:49, Raman Shishniou wrote: On 02/20/2018 07:02 PM, Georg Chini wrote: On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Georg Chini
On 20.02.2018 22:34, Raman Shishniou wrote: On 02/20/2018 11:04 PM, Georg Chini wrote: On 20.02.2018 19:49, Raman Shishniou wrote: On 02/20/2018 07:02 PM, Georg Chini wrote: On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will remain running even if no writer is

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shishniou
On 02/20/2018 11:04 PM, Georg Chini wrote: > On 20.02.2018 19:49, Raman Shishniou wrote: >> On 02/20/2018 07:02 PM, Georg Chini wrote: >>> On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will remain running even if no writer is connected and therefore no data is

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shishniou
On 02/20/2018 11:04 PM, Georg Chini wrote: > On 20.02.2018 19:49, Raman Shishniou wrote: >> On 02/20/2018 07:02 PM, Georg Chini wrote: >>> On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will remain running even if no writer is connected and therefore no data is

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Georg Chini
On 20.02.2018 19:49, Raman Shishniou wrote: On 02/20/2018 07:02 PM, Georg Chini wrote: On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will remain running even if no writer is connected and therefore no data is produced. This patch adds the autosuspend= option to prevent

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shishniou
On 02/20/2018 07:02 PM, Georg Chini wrote: > On 20.02.2018 16:38, Raman Shyshniou wrote: >> Currently the pipe-source will remain running even if no >> writer is connected and therefore no data is produced. >> This patch adds the autosuspend= option to prevent this. >> Source will stay suspended

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shishniou
On 02/20/2018 07:19 PM, Raman Shishniou wrote: >>> +if (chunk.length) { If the source is running or idle, the chunk.length always will be 0. if (PA_SOURCE_IS_OPENED(u->source->thread_info.state)) { ... chunk.length = 0; } guaranteed that. >>> +/* We have a pending

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shishniou
On 02/20/2018 07:02 PM, Georg Chini wrote: > On 20.02.2018 16:38, Raman Shyshniou wrote: >> Currently the pipe-source will remain running even if no >> writer is connected and therefore no data is produced. >> This patch adds the autosuspend= option to prevent this. >> Source will stay suspended

Re: [pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Georg Chini
On 20.02.2018 16:38, Raman Shyshniou wrote: Currently the pipe-source will remain running even if no writer is connected and therefore no data is produced. This patch adds the autosuspend= option to prevent this. Source will stay suspended if no writer is connected. This option is enabled by

[pulseaudio-discuss] [PATCH v8] pipe-source: implement autosuspend option

2018-02-20 Thread Raman Shyshniou
Currently the pipe-source will remain running even if no writer is connected and therefore no data is produced. This patch adds the autosuspend= option to prevent this. Source will stay suspended if no writer is connected. This option is enabled by default. --- src/modules/module-pipe-source.c |