From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:54806)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <dirty.ice.hu@gmail.com>) id 1ZSlcz-0000GI-8g
	for qemu-devel@nongnu.org; Fri, 21 Aug 2015 08:41:02 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <dirty.ice.hu@gmail.com>) id 1ZSlcw-0001UY-40
	for qemu-devel@nongnu.org; Fri, 21 Aug 2015 08:41:01 -0400
Received: from mail-wi0-x233.google.com ([2a00:1450:400c:c05::233]:33627)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <dirty.ice.hu@gmail.com>) id 1ZSlcv-0001T7-TV
	for qemu-devel@nongnu.org; Fri, 21 Aug 2015 08:40:58 -0400
Received: by wijp15 with SMTP id p15so18810867wij.0
	for <qemu-devel@nongnu.org>; Fri, 21 Aug 2015 05:40:57 -0700 (PDT)
From: "=?UTF-8?B?S8WRdsOhZ8OzIFpvbHTDoW4=?=" <dirty.ice.hu@gmail.com>
References: <cover.1438884611.git.DirtY.iCE.hu@gmail.com>
	<48dee3b2d117f59d62cd8e6d4adf735cb2ee2da7.1438884611.git.DirtY.iCE.hu@gmail.com>
	<CAJ+F1CJ2fTMZx60Ny+Uh99v9omKTupZma5YMbjZsMHPnWE6Bjg@mail.gmail.com>
Message-ID: <55D71C5F.2060604@gmail.com>
Date: Fri, 21 Aug 2015 14:41:03 +0200
MIME-Version: 1.0
In-Reply-To: <CAJ+F1CJ2fTMZx60Ny+Uh99v9omKTupZma5YMbjZsMHPnWE6Bjg@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Subject: Re: [Qemu-devel] [PATCH 15/25] paaudio: do not create multiple
 connections to the same server
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= <marcandre.lureau@gmail.com>
Cc: QEMU <qemu-devel@nongnu.org>, Gerd Hoffmann <kraxel@redhat.com>

Hi,

2015-08-20 21:38 keltezéssel, Marc-André Lureau írta:
[snip]
>> +    flags =
>> +        PA_STREAM_INTERPOLATE_TIMING
>> +#ifdef PA_STREAM_ADJUST_LATENCY
>> +        |PA_STREAM_ADJUST_LATENCY
>> +#endif
>> +        |PA_STREAM_AUTO_TIMING_UPDATE;
>> +    if (dev) {
>> +        /* don't move the stream if the user specified a sink/source */
>> +        flags |= PA_STREAM_DONT_MOVE;
>
> This is unrelated, and I don't think it's justified, imho user should
> be allowed to move the stream later if needed.

True, I will remove it from this commit.  But not sure if I want to 
remove it completely, because it looks like pulseaudio has a tendency to 
automatically move the stream on connect, even if you explicitly specify 
a sink/source when connecting.  (And the problem is that all streams 
appear under the name qemu in pulseaudio, so once the user moves one 
stream, pulseaudio will assume all streams subsequently created by qemu 
are the same on, and move them too.  Maybe we shold add some extra 
options to the pa backend?)