From: Abramo Bagnara <abramo.bagnara@libero.it>
To: Jaroslav Kysela <perex@perex.cz>
Cc: ALSA development <alsa-devel@alsa-project.org>
Subject: Re: PCM API update
Date: Sat, 12 Oct 2002 12:24:08 +0200 [thread overview]
Message-ID: <3DA7F848.CF341696@libero.it> (raw)
In-Reply-To: Pine.LNX.4.33.0210121208550.1975-100000@pnote.perex-int.cz
Jaroslav Kysela wrote:
>
> On Sat, 12 Oct 2002, Abramo Bagnara wrote:
>
> > Jaroslav Kysela wrote:
> > >
> > >
> > > In this case, I propose to change snd_pcm_avail() to snd_pcm_hwsync()
> > > function with description: "synchronize r/w pointers with hardware".
> > > Really, after some thinking, the return value from snd_pcm_avail() cannot
> > > be used for nothing serious. I simply don't like that delay() functions
> > > do more arithmetic than necessary. Overdesign has been criticized in this
> > > list, too.
> >
> > Let examine kernel level: snd_pcm_{plaback,capture}_delay do a
> > snd_pcm_update_hw_ptr and a snd_pcm_{playback_hw,capture}_avail (that's
> > only a wrap safe difference).
> >
> > Are you sure you want to add another ioctl and another API function in
> > all the PCM classes just to avoid a subtraction?
>
> You've already noted that delay() expression might be more complex, so the
> code path eats more CPU than needed for sync operation.
Actually this might be true in future, although it's hard to know if
your proposal will worth that change. If you believe so, I've no further
objections.
--
Abramo Bagnara mailto:abramo.bagnara@libero.it
Opera Unica Phone: +39.546.656023
Via Emilia Interna, 140
48014 Castel Bolognese (RA) - Italy
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
prev parent reply other threads:[~2002-10-12 10:25 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-11 18:43 PCM API update Jaroslav Kysela
2002-10-11 20:13 ` Abramo Bagnara
2002-10-12 8:54 ` Jaroslav Kysela
2002-10-12 9:12 ` Abramo Bagnara
2002-10-12 9:37 ` Jaroslav Kysela
2002-10-12 9:54 ` Abramo Bagnara
2002-10-12 10:10 ` Jaroslav Kysela
2002-10-12 10:24 ` Abramo Bagnara [this message]
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=3DA7F848.CF341696@libero.it \
--to=abramo.bagnara@libero.it \
--cc=alsa-devel@alsa-project.org \
--cc=perex@perex.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.