All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org, "\"Kővágó, Zoltán\"" <dirty.ice.hu@gmail.com>
Subject: Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup
Date: Tue, 02 Jun 2015 12:12:16 +0300	[thread overview]
Message-ID: <556D7370.5020303@msgid.tls.msk.ru> (raw)
In-Reply-To: <1433232504.1637.20.camel@nilsson.home.kraxel.org>

02.06.2015 11:08, Gerd Hoffmann wrote:
> On Mo, 2015-06-01 at 16:38 +0300, Michael Tokarev wrote:
>> 01.06.2015 16:23, Kővágó, Zoltán wrote:
>>> This series of patch removes the following audio backends:
>>> esd, fmod, sdl, winwave.
>>
>> Why do you remove sdl backend?
>>
>> This is my preferred backend so far, it works better
>> than all other backends available to me...
> 
> What other backends can you use?  What are the problems?
> Do you use sdl audio together with sdl ui?

I haven't had a chance to check, planned to do that
today morning but had some other things to do.  Will
check asap.

Yes, I use sdl audio with sdl ui, sdl1 version (since
sdl2 had its shortcomings in qemu, not all functionality
worked).

I also tried alsa, oss and pa, with these audio was more
distorted than with sdl.  Sdl version sounds cleanly, while
alsa and pa, at least in the past, and at least if memory
serves me right, sounded like a if a good portion of noize
has been mixed in with sound, or like when sound is with
quick monmentary drops which happens at high frequency
(maybe 200Hz).

SDL audio has been the default in Debian for at least two
debian releases because of that, we explicitly used
--audio-drv-list=sdl,... to list sdl first.  Now we have
pa first, but I don't really remember why we changed it.

But I have QEMU_AUDIO_DRV=sdl in my ~/.profile for a very
long time.  Not that I use audio all the time, only rarely,
but I never had any issues especially with sdl (this is
ofcourse NOT to say that there are no probs in there :)

Thanks,

/mjt

  reply	other threads:[~2015-06-02  9:12 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-01 13:23 [Qemu-devel] [PATCH 00/12] Audio backend cleanup Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 01/12] audio: remove esd backend Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 02/12] audio: remove fmod backend Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 03/12] audio: remove sdl backend Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 04/12] audio: remove winwave audio driver Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 05/12] audio: expose drv_opaque to init_out and init_in Kővágó, Zoltán
2015-06-02  7:56   ` Gerd Hoffmann
2015-06-02 10:03     ` Kővágó Zoltán
2015-06-02 15:24       ` Gerd Hoffmann
2015-06-01 13:23 ` [Qemu-devel] [PATCH 06/12] alsaaudio: do not use global variables Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 07/12] paaudio: " Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 08/12] ossaudio: " Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 09/12] wavaudio: " Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 10/12] dsoundaudio: " Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 11/12] paaudio: fix possible resource leak Kővágó, Zoltán
2015-06-01 13:23 ` [Qemu-devel] [PATCH 12/12] coreaudio: do not use global variables where possible Kővágó, Zoltán
2015-06-01 13:38 ` [Qemu-devel] [PATCH 00/12] Audio backend cleanup Michael Tokarev
2015-06-01 13:48   ` Kővágó Zoltán
2015-06-02  8:08   ` Gerd Hoffmann
2015-06-02  9:12     ` Michael Tokarev [this message]
2015-06-02  9:28       ` Gerd Hoffmann
2015-06-02 10:33         ` Paolo Bonzini
2015-06-02 10:45       ` Peter Maydell
2015-06-02 11:40         ` Michael Tokarev
2015-06-02 12:42           ` François Revol
2015-06-02 15:28             ` Gerd Hoffmann

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=556D7370.5020303@msgid.tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=dirty.ice.hu@gmail.com \
    --cc=kraxel@redhat.com \
    --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.