All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Lennart Poettering <mznyfn@0pointer.de>
Cc: Sophie Hamilton <sophie-alsa@theblob.org>,
	alsa-devel@alsa-project.org, Tony Vroon <tony@linx.net>
Subject: Re: Problems with safe API and snd-cs46xx
Date: Wed, 09 Sep 2009 16:14:51 +0200	[thread overview]
Message-ID: <s5h63bs19yc.wl%tiwai@suse.de> (raw)
In-Reply-To: <20090909140735.GC14530@tango.0pointer.de>

At Wed, 9 Sep 2009 16:07:35 +0200,
Lennart Poettering wrote:
> 
> On Wed, 09.09.09 14:35, Takashi Iwai (tiwai@suse.de) wrote:
> 
> > > I can confirm now that Audacious does indeed play correctly where it
> > > didn't before. However, using mplayer with the "-ao openal" switch
> > > still doesn't play correctly - in fact, it sounds the same as before -
> > > so it looks like OpenAL is actually doing things slightly differently
> > > than I thought. :/
> > 
> > Yes, likely.  The app like openal is usually more sensible regarding
> > latency, so "safe API" described there wasn't appropriate at all.
> 
> Hmm, so are you suggesting I should change that little text about the
> safe API subset I wrote?

Maybe a bit more addition would be helpful.  The realtime apps do
care the latency.  So, obviously it's not the target of your
description.

> So, users should always set first the buffer size, followed by the
> period size, is that correct?

Yes, in general, this order gives more chance for a larger buffer
size.  But, if you specify both buffer and period sizes, there
shouldn't be much difference.  Specifying the buffer size is
especially good if you don't give any period size.

The situation is improved now with 1.0.21a since I changed the
determination order in alsa-lib.  You'll get the largest buffer size
as default with 1.0.21a (let's see whether this gives any regressions
;)  But, if portability matters, setting thebuffer size would be
safer.

>  And if that fails, try the other way
> round, and if that fails set buffer size only? And if that fails set
> nothing?

Yes, this sounds reasonable.


thanks,

Takashi

  reply	other threads:[~2009-09-09 14:14 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-05 12:24 Problems with safe API and snd-cs46xx Sophie Hamilton
2009-09-07 12:32 ` Takashi Iwai
2009-09-07 13:29   ` Tony Vroon
2009-09-07 14:18     ` Raymond Yau
2009-09-07 14:25     ` Error with custom driver: playback drain error (DMA or IRQ trouble?) Stefan Schoenleitner
2009-09-07 15:01     ` Problems with safe API and snd-cs46xx Takashi Iwai
2009-09-07 15:07       ` Raymond Yau
2009-09-07 15:15         ` Takashi Iwai
2009-09-07 22:47       ` Lennart Poettering
2009-09-07 23:10         ` Sophie Hamilton
2009-09-08  6:29         ` Takashi Iwai
2009-09-08  7:46           ` Sophie Hamilton
2009-09-08  8:53             ` Takashi Iwai
2009-09-09 11:04               ` Takashi Iwai
2009-09-09 12:29                 ` Sophie Hamilton
2009-09-09 12:35                   ` Takashi Iwai
2009-09-09 14:07                     ` Lennart Poettering
2009-09-09 14:14                       ` Takashi Iwai [this message]
2009-09-09 14:27                         ` Lennart Poettering
2009-09-09 14:37                           ` Takashi Iwai
2009-09-10  8:47                             ` Raymond Yau
2009-09-08 13:38           ` Lennart Poettering
2009-09-08 14:42             ` Takashi Iwai
2009-09-09  2:18               ` Raymond Yau
2009-09-07 15:02   ` Sophie Hamilton
2009-09-07 17:04     ` Sophie Hamilton
2009-09-07 17:27       ` Takashi Iwai
2009-09-07 19:06         ` Sophie Hamilton
2009-09-08  6:38           ` Takashi Iwai
2009-09-08  8:19             ` Sophie Hamilton
2009-09-08  9:03               ` Takashi Iwai
2009-09-08 13:18               ` Raymond Yau
2009-09-08 13:21                 ` Takashi Iwai

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=s5h63bs19yc.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=mznyfn@0pointer.de \
    --cc=sophie-alsa@theblob.org \
    --cc=tony@linx.net \
    /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.