All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Davis <paul@linuxaudiosystems.com>
To: Adam Tla/lka <atlka@pg.gda.pl>
Cc: Jaroslav Kysela <perex@suse.cz>, alsa-devel@alsa-project.org
Subject: Re: Re: [Alsa-user] AD1985 full-duplex(?)
Date: Thu, 09 Sep 2004 08:59:51 -0400	[thread overview]
Message-ID: <200409091259.i89CxpQM019899@localhost.localdomain> (raw)
In-Reply-To: Your message of "Thu, 09 Sep 2004 07:52:07 +0200." <20040909055206.GD4584@sunrise.pg.gda.pl>

>OK, that's correct if you need this for plugins in ALSA lib, but from
>app point of view it should be as simple as possible. An app working in
>mmapped mode should only use function which gets hw_ptr and not push any
>pointers forward. If I need load huge data into an app I just fill ring

if you don't "push pointers forward", there is no way for ALSA to
determine whether there has been an xrun. ALSA can automatically track
the h/w pointer (most of the time), but if it doesn't know how much
data the application has written (and where), it cannot compare the
two values to check for xruns.

>update_sound() {
>	hw_ptr = get_driver_hw_ptr();
>	avail = hw_ptr - old_hw_ptr;
>	old_hw_ptr = hw_ptr;
>	appl_ofs = hw_ofs = hw_ptr % buffer_size;
>	
>	if (avail < 0)
>		avail += buffer_boundary;
>	if (avail > buffer_size)
>		avail = buffer_size;
>	else {
>		appl_ofs -= avail;
>		if (appl_ofs < 0)
>			appl_ofs += buffer_size;
>	}
>	fill_buffer(appl_ofs, avail);
>}

>Clear and simple. I can modify also the data previously written in
>case of game events. I think ALSA API overcomplicates the case from
>application point of view.

the problem with this approach is that it doesn't take into account
several possibilities:

	a) there is no actual mmap-able buffer; ALSA may be providing
	     this via emulation (for certain hardware or certain kinds
	     of virtual devices), and therefore needs to know how much
	     data you actually wrote.
	b) the accessible region of the buffer may not be contiguous.
	c) the h/w pointer may have already wrapped around to the
	     start of the buffer because of scheduling delays - your code
	     will fail miserably in this case.
	d) the "previously written" data may not be physically
	     accessible - there are a few audio interfaces that
	     are not bus masters, and require the CPU to transfer
	     data to them. the "old" part of the buffer will
	     never be resent.
        e) if the PCM device is "virtual", the stride distances
	     within the buffer will vary.

The simplicity of the OSS approach is appealing, very appealing. But
it fails to handle some very important situations, and thats why
ALSA's mmap API is a bit more complex.

--p


-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click

  reply	other threads:[~2004-09-09 12:59 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-31  8:52 Re: [Alsa-user] AD1985 full-duplex(?) Peter Zubaj
2004-08-31  9:39 ` Jaroslav Kysela
2004-09-06 20:45   ` Adam Tla/lka
2004-09-07  9:05     ` Jaroslav Kysela
2004-09-07 10:34       ` Adam Tla/lka
2004-09-07 13:23         ` Paul Davis
2004-09-07 13:40         ` Jaroslav Kysela
2004-09-08 17:15           ` Adam Tla/lka
     [not found]             ` <20040909122253.GE4584@sunrise.pg.gda.pl>
     [not found]               ` <Pine.LNX.4.58.0409091728420.4150@server.perex-int.cz>
2004-09-10  6:46                 ` Adam Tla/lka
2004-09-09  5:52       ` Adam Tla/lka
2004-09-09 12:59         ` Paul Davis [this message]
2004-09-09 13:28           ` Adam Tla/lka
2004-09-09 15:14         ` Jaroslav Kysela
2004-09-10  7:16           ` Adam Tla/lka
2004-09-10 11:44             ` Paul Davis
2004-09-10 19:04               ` Adam Tla/lka
2004-09-13 13:05                 ` Paul Davis
2004-09-13 17:24                   ` Adam Tla/lka
2004-09-26 22:21                   ` Adam Tlałka
2004-09-27  3:00                     ` Paul Davis
2004-09-27  6:38                       ` Adam Tlałka
2004-09-27 12:43                         ` Jaroslav Kysela
2004-09-28  5:11                           ` Adam Tlałka
2004-09-28 14:47                             ` Paul Davis
2004-09-29  5:51                               ` Adam Tlałka
2004-09-27 20:14                         ` Paul Davis
2004-09-28  6:10                           ` Adam Tlałka
     [not found] <200409281113.i8SBDo5U021462@localhost.localdomain>
2004-09-28 13:22 ` Adam Tlałka
2004-09-28 14:48   ` Jaroslav Kysela
2004-09-28 14:57   ` Paul Davis
2004-09-28 15:21     ` Takashi Iwai
2004-09-29  6:15     ` Adam Tlałka
     [not found] <Pine.HPX.4.33n.0408181538550.24798-100000@studcom.urz.uni-halle.de>
     [not found] ` <1092842830.13603.3.camel@localhost.localdomain>
     [not found]   ` <20040818181350.2b38e875@mango.fruits.de>
2004-08-18 17:37     ` Jaroslav Kysela
2004-08-18 18:15       ` Florian Schmidt
2004-08-19  8:58         ` Jaroslav Kysela
2004-08-19  9:46           ` Takashi Iwai
2004-08-19 10:28             ` Jaroslav Kysela
2004-08-23 11:36               ` Adam Tlałka
2004-08-23 11:54                 ` Jaroslav Kysela
2004-08-23 12:34                   ` Adam Tlałka
2004-08-23 14:39                     ` Jaroslav Kysela
2004-08-24  6:01                       ` Adam Tla/lka
2004-08-23 15:30                 ` Takashi Iwai
2004-08-28 19:10                   ` Adam Tlałka
2004-08-29  9:54                     ` Jaroslav Kysela
2004-08-29 18:35                       ` Adam Tlałka
2004-08-31  8:09                         ` Jaroslav Kysela
2004-08-19  9:48           ` Florian Schmidt
2004-08-20 10:58             ` Jaroslav Kysela

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=200409091259.i89CxpQM019899@localhost.localdomain \
    --to=paul@linuxaudiosystems.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=atlka@pg.gda.pl \
    --cc=perex@suse.cz \
    /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.