All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: malc <av1474@comtv.ru>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH] alsaaudio: increase default buffer sizes
Date: Mon, 05 May 2008 20:43:23 +0200	[thread overview]
Message-ID: <481F554B.1020100@web.de> (raw)
In-Reply-To: <Pine.LNX.4.64.0805052154490.5084@linmac.oyster.ru>

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

malc wrote:
> On Sun, 4 May 2008, Jan Kiszka wrote:
> 
>> Jan Kiszka wrote:
>>> malc wrote:
>>>> On Sun, 4 May 2008, Jan Kiszka wrote:
>>>>
> 
> [..snip..]
> 
>>>
>>> Yeah, great, that was the key! Find below the version that works for me
>>> (on 64 bit 8) ). It even obsoletes my buffer size patch. Please merge!
>>
>> Declaring the buffer size tweak obsolete was too quick. I still need
>> more under certain guest load, but now I'm already fine with
>> DEFAULT_BUFFER_SIZE=4096. Or all in one:
> 
> I wonder if everyone will be happy with quadrupled latency, apart from

Hmm, I wonder about the scenario where this latency may actually hurt.
QEMU is not /that/ fast anyway. ;)

> you and this other user on some (kvm was it) mailing list there are no
> huge outcries of dissatisfaction, then again i'm not sure how many people
> use ALSA+QEMU or QEMUs+AUDIO in general.

I don't think many users are actually running QEMU (or KVM) against
ALSA. You have to
 - manually enable it during configure
 - pass QEMU_AUDIO_DRV=alsa (OSS remains default even if ALSA is on - I
   ran into this trap first.)
 - possibly have to fix up your configuration with FIXED_FREQ and
   BUFFER_SIZE (as reported by >1 kvm users) until

As OSS does not give us shared sound device access and, in the end, is a
deprecated API under Linux, my goal is to overcome remaining issues of
the ALSA support and then make /this/ the default one (under Linux).

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 254 bytes --]

  reply	other threads:[~2008-05-05 18:45 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-02 19:06 [Qemu-devel] [PATCH] alsaaudio: increase default buffer sizes Jan Kiszka
2008-05-02 19:48 ` [Qemu-devel] " malc
2008-05-02 21:00   ` Jan Kiszka
2008-05-03 19:43     ` malc
2008-05-04  7:34       ` Jan Kiszka
2008-05-04 17:09         ` malc
2008-05-04 17:41           ` Jan Kiszka
2008-05-04 19:35             ` Jan Kiszka
2008-05-05 18:03               ` malc
2008-05-05 18:43                 ` Jan Kiszka [this message]
2008-05-06 17:37                   ` malc
2008-05-06 17:59                     ` Jan Kiszka
2008-05-07 18:36                       ` malc
2008-05-07 22:24                         ` Jan Kiszka
2008-05-08 17:44                           ` 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=481F554B.1020100@web.de \
    --to=jan.kiszka@web.de \
    --cc=av1474@comtv.ru \
    --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 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.