From: malc <av1474@comtv.ru>
To: "Bjørn Mork" <bjorn@mork.no>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] alsa broken when the host is suspended (or hibernated)
Date: Wed, 29 Jul 2009 15:57:15 +0400 (MSD) [thread overview]
Message-ID: <Pine.LNX.4.64.0907291551250.3728@linmac.oyster.ru> (raw)
In-Reply-To: <1248864666-31387-1-git-send-email-bjorn@mork.no>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1451 bytes --]
On Wed, 29 Jul 2009, Bjrn Mork wrote:
> audio output fails after resuming a host running a guest using alsa
> audio output. Messages such as
>
> alsa: Failed to write 882 frames to 0x1804b98
> alsa: Reason: Streams pipe error
>
> will appear repeatedly in the monitor. This is caused by alsaaudio.c
> not handling ESTRPIPE. Fix this by calling snd_pcm_resume() on
> ESTRPIPE.
>
> This bug is similar to the vlc bug discussed on
> https://trac.videolan.org/vlc/ticket/1286 and the fix is insired by
> the patch attached to that bug report
>
> Signed-off-by: BjЪЪrn Mork <bjorn@mork.no>
> ---
> audio/alsaaudio.c | 22 ++++++++++++++++++++++
> 1 files changed, 22 insertions(+), 0 deletions(-)
Thank you. I've massaged the patch a bit:
a. Trimmed the comment line to fit 80 columns
b. Suppressed tabs
c. Added -ESTRPIPE handling to alsa_run_in
d. Changed the verbose message to indicate which of the streams
is being resumed (input/output)
It would be very nice if you could verify that it still works since
my machine is not capable of suspending nor hibernating, it would be
very informative if you could run audio capture inside the guest to
verify c. and d.
The tree is at:
http://repo.or.cz/w/qemu/malc.git?a=shortlog;h=refs/heads/alsa
Or you can just apply first commit's patch.
[..snip..]
P.S. Perhaps you could also come up with a better subject line?
--
mailto:av1474@comtv.ru
next prev parent reply other threads:[~2009-07-29 11:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-29 10:51 [Qemu-devel] [PATCH] alsa broken when the host is suspended (or hibernated) Bjørn Mork
2009-07-29 11:57 ` malc [this message]
2009-07-29 12:36 ` Bjørn Mork
2009-07-29 13:45 ` malc
2009-07-29 13:46 ` Bjørn Mork
2009-07-29 13:51 ` malc
2009-07-30 7:44 ` [Qemu-devel] [PATCH] alsa: add host suspend/resume support Bjørn Mork
2009-07-30 10:47 ` [Qemu-devel] " malc
2009-07-30 7:47 ` [Qemu-devel] [PATCH] alsa broken when the host is suspended (or hibernated) Bjørn Mork
2009-07-29 16:58 ` Rob Landley
2009-07-29 18:57 ` malc
2009-07-30 10:29 ` malc
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=Pine.LNX.4.64.0907291551250.3728@linmac.oyster.ru \
--to=av1474@comtv.ru \
--cc=bjorn@mork.no \
--cc=qemu-devel@nongnu.org \
/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;
as well as URLs for NNTP newsgroup(s).