All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches
@ 2018-12-23 20:51 Kővágó, Zoltán
  2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 01/52] qapi: support alternates in OptsVisitor Kővágó, Zoltán
                   ` (50 more replies)
  0 siblings, 51 replies; 91+ messages in thread
From: Kővágó, Zoltán @ 2018-12-23 20:51 UTC (permalink / raw)
  To: qemu-devel; +Cc: Gerd Hoffmann

Hi,

I've updated my audio patchset to the current git master. Other than that not
much happened since my last update [1], fixed a few small problems that I
noticed while rebasing my patches.

Please review.

[1]: https://lists.gnu.org/archive/html/qemu-devel/2016-01/msg02451.html

Kővágó, Zoltán (52):
  qapi: support alternates in OptsVisitor
  qapi: support nested structs in OptsVisitor
  qapi: qapi for audio backends
  audio: use qapi AudioFormat instead of audfmt_e
  audio: -audiodev command line option: documentation
  audio: -audiodev command line option basic implementation
  alsaaudio: port to -audiodev config
  coreaudio: port to -audiodev config
  dsoundaudio: port to -audiodev config
  noaudio: port to -audiodev config
  ossaudio: port to -audiodev config
  paaudio: port to -audiodev config
  sdlaudio: port to -audiodev config
  spiceaudio: port to -audiodev config
  wavaudio: port to -audiodev config
  audio: -audiodev command line option: cleanup
  audio: reduce glob_audio_state usage
  audio: basic support for multi backend audio
  audio: add audiodev properties to frontends
  audio: audiodev= parameters no longer optional when -audiodev present
  paaudio: do not move stream when sink/source name is specified
  paaudio: properly disconnect streams in fini_*
  audio: remove audio_MIN, audio_MAX
  audio: do not run each backend in audio_run
  paaudio: fix playback glitches
  audio: remove read and write pcm_ops
  audio: use size_t where makes sense
  audio: api for mixeng code free backends
  alsaaudio: port to the new audio backend api
  coreaudio: port to the new audio backend api
  dsoundaudio: port to the new audio backend api
  noaudio: port to the new audio backend api
  ossaudio: port to the new audio backend api
  paaudio: port to the new audio backend api
  sdlaudio: port to the new audio backend api
  spiceaudio: port to the new audio backend api
  wavaudio: port to the new audio backend api
  audio: remove remains of the old backend api
  audio: unify input and output mixeng buffer management
  audio: remove hw->samples, buffer_size_in/out pcm_ops
  audio: common rate control code for timer based outputs
  audio: split ctl_* functions into enable_* and volume_*
  audio: add mixeng option (documentation)
  audio: make mixeng optional
  paaudio: get/put_buffer functions
  audio: support more than two channels in volume setting
  audio: replace shift in audio_pcm_info with bytes_per_frame
  audio: basic support for multichannel audio
  paaudio: channel-map option
  usb-audio: do not count on avail bytes actually available
  usb-audio: support more than two channels of audio
  usbaudio: change playback counters to 64 bit

 Makefile.objs                           |    6 +-
 audio/Makefile.objs                     |    3 +-
 audio/alsaaudio.c                       |  755 ++++-------
 audio/audio.c                           | 1561 ++++++++++++-----------
 audio/audio.h                           |   78 +-
 audio/audio_int.h                       |  131 +-
 audio/audio_legacy.c                    |  342 +++++
 audio/audio_pt_int.c                    |  174 ---
 audio/audio_pt_int.h                    |   22 -
 audio/audio_template.h                  |  112 +-
 audio/audio_win_int.c                   |   18 +-
 audio/coreaudio.c                       |  198 ++-
 audio/dsound_template.h                 |   63 +-
 audio/dsoundaudio.c                     |  449 ++-----
 audio/mixeng.h                          |   11 +-
 audio/noaudio.c                         |   98 +-
 audio/ossaudio.c                        |  594 ++++-----
 audio/paaudio.c                         |  881 ++++++-------
 audio/rate_template.h                   |    2 +-
 audio/sdlaudio.c                        |  218 ++--
 audio/spiceaudio.c                      |  286 ++---
 audio/wavaudio.c                        |  158 +--
 audio/wavcapture.c                      |    8 +-
 configure                               |    5 -
 hmp-commands.hx                         |   11 +-
 hw/acpi/core.c                          |    2 +-
 hw/arm/omap2.c                          |    2 +-
 hw/audio/ac97.c                         |   13 +-
 hw/audio/adlib.c                        |    7 +-
 hw/audio/cs4231a.c                      |   11 +-
 hw/audio/es1370.c                       |   17 +-
 hw/audio/gus.c                          |    9 +-
 hw/audio/hda-codec.c                    |   35 +-
 hw/audio/lm4549.c                       |    6 +-
 hw/audio/milkymist-ac97.c               |   16 +-
 hw/audio/pcspk.c                        |    5 +-
 hw/audio/pl041.c                        |    1 +
 hw/audio/sb16.c                         |   17 +-
 hw/audio/wm8750.c                       |   14 +-
 hw/core/qdev-properties-system.c        |   57 +
 hw/display/xlnx_dp.c                    |    2 +-
 hw/input/tsc210x.c                      |    2 +-
 hw/usb/dev-audio.c                      |  460 ++++++-
 include/hw/qdev-properties.h            |    3 +
 include/qapi/opts-visitor.h             |    9 +-
 include/sysemu/replay.h                 |    4 +-
 monitor.c                               |   12 +-
 net/net.c                               |    2 +-
 numa.c                                  |    2 +-
 qapi/audio.json                         |  261 ++++
 qapi/opts-visitor.c                     |  145 ++-
 qapi/qapi-schema.json                   |    1 +
 qemu-options.hx                         |  242 +++-
 qom/object_interfaces.c                 |    2 +-
 replay/replay-audio.c                   |   16 +-
 tests/qapi-schema/qapi-schema-test.json |    9 +-
 tests/qapi-schema/qapi-schema-test.out  |    4 +
 tests/test-opts-visitor.c               |   43 +-
 ui/vnc.c                                |   41 +-
 ui/vnc.h                                |    2 +
 vl.c                                    |   11 +-
 61 files changed, 3921 insertions(+), 3748 deletions(-)
 create mode 100644 audio/audio_legacy.c
 delete mode 100644 audio/audio_pt_int.c
 delete mode 100644 audio/audio_pt_int.h
 create mode 100644 qapi/audio.json

-- 
2.20.1

^ permalink raw reply	[flat|nested] 91+ messages in thread
[parent not found: <mailman.31873.1545598364.1282.qemu-devel@nongnu.org>]
* Re: [Qemu-devel] Qemu-devel Digest, Vol 189, Issue 424
@ 2019-01-01  4:20 Programmingkid
  2019-01-01  4:24 ` [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches Programmingkid
  0 siblings, 1 reply; 91+ messages in thread
From: Programmingkid @ 2019-01-01  4:20 UTC (permalink / raw)
  To: qemu-devel


> On Dec 31, 2018, at 12:00 PM, qemu-devel-request@nongnu.org wrote:
> 
> Message: 2
> Date: Mon, 31 Dec 2018 16:50:47 +0100
> From: "=?UTF-8?B?S8WRdsOhZ8OzIFpvbHTDoW4=?=" <dirty.ice.hu@gmail.com>
> To: Programmingkid <programmingkidx@gmail.com>
> Cc: QEMU Developers <qemu-devel@nongnu.org>, Gerd Hoffmann
> 	<kraxel@redhat.com>
> Subject: Re: [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches
> Message-ID: <d6966a53-3e5e-83c4-7900-38784969744a@gmail.com>
> Content-Type: text/plain; charset=utf-8
> 
> On 2018-12-29 01:49, Programmingkid wrote:
>> 
>>> On Dec 28, 2018, at 7:19 PM, Zolt?n K?v?g? <dirty.ice.hu@gmail.com> wrote:
>>> 
>>> On 2018-12-29 01:12, Programmingkid wrote:
>>>> 
>>>>> On Dec 28, 2018, at 3:05 PM, K?v?g? Zolt?n <dirty.ice.hu@gmail.com> wrote:
>>>>> 
>>>>> On 2018-12-28 01:46, Programmingkid wrote:
>>>>>> 
>>>>>>> On Dec 27, 2018, at 8:33 AM, K?v?g? Zolt?n <dirty.ice.hu@gmail.com> wrote:
>>>>>>> 
>>>>>>> Hi,
>>>>>>> 
>>>>>>> I've pushed it to my github (modulo some random fixes not yet on the
>>>>>>> mailing list):
>>>>>>> https://github.com/DirtYiCE/qemu/tree/audio-51-2018
>>>>>>> 
>>>>>>> I don't have a mac so I have no idea whether it works or not.
>>>>>>> 
>>>>>>> Regards,
>>>>>>> Zoltan
>>>>>>> 
>>>>>>> On 2018-12-26 12:24, Programmingkid wrote:
>>>>>>>> 
>>>>>>>>> On Dec 23, 2018, at 3:52 PM, qemu-devel-request@nongnu.org wrote:
>>>>>>>>> 
>>>>>>>>> Message: 4
>>>>>>>>> Date: Sun, 23 Dec 2018 21:51:36 +0100
>>>>>>>>> From: "=?UTF-8?q?K=C5=91v=C3=A1g=C3=B3=2C=20Zolt=C3=A1n?="
>>>>>>>>> 	<dirty.ice.hu@gmail.com>
>>>>>>>>> To: qemu-devel@nongnu.org
>>>>>>>>> Cc: Gerd Hoffmann <kraxel@redhat.com>
>>>>>>>>> Subject: [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches
>>>>>>>>> Message-ID: <cover.1545598229.git.DirtY.iCE.hu@gmail.com>
>>>>>>>>> Content-Type: text/plain; charset=UTF-8
>>>>>>>>> 
>>>>>>>>> Hi,
>>>>>>>>> 
>>>>>>>>> I've updated my audio patchset to the current git master. Other than that not
>>>>>>>>> much happened since my last update [1], fixed a few small problems that I
>>>>>>>>> noticed while rebasing my patches.
>>>>>>>>> 
>>>>>>>>> Please review.
>>>>>>>> 
>>>>>>>> <snip>
>>>>>>>> 
>>>>>>>> Hi I would like to run your patches. Do you have a repository that I may clone? 
>>>>>>>> 
>>>>>>>> Also have you been able to test these patches using a Mac OS X guest yet? 
>>>>>>>> 
>>>>>>>> Thank you.
>>>>>>>> 
>>>>>>> 
>>>>>> 
>>>>>> Hi, thanks for the link. This is what I did:
>>>>>> 
>>>>>> - git clone https://github.com/DirtYiCE/qemu.git
>>>>>> - git checkout audio-51-2018
>>>>>> - ./configure --target-list=ppc-softmmu
>>>>>> - make -j 4
>>>>>> 
>>>>>> The result was unfortunately some errors:
>>>>>> 
>>>>>> CC      audio/coreaudio.o
>>>>>> audio/coreaudio.c:413:49: error: unknown type name 'HWVocieOut'; did you mean
>>>>>>    'HWVoiceOut'?
>>>>>> COREAUDIO_WRAPPER_FUNC(get_buffer_out, void *, (HWVocieOut *hw, size_t *size),
>>>>>>                                              ^~~~~~~~~~
>>>>>>                                              HWVoiceOut
>>>>>> 
>>>>>> 
>>>>>> audio/coreaudio.c:578:29: error: passing 'struct audio_pcm_info' to parameter of
>>>>>>    incompatible type 'struct audio_pcm_info *'; take the address with &
>>>>>>      coreaudio_get_flags(hw->info, as);
>>>>>>                          ^~~~~~~~
>>>>>>                          &
>>>>>> 
>>>>>> 
>>>>> 
>>>>> Hi,
>>>>> 
>>>>> I pushed an updated version, it should fix the compile errors (hopefully).
>>>>> 
>>>>> Regards,
>>>>> Zoltan
>>>> 
>>>> Thank you for the update. I tested this series using a Mac OS 10.4 and Windows 2000 guest. Both were using a USB sound card. The sound that comes out of my speakers is demonic! It is the loudest, scariest sound I have ever heard. I'm sorry but this patch series ruins the USB sound card.
>>>> 
>>>> I am happy to test out any future updates that you make this patch series. 
>>>> 
>>>> Thank you.
>>>> 
>>> 
>>> I'm sorry to hear that. Could you please test it with some other card,
>>> like the hda? Even though usb works fine for me on Linux with alsa so
>>> the problem is probably with the coreaudio backend.
>>> 
>>> Regards,
>>> Zoltan
>> 
>> I tried ac97 with a Windows 2000 guest in qemu-system-i386 - same demonic sound.
>> With the above configuration but with an es1370 sound card I heard the same sound. I kept seeing "es1370: warning: non looping mode" being printed in the terminal. This problem might need its own patch.
>> Using the sb16 sound card I heard the same unsettling sound.
>> 
>> I could not test the HDA driver due to problems with my Windows 7 VM.  
>> 
> 
> 
> I have a feeling that bf870a0cf5e2c2dd7438e65473b4fca1fb0ca5d1
> (coreaudio: port to the new audio backend api) is the commit which
> breaks it, but without a mac I don't know how could I debug it. The way
> it initializes coreaudio changed a bit, that can cause problems.
> Could you please test whether this commit is the culprit or not?
> 
> Regards,
> Zoltan


Hi how do you want me to test your theory? Revert the patch? If I did would QEMU still compile? 

^ permalink raw reply	[flat|nested] 91+ messages in thread

end of thread, other threads:[~2019-01-16 22:40 UTC | newest]

Thread overview: 91+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-23 20:51 [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 01/52] qapi: support alternates in OptsVisitor Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 02/52] qapi: support nested structs " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 03/52] qapi: qapi for audio backends Kővágó, Zoltán
2019-01-10  2:49   ` Eric Blake
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 04/52] audio: use qapi AudioFormat instead of audfmt_e Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 05/52] audio: -audiodev command line option: documentation Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 06/52] audio: -audiodev command line option basic implementation Kővágó, Zoltán
2019-01-07 13:13   ` Markus Armbruster
2019-01-07 20:48     ` Zoltán Kővágó
2019-01-08  3:42       ` Markus Armbruster
2019-01-10  0:13         ` Zoltán Kővágó
2019-01-10  7:25           ` Gerd Hoffmann
2019-01-10  9:40             ` Zoltán Kővágó
2019-01-10 10:37               ` Gerd Hoffmann
2019-01-08  6:06       ` Gerd Hoffmann
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 07/52] alsaaudio: port to -audiodev config Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 08/52] coreaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 09/52] dsoundaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 10/52] noaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 11/52] ossaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 12/52] paaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 13/52] sdlaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 14/52] spiceaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 15/52] wavaudio: " Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 16/52] audio: -audiodev command line option: cleanup Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 17/52] audio: reduce glob_audio_state usage Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 18/52] audio: basic support for multi backend audio Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 19/52] audio: add audiodev properties to frontends Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 20/52] audio: audiodev= parameters no longer optional when -audiodev present Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 21/52] paaudio: do not move stream when sink/source name is specified Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 22/52] paaudio: properly disconnect streams in fini_* Kővágó, Zoltán
2018-12-23 20:51 ` [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX Kővágó, Zoltán
2018-12-23 23:49   ` Philippe Mathieu-Daudé
2018-12-24  2:16     ` Zoltán Kővágó
2018-12-24 17:16       ` Philippe Mathieu-Daudé
2018-12-24 20:48         ` Kővágó Zoltán
2018-12-25 10:40           ` Philippe Mathieu-Daudé
2018-12-27 12:49             ` Kővágó Zoltán
2019-01-07  9:54               ` Gerd Hoffmann
2019-01-07 14:26                 ` Eric Blake
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 24/52] audio: do not run each backend in audio_run Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 25/52] paaudio: fix playback glitches Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 26/52] audio: remove read and write pcm_ops Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 27/52] audio: use size_t where makes sense Kővágó, Zoltán
2018-12-24  6:19   ` Pavel Dovgalyuk
2018-12-25 11:08   ` Philippe Mathieu-Daudé
2019-01-07  9:58     ` Gerd Hoffmann
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 28/52] audio: api for mixeng code free backends Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 29/52] alsaaudio: port to the new audio backend api Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 30/52] coreaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 31/52] dsoundaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 32/52] noaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 33/52] ossaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 34/52] paaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 35/52] sdlaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 36/52] spiceaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 37/52] wavaudio: " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 38/52] audio: remove remains of the old " Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 39/52] audio: unify input and output mixeng buffer management Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 40/52] audio: remove hw->samples, buffer_size_in/out pcm_ops Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 41/52] audio: common rate control code for timer based outputs Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 42/52] audio: split ctl_* functions into enable_* and volume_* Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 43/52] audio: add mixeng option (documentation) Kővágó, Zoltán
2019-01-10  1:43   ` Eric Blake
2019-01-10  9:12     ` Markus Armbruster
2019-01-16 20:27     ` Zoltán Kővágó
2019-01-16 22:40       ` Eric Blake
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 44/52] audio: make mixeng optional Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 46/52] audio: support more than two channels in volume setting Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 47/52] audio: replace shift in audio_pcm_info with bytes_per_frame Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 48/52] audio: basic support for multichannel audio Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 49/52] paaudio: channel-map option Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 50/52] usb-audio: do not count on avail bytes actually available Kővágó, Zoltán
2018-12-23 20:52 ` [Qemu-devel] [PATCH v2 52/52] usbaudio: change playback counters to 64 bit Kővágó, Zoltán
2018-12-25 10:50   ` Philippe Mathieu-Daudé
2018-12-27 22:08     ` Kővágó Zoltán
2018-12-25 10:43 ` [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches Philippe Mathieu-Daudé
     [not found] <mailman.31873.1545598364.1282.qemu-devel@nongnu.org>
2018-12-26 11:24 ` Programmingkid
2018-12-27 13:33   ` Kővágó Zoltán
2018-12-28  0:46     ` Programmingkid
2018-12-28 20:05       ` Kővágó Zoltán
2018-12-29  0:12         ` Programmingkid
2018-12-29  0:19           ` Zoltán Kővágó
2018-12-29  0:49             ` Programmingkid
2018-12-31 15:50               ` Kővágó Zoltán
2018-12-27 22:16   ` Philippe Mathieu-Daudé
2019-01-01  4:20 [Qemu-devel] Qemu-devel Digest, Vol 189, Issue 424 Programmingkid
2019-01-01  4:24 ` [Qemu-devel] [PATCH v2 00/52] Audio 5.1 patches Programmingkid
2019-01-01 12:27   ` Zoltán Kővágó
2019-01-01 16:17     ` Programmingkid
2019-01-04 21:54       ` Zoltán Kővágó

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.