Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Gerd Hoffmann
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

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Michael Tokarev
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

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Gerd Hoffmann
Hi, Yes, I use sdl audio with sdl ui, sdl1 version (since sdl2 had its shortcomings in qemu, not all functionality worked). Does it work equally well with the gtk ui? Possibly it makes sense to use keep sdl audio and default to it only in case sdl ui is active (simliar to how spice audio

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Michael Tokarev
02.06.2015 13:45, Peter Maydell wrote: On 2 June 2015 at 10:12, Michael Tokarev m...@tls.msk.ru wrote: 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

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Paolo Bonzini
On 02/06/2015 11:28, Gerd Hoffmann wrote: 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. Useful data point.

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Peter Maydell
On 2 June 2015 at 10:12, Michael Tokarev m...@tls.msk.ru wrote: 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

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread François Revol
On 02/06/2015 13:40, Michael Tokarev wrote: Yes, that's true. As long as the native version actually works. As I said before, initially it didn't, and sdl was the only driver that worked. Maybe in 0.12, maybe even before, I don't remember anymore, but the fact remains, in our testing only

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-02 Thread Gerd Hoffmann
On Di, 2015-06-02 at 14:42 +0200, François Revol wrote: On 02/06/2015 13:40, Michael Tokarev wrote: Yes, that's true. As long as the native version actually works. As I said before, initially it didn't, and sdl was the only driver that worked. Maybe in 0.12, maybe even before, I don't

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-01 Thread Michael Tokarev
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... Thanks, /mjt

Re: [Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-01 Thread Kővágó Zoltán
2015-06-01 15:38 keltezéssel, Michael Tokarev írta: 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

[Qemu-devel] [PATCH 00/12] Audio backend cleanup

2015-06-01 Thread Kővágó, Zoltán
This series of patch removes the following audio backends: esd, fmod, sdl, winwave. It also cleans up the remaining drivers to do not use global variables where possible. This is a preparation for my GSoC project where I will enable multiple simultaneous audio backends. Please also test the