[PATCH] trace/control: Fix -Wunused-but-set-global warning with clang 23+

[PATCH] trace/control: Fix -Wunused-but-set-global warning with clang 23+

[Tom Stellard]

../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(+)

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

Re: [PATCH] trace/control: Fix -Wunused-but-set-global warning with clang 23+

[Daniel P. Berrangé]

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 :|