All of lore.kernel.org
 help / color / mirror / Atom feed
From: Colin Guthrie <gmane@colin.guthr.ie>
To: pulseaudio-discuss@mail.0pointer.de
Cc: alsa-devel@alsa-project.org
Subject: Re: alsa pulse bugs
Date: Mon, 05 May 2008 10:44:29 +0100	[thread overview]
Message-ID: <fvmktt$ofl$1@ger.gmane.org> (raw)
In-Reply-To: <fvmjd0$it8$1@ger.gmane.org>

Colin Guthrie wrote:
> tom@dbservice.com wrote:
>>> Re the snd_pcm_delay() including network latency (#3945), this clearly
>>> makes sense for network streams. Does you proposed fix include this
>>> delay (albeit with the improvement that it also will drop to 0 if there
>>> are no samples queued)?
>> snd_pcm_delay() should not include any network latency. The API is  
>> defined as 'read pointer - write pointer', and applications expect  
>> that. Or at least they expect that when all samples are played that  
>> the delay drops to zero.
> 
> With the caveat of very limited technical knowledge, I can agree on the 
> latter point (drop to 0 when all samples are played), but if it was 
> implemented sans net-delay in pulse would this not cause e.g. a-v sync 
> issues when playing via alsa to a networked PA server? If so then this 
> fix would introduce another bug.

Actually just having a very quick glance at the Alsa API docs, it 
doesn't mention that this value should be 0 if there are no samples to play:

http://www.alsa-project.org/alsa-doc/alsa-lib/group___p_c_m.html#ga0d9e14a4be65209eb549e48a9f07302

Closest it says is: "It's positive and less than buffer size in normal 
situation".

So perhaps this is an invalid assumption at the wine side?

Is there perhaps a more appropriate API call they can use to do whatever 
test they are doing?

Col

  parent reply	other threads:[~2008-05-05  9:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080505103410.zv8dezd9z44cksoc@dbservice.com>
2008-05-05  8:58 ` alsa pulse bugs Colin Guthrie
2008-05-05  9:11   ` tom
2008-05-05  9:18     ` Colin Guthrie
2008-05-05  9:37       ` tom
2008-05-05  9:44       ` Colin Guthrie [this message]
2008-05-05 10:22         ` tom
2008-05-05 10:31           ` Colin Guthrie
2008-05-06 19:38   ` [pulseaudio-discuss] " Lennart Poettering
     [not found] ` <20080506193616.GB25436@tango.0pointer.de>
2008-05-07  1:45   ` Colin Guthrie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='fvmktt$ofl$1@ger.gmane.org' \
    --to=gmane@colin.guthr.ie \
    --cc=alsa-devel@alsa-project.org \
    --cc=pulseaudio-discuss@mail.0pointer.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.