Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Jaroslav Kysela <perex@perex.cz>
Cc: alsa-devel@alsa-project.org
Subject: Re: [alsa-cvslog] [ALSA GIT]ALSA utilities repository branch master updated. v1.0.26-26-ge05b903
Date: Tue, 09 Apr 2013 18:11:35 +0200	[thread overview]
Message-ID: <s5hhajfwuo8.wl%tiwai@suse.de> (raw)
In-Reply-To: <51643937.5020602@perex.cz>

At Tue, 09 Apr 2013 17:52:23 +0200,
Jaroslav Kysela wrote:
> 
> Date 9.4.2013 14:47, Takashi Iwai wrote:
> > At Mon,  8 Apr 2013 14:51:54 +0200 (CEST),
> > GIT server wrote:
> >>
> >> This is an automated email from the git hooks/post-receive script. It was
> >> generated because a ref change was pushed to the repository containing
> >> the project "ALSA utilities repository".
> >>
> >> The branch, master has been updated
> >>        via  e05b903b1fb16e967d838edac408304cd4470fee (commit)
> >>       from  46cb355d4ff41c8065b02644b0534ab42d16cb72 (commit)
> >>
> >> Those revisions listed above that are new to this repository have
> >> not appeared on any other notification email; so we list those
> >> revisions in full, below.
> >>
> >> - Log -----------------------------------------------------------------
> >> commit e05b903b1fb16e967d838edac408304cd4470fee
> >> Author: Jaroslav Kysela <perex@perex.cz>
> >> Date:   Mon Apr 8 14:49:31 2013 +0200
> >>
> >>     alsactl: move systemd config to the daemon mode
> >>     
> >>     Signed-off-by: Jaroslav Kysela <perex@perex.cz>
> > 
> > I'm not thrilled by the silent default behavior change like this.
> > This will affect all systems using systemd from now on.
> > 
> > I understand the reason behind it, but I wonder whether it's an
> > overkill.  Yet another daemon, unconditionally no matter whether
> > hotplug or not?  Hmm...
> 
> The question is, how we can detect the hotplug scheme. Almost all
> systems have USB today and laptops have PCI express card slots, so....
> Fedora has the systemd configs in the alsa-utils package. A removal of
> this package is sufficient do disable the daemon.

Yes, but then this would break the formerly working environment, too.

> Eventually, we can
> save the state periodically using cron (without the changes tracking),
> but my measurement is that the alsactl daemon eats approx. 150kb of memory.

I'm not concerned about alsactl memory footprint so much.
But having another daemon unconditionally just to save/restore the
mixer status is a question.

> Some other quick ideas:
> 
> - make the static/hotplug schemes depending on an environment variable
>   passed through the bootloader
> - another two packages on top of alsa-utils with two configs

Manageable, but maybe will become a bit messy...

> - save the last state inside the driver and offer it to the userspace
>   upon the card removal (seems overkill)

This won't be that bad, I think.  But, accessing the information after
removal is the problem, after all.  The device and proc files are
removed at disconnection, so user-space can't access it.  Otherwise
the normal "alsactl store" should have worked.  So this can't be a
card-basis interface.

A possible w/a would be that we can keep some stable ctl lists and
provide it as a (one-time) death message from a global proc or a sysfs
file, for example.  Then we don't need periodical sync.

Whether this is a better option?  I dunno, honestly, too...

> - run multiple daemon instances per hotplug card; the question is how
>   to detect the static card in the system (perhaps checking the PCI
>   config?) to avoid the daemon startup for those static cards

Yeah, when thinking a PCI hotplug, it's not easy.


Takashi

  reply	other threads:[~2013-04-09 16:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130408125154.6264F2651CD@alsa0.perex.cz>
2013-04-09 12:47 ` [alsa-cvslog] [ALSA GIT]ALSA utilities repository branch master updated. v1.0.26-26-ge05b903 Takashi Iwai
2013-04-09 15:52   ` Jaroslav Kysela
2013-04-09 16:11     ` Takashi Iwai [this message]
2013-04-09 16:38       ` Jaroslav Kysela
2013-04-09 20:07       ` David Henningsson
2013-04-10  6:10         ` Takashi Iwai
2013-04-10  6:32           ` Clemens Ladisch
2013-04-10  6:32           ` David Henningsson
2013-04-10  6:58             ` Takashi Iwai
2013-04-10  7:11               ` David Henningsson
2013-04-10  7:20                 ` Takashi Iwai
2013-04-10  9:20                   ` Jaroslav Kysela
2013-04-10  9:33                     ` Takashi Iwai
2013-04-10  7:10             ` 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=s5hhajfwuo8.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox