Hi On Fri, Apr 18, 2025 at 3:30 PM Dietmar Maurer <diet...@proxmox.com> wrote: > > So that we can set --gst- options on the qemu command line. > > Signed-off-by: Dietmar Maurer <diet...@proxmox.com>
Nice, could you move that up in the patch series? As a second patch after linking gst seems fitting. > --- > system/vl.c | 8 ++++++++ > ui/vnc.c | 4 ---- > 2 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/system/vl.c b/system/vl.c > index ec93988a03..c7fff02da2 100644 > --- a/system/vl.c > +++ b/system/vl.c > @@ -140,6 +140,10 @@ > #include "qemu/guest-random.h" > #include "qemu/keyval.h" > > +#ifdef CONFIG_GSTREAMER > +#include <gst/gst.h> > +#endif > + > #define MAX_VIRTIO_CONSOLES 1 > > typedef struct BlockdevOptionsQueueEntry { > @@ -2848,6 +2852,10 @@ void qemu_init(int argc, char **argv) > bool userconfig = true; > FILE *vmstate_dump_file = NULL; > > +#ifdef CONFIG_GSTREAMER > + gst_init(&argc, &argv); > +#endif > + > qemu_add_opts(&qemu_drive_opts); > qemu_add_drive_opts(&qemu_legacy_drive_opts); > qemu_add_drive_opts(&qemu_common_drive_opts); > diff --git a/ui/vnc.c b/ui/vnc.c > index 6db03a1550..8f6287e2e6 100644 > --- a/ui/vnc.c > +++ b/ui/vnc.c > @@ -4350,10 +4350,6 @@ int vnc_init_func(void *opaque, QemuOpts *opts, Error > **errp) > Error *local_err = NULL; > char *id = (char *)qemu_opts_id(opts); > > -#ifdef CONFIG_GSTREAMER > - gst_init(NULL, NULL); > -#endif > - > assert(id); > vnc_display_init(id, &local_err); > if (local_err) { > -- > 2.39.5 >