[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/18] ui/vnc: Require audiodev=
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 06/18] ui/vnc: Require audiodev= |
Date: |
Wed, 27 Apr 2022 10:41:15 +0100 |
User-agent: |
Mutt/2.1.5 (2021-12-30) |
On Wed, Apr 27, 2022 at 11:32:41AM +0200, Paolo Bonzini wrote:
> On 4/25/22 10:21, Martin Kletzander wrote:
> > @@ -4188,12 +4188,15 @@ void vnc_display_open(const char *id, Error **errp)
> > vd->ledstate = 0;
> > audiodev = qemu_opt_get(opts, "audiodev");
> > - if (audiodev) {
> > - vd->audio_state = audio_state_by_name(audiodev);
> > - if (!vd->audio_state) {
> > - error_setg(errp, "Audiodev '%s' not found", audiodev);
> > - goto fail;
> > - }
> > + if (!audiodev) {
> > + error_setg(errp, "Audiodev parameter for vnc required");
> > + goto fail;
> > + }
> > +
>
> Wouldn't this break "-vnc :0"? You can just ignore the audio commands if
> vd->audio_state is NULL.
Yep, that's wha I suggested with skipping advertizing VNC_ENCODING_AUDIO
when audiodev is NULL
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH 13/18] audio: Make AUD_register_card fallible and require audiodev=, (continued)
- [PATCH 09/18] hw/display/xlnx_dp.c: Add audiodev property, Martin Kletzander, 2022/04/25
- [PATCH 04/18] hw/audio/lm4549: Add errp error reporting to init function, Martin Kletzander, 2022/04/25
- [PATCH 12/18] hw/ppc: Support machine-default audiodev with fallback, Martin Kletzander, 2022/04/25
- [PATCH 02/18] hw/input/tsc210x: Extract common init code into new function, Martin Kletzander, 2022/04/25
- [PATCH 06/18] ui/vnc: Require audiodev=, Martin Kletzander, 2022/04/25
- [PATCH 15/18] audio: Be more strict during audio backend initialisation, Martin Kletzander, 2022/04/25
- [PATCH 03/18] hw/audio: Simplify hda audio init, Martin Kletzander, 2022/04/25
- [PATCH 01/18] hw/audio: Remove -soundhw support, Martin Kletzander, 2022/04/25
- [PATCH 10/18] hw/input/tsc210x.c: Support machine-default audiodev with fallback, Martin Kletzander, 2022/04/25