qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Joachim Henke <he.jo@gmx.de>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH] PC speaker emulation (fixed)
Date: Thu, 19 Jan 2006 21:44:56 +0100	[thread overview]
Message-ID: <E05AB379-D924-4CBA-88E6-DFDFDACC2C07@gmx.de> (raw)
In-Reply-To: <Pine.LNX.4.61.0601190326030.4127@home.oyster.ru>

[-- Attachment #1: Type: text/plain, Size: 869 bytes --]

Thanks a lot for your help! I wasn't aware that AUD_write could  
return zero. Seems that I was just lucky, it didn't do that with  
44100 Hz sample rate (c:

Now it works fine with 32000 Hz, and I really encourage people  
(especially PC users) to test the attached patch. Call QEMU with the  
switch '-soundhw pcspk', and try to make some noise.

Regards,
Jo.


malc wrote:
> b. Once AUD_write return zero one should stop writing untill some  
> space
>    becomes available (for all intents and purposes after callback
>    will be called next time)
>
> Item b is probably why it hangs: you try to write N bytes, AUD_write
> returns zero, yet you are tring again immediately - ad nauseam. We
> have only one thread - hence audio can not push the data out of
> internal buffers into the host, and what you get is an infinite loop.

-- 
Joachim Henke
http://he-jo.net/

[-- Attachment #2: pcspeaker.diff.gz --]
[-- Type: application/x-gzip, Size: 3162 bytes --]

  reply	other threads:[~2006-01-19 20:47 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-09 19:01 [Qemu-devel] [patch] minimal PC speaker output Joachim Henke
2006-01-09 20:40 ` Fabrice Bellard
2006-01-10  9:11   ` he.jo
2006-01-18 23:05   ` [Qemu-devel] [PATCH] PC speaker emulation Joachim Henke
2006-01-18 23:32     ` [Qemu-devel] " Joachim Henke
2006-01-19  0:32     ` [Qemu-devel] " malc
2006-01-19 20:44       ` Joachim Henke [this message]
2006-01-19 23:29         ` [Qemu-devel] [PATCH] PC speaker emulation (fixed) malc
2006-01-20  9:42           ` Joachim Henke
2006-01-20 13:03             ` Johannes Schindelin
2006-01-20 12:03           ` Sebastian Kaliszewski
2006-01-23 21:10             ` [Qemu-devel] [PATCH] PC speaker emulation (fixed point) Joachim Henke
2006-01-24 12:35               ` Sebastian Kaliszewski
2006-01-24 15:50                 ` Joachim Henke
2006-01-24 18:29                   ` Sebastian Kaliszewski
2006-01-24 21:45                     ` Fabrice Bellard
2006-01-25  0:38                       ` Sebastian Kaliszewski
2006-01-25 21:54                       ` [Qemu-devel] [PATCH] PC speaker emulation (square wave) Joachim Henke
2006-01-26  4:50                         ` Jamie Lokier
2006-01-26 16:55                         ` Sebastian Kaliszewski
2006-01-29 10:44                           ` Joachim Henke

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=E05AB379-D924-4CBA-88E6-DFDFDACC2C07@gmx.de \
    --to=he.jo@gmx.de \
    --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).