All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: "Kővágó, Zoltán" <dirty.ice.hu@gmail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 00/24] Audio: Mixeng-free 5.1/7.1 audio support
Date: Thu, 12 Sep 2019 12:20:07 +0200	[thread overview]
Message-ID: <20190912102007.73bch4avvmx3ugpe@sirius.home.kraxel.org> (raw)
In-Reply-To: <cover.1567965065.git.DirtY.iCE.hu@gmail.com>

On Sun, Sep 08, 2019 at 11:22:00PM +0200, Kővágó, Zoltán wrote:
> Hi,
> 
> This is the v2 of my patch series that makes mixeng optional and enables
> more than two audio channels.
> 
> Changes from v1:
> 
> * renamed "mixeng" option to "mixing-engine"
> * dropped patch "audio: remove hw->samples, buffer_size_in/out pcm_ops"

What is the testing status of this?

I've tried (quick playback smoke test):
  - pa -- works
  - alsa -- works.
  - oss -- not working on linux (also in master), probably lack of
    support in the linux kernel, should be testing with bsd.
  - spice -- broken (see below), master works fine.
  - sdl -- not working: "audio: Failed to create voice `adc'" (also in
    master).

What about windows+macos?

cheers,
  Gerd

spice:

qemu-system-x86_64: /home/kraxel/projects/qemu/audio/spiceaudio.c:143:
line_out_put_buffer: Assertion `buf == out->frame + out->fpos &&
out->fpos <= out->fsize' failed.

#0  0x00007f55ac24d377 in raise () at /lib64/libc.so.6
#1  0x00007f55ac24ea68 in abort () at /lib64/libc.so.6
#2  0x00007f55ac246196 in __assert_fail_base () at /lib64/libc.so.6
#3  0x00007f55ac246242 in  () at /lib64/libc.so.6
#4  0x0000557b648dfe30 in line_out_put_buffer (hw=0x557b66773a40,
buf=0x557b66709f5c, size=1764)
    at /home/kraxel/projects/qemu/audio/spiceaudio.c:143
#5  0x0000557b648d8bc6 in audio_pcm_hw_run_out (hw=0x557b66773a40,
live=441)
    at /home/kraxel/projects/qemu/audio/audio.c:1081
#6  0x0000557b648d8f16 in audio_run_out (s=0x557b65b68100)
    at /home/kraxel/projects/qemu/audio/audio.c:1166
#7  0x0000557b648d9804 in audio_run (s=0x557b65b68100,
msg=0x557b64e41f58 "timer")
    at /home/kraxel/projects/qemu/audio/audio.c:1349
#8  0x0000557b648d830b in audio_timer (opaque=0x557b65b68100)
    at /home/kraxel/projects/qemu/audio/audio.c:830
#9  0x0000557b64cdd1ec in timerlist_run_timers
(timer_list=0x557b65b23730)
    at /home/kraxel/projects/qemu/util/qemu-timer.c:592
#10 0x0000557b64cdd296 in qemu_clock_run_timers
(type=QEMU_CLOCK_VIRTUAL)
    at /home/kraxel/projects/qemu/util/qemu-timer.c:606
#11 0x0000557b64cdd556 in qemu_clock_run_all_timers ()
    at /home/kraxel/projects/qemu/util/qemu-timer.c:692
#12 0x0000557b64cddcf2 in main_loop_wait (nonblocking=0)
    at /home/kraxel/projects/qemu/util/main-loop.c:524
#13 0x0000557b648c9c9f in main_loop () at
/home/kraxel/projects/qemu/vl.c:1806
#14 0x0000557b648d135d in main (argc=18, argv=0x7ffda3815208,
envp=0x7ffda38152a0)
    at /home/kraxel/projects/qemu/vl.c:4488

(sleep 3; remote-viewer spice://localhost:12345) & qemu-default
-enable-kvm -name "qemu fedora audio testing" -M q35 -m 4G -cdrom
/vmdisk/iso/Fedora-Workstation-Live-x86_64-30-1.2.iso -device intel-hda
-device hda-micro,audiodev=snd0 -spice port=12345,disable-ticketing
-audiodev spice,id=snd0


  parent reply	other threads:[~2019-09-12 10:21 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-08 21:22 [Qemu-devel] [PATCH v2 00/24] Audio: Mixeng-free 5.1/7.1 audio support Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 01/24] audio: api for mixeng code free backends Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 02/24] alsaaudio: port to the new audio backend api Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 03/24] coreaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 04/24] dsoundaudio: " Kővágó, Zoltán
2019-09-11  7:00   ` Gerd Hoffmann
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 05/24] noaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 06/24] ossaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 07/24] paaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 08/24] sdlaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 10/24] wavaudio: " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 11/24] audio: remove remains of the old " Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 12/24] audio: unify input and output mixeng buffer management Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 13/24] audio: common rate control code for timer based outputs Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 14/24] audio: split ctl_* functions into enable_* and volume_* Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 15/24] audio: add mixeng option (documentation) Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 16/24] audio: make mixeng optional Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 18/24] audio: support more than two channels in volume setting Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 19/24] audio: replace shift in audio_pcm_info with bytes_per_frame Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 20/24] audio: basic support for multichannel audio Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 21/24] paaudio: channel-map option Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 22/24] usb-audio: do not count on avail bytes actually available Kővágó, Zoltán
2019-09-08 21:22 ` [Qemu-devel] [PATCH v2 24/24] usbaudio: change playback counters to 64 bit Kővágó, Zoltán
2019-09-12 10:20 ` Gerd Hoffmann [this message]
2019-09-15  1:08   ` [Qemu-devel] [PATCH v2 00/24] Audio: Mixeng-free 5.1/7.1 audio support Zoltán Kővágó
2019-09-17 11:38     ` 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=20190912102007.73bch4avvmx3ugpe@sirius.home.kraxel.org \
    --to=kraxel@redhat.com \
    --cc=dirty.ice.hu@gmail.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.