From: Takashi Iwai <tiwai@suse.de>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
alsa-devel@alsa-project.org
Subject: Re: Re: No snd_pcm_resume_all() ?
Date: Tue, 18 Apr 2006 20:54:39 +0200 [thread overview]
Message-ID: <s5hd5fevhn4.wl%tiwai@suse.de> (raw)
In-Reply-To: <1145385653.4116.88.camel@localhost>
At Tue, 18 Apr 2006 20:40:53 +0200,
Johannes Berg wrote:
>
> On Tue, 2006-04-18 at 20:25 +0200, Takashi Iwai wrote:
>
> > Yes, of course, it's not an XRUN but "another type" of state.
> > The driver just notifes the SUSPENDED state when any access comes from
> > the app.
>
> I still don't see why it is necessary. I can see two cases:
>
> 1) the app is frozen before the driver suspends, so that we get into a
> regular xrun situation. That can be handled as an xrun
>
> 2) the driver suspends before the app is frozen, which means that the
> app won't be getting any pcm progress for a bit of time, which doesn't
> matter at all. Or is that the point where I'm mistaken, and it does
> matter to the app if the hardware pauses for a bit (say for video/audio
> sync)?
Another reason is that not all drivers provide the full resume
capability. Most drivers provide only the partial suspend -- the
driver can restore the hardware setting but the PCM must be restarted
for a proper operation. In this case, the app requires the explicit
reset-and-restart. So, it matter for apps what the current status
is.
IIRC, I discussed it for using a XRUN instead of an individual
SUSPENDED state in order to simplify the system, but it was rejected
by reasons I forgot :)
Takashi
-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
next prev parent reply other threads:[~2006-04-18 18:54 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-18 0:58 No snd_pcm_resume_all() ? Benjamin Herrenschmidt
2006-04-18 1:20 ` Lee Revell
2006-04-18 1:27 ` Benjamin Herrenschmidt
2006-04-18 1:36 ` Lee Revell
2006-04-18 1:45 ` Lee Revell
2006-04-18 2:13 ` Benjamin Herrenschmidt
2006-04-18 2:12 ` Benjamin Herrenschmidt
2006-04-18 13:12 ` Takashi Iwai
2006-04-18 17:55 ` Johannes Berg
2006-04-18 18:05 ` Takashi Iwai
2006-04-18 18:06 ` Johannes Berg
2006-04-18 18:10 ` Takashi Iwai
2006-04-18 18:12 ` Johannes Berg
2006-04-18 18:25 ` Takashi Iwai
2006-04-18 18:40 ` Johannes Berg
2006-04-18 18:54 ` Takashi Iwai [this message]
2006-04-18 19:27 ` Jaroslav Kysela
2006-04-18 20:59 ` Benjamin Herrenschmidt
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=s5hd5fevhn4.wl%tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=benh@kernel.crashing.org \
--cc=johannes@sipsolutions.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.