On Thu, Jun 18, 2026 at 08:55:07AM +0100, Daniel P. Berrangé wrote:
> CC trace maintainer.
> 
> On Wed, Jun 17, 2026 at 03:41:45PM -0700, Tom Stellard wrote:
> > ../trace/control.c:41:13: error: variable 'init_trace_on_startup' set but not used [-Werror,-Wunused-but-set-global]
> >    41 | static bool init_trace_on_startup;
> >       |             ^
> > 1 error generated.
> > 
> > Signed-off-by: Tom Stellard <tstellar@redhat.com>
> > ---
> >  trace/control.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> 
> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> 
> > 
> > diff --git a/trace/control.c b/trace/control.c
> > index 1c8c50064a..49f0a4c5cd 100644
> > --- a/trace/control.c
> > +++ b/trace/control.c
> > @@ -38,7 +38,9 @@ static TraceEventGroup *event_groups;
> >  static size_t nevent_groups;
> >  static uint32_t next_id;
> >  static uint32_t next_vcpu_id;
> > +#ifdef CONFIG_TRACE_SIMPLE
> >  static bool init_trace_on_startup;
> > +#endif
> >  static char *trace_opts_file;
> >  
> >  QemuOptsList qemu_trace_opts = {
> > @@ -295,7 +297,9 @@ void trace_opt_parse(const char *optstr)
> >          trace_enable_events(qemu_opt_get(opts, "enable"));
> >      }
> >      trace_init_events(qemu_opt_get(opts, "events"));
> > +#ifdef CONFIG_TRACE_SIMPLE
> >      init_trace_on_startup = true;
> > +#endif
> >      g_free(trace_opts_file);
> >      trace_opts_file = g_strdup(qemu_opt_get(opts, "file"));
> >      qemu_opts_del(opts);
> > -- 
> > 2.51.0
> > 
> > 
> 
> With regards,
> Daniel
> -- 
> |: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
> |: https://libvirt.org          ~~          https://entangle-photo.org :|
> |: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|
> 

Thanks, applied to my tracing tree:
https://gitlab.com/stefanha/qemu/commits/tracing

Stefan