All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: batyi@mail.ru
Cc: alsa-devel@lists.sourceforge.net
Subject: Re: volume PCM plugin
Date: Mon, 17 Jan 2005 11:43:29 +0100	[thread overview]
Message-ID: <s5hoefoxrn2.wl@alsa2.suse.de> (raw)
In-Reply-To: <41EB82B0.2050900@mail.ru>

At Mon, 17 Jan 2005 15:17:36 +0600,
Petr Gladkikh wrote:
> 
> Hello.
> 
> Here is the patch (for alsa-lib-1.0.7) which is another attempt 
> (compare to coftvol plugin) to emulate volume control in alsa-lib (see 
> my previous post). The volume plugin is based on copy PCM plugin.

Thanks for the patch.

> IMO the approach which is used here would allow more flexible 
> configurations than softvol. E.g. it allows bind to master volume 
> control in addition to emulated volume control. If necessary the 
> volume plugin allows to easily extend number of hooks which can be 
> attached to it (for now only one is supported).

Handling with pthread looks nice.  The only concern is that I'm not
100% sure whether pthread works in every application.

As in my last reply, the other features can be easiply implemented in
softvol (the patch was attached there).  The multiple hooks are easy
to add as well.

So, for my eyes, only missing feature is the optimization of volume
control change via pthread and poll().  Although ioctl is relative
fast, it's of course better to eliminate to call at each time.
Maybe we can the behavior selectable via config option.
(The best optimization for this would be (read-only) mmap of
 contorls, but it's not implemented :)

IMO, adding multiple plugins which work almost identically is just
confusing.  Let's integrate them, instead.


Takashi


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt

  reply	other threads:[~2005-01-17 10:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-17 15:11 Problem closing device handle igor
2005-01-17  9:17 ` volume PCM plugin Petr Gladkikh
2005-01-17 10:43   ` Takashi Iwai [this message]
2005-01-17 11:16     ` Petr Gladkikh

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=s5hoefoxrn2.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=batyi@mail.ru \
    /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.