All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Jaroslav Kysela <perex@suse.cz>
Cc: Russell King <rmk+alsa@arm.linux.org.uk>,
	Roc Wu <cooloney@yahoo.com.cn>,
	Clemens Ladisch <clemens@ladisch.de>,
	Alsa-devel@lists.sourceforge.net
Subject: Re: An driver error when I using aplay!
Date: Mon, 07 Jun 2004 14:45:20 +0200	[thread overview]
Message-ID: <s5hk6yj4jvj.wl@alsa2.suse.de> (raw)
In-Reply-To: <Pine.LNX.4.58.0406071239110.3294@pnote.perex-int.cz>

At Mon, 7 Jun 2004 12:43:01 +0200 (CEST),
Jaroslav wrote:
> 
> On Mon, 7 Jun 2004, Russell King wrote:
> 
> > Actually, I disagree.  It's an ALSA bug.  The warning is created if
> > the AACI close method is called while the DMA or IO is still running.
> > If DMA is still running here, we've already freed the DMA buffer, so
> > we're either reading from or writing to memory we don't own - which is
> > a major bug.
> > 
> > The question is therefore: why is ALSA trying to shut down and free a
> > device which still has DMA running?  To be more explicit, why didn't
> > ALSA call the trigger callback with SNDRV_PCM_TRIGGER_STOP prior to
> > calling the hw_free or close methods?
> 
> The midlevel calls *drop() (which must stop the running stream) and then
> ->hw_free and ->close callbacks. I've never seen this error, so I suspect
> that something else is wrong.

i guess so, too.  as you can see in the original post, the error
returned from hw_params callback (sample not available), thus it
doesn't call trigger(START) callback yet at all.

unfurtunately i can't tell any more unless i read the driver code.
where can i find the code?


Takashi


-------------------------------------------------------
This SF.Net email is sponsored by the new InstallShield X.
>From Windows to Linux, servers to mobile, InstallShield X is the one
installation-authoring solution that does it all. Learn more and
evaluate today! http://www.installshield.com/Dev2Dev/0504

  reply	other threads:[~2004-06-07 12:45 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-04  9:13 An driver error when I using aplay! Roc Wu
2004-06-07  3:04 ` Roc Wu
2004-06-07  7:24   ` Clemens Ladisch
2004-06-07  9:25     ` Roc Wu
2004-06-07 10:17       ` Russell King
2004-06-07 10:43         ` Jaroslav Kysela
2004-06-07 12:45           ` Takashi Iwai [this message]
2004-06-07 13:08             ` Russell King
2004-06-07 13:40               ` Takashi Iwai
2004-06-07 13:51                 ` Russell King
2004-06-07 14:18                   ` Takashi Iwai
2004-06-07 15:04                     ` Russell King
2004-06-07 15:13                       ` Takashi Iwai
2004-06-07 15:18                         ` Russell King
2004-06-07 15:32                           ` Takashi Iwai
2004-06-07 15:44                             ` Russell King
2004-06-07 16:25                               ` Takashi Iwai
2004-06-07 18:04                                 ` Russell King
2004-06-08 15:48                                   ` Takashi Iwai
2004-06-08 16:40                                     ` Russell King
2004-06-08 16:48                                       ` Takashi Iwai
2004-06-07 14:24                   ` James Courtier-Dutton
2004-06-07 15:08                     ` Russell King
2004-06-08  4:01               ` Roc Wu
2004-06-07 10:44       ` Developer docs missing from ALSA web server - was:Re: [Alsa-devel] " James Courtier-Dutton

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=s5hk6yj4jvj.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=Alsa-devel@lists.sourceforge.net \
    --cc=clemens@ladisch.de \
    --cc=cooloney@yahoo.com.cn \
    --cc=perex@suse.cz \
    --cc=rmk+alsa@arm.linux.org.uk \
    /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.