Re: [PATCH] KVM: arm64: avoid unused-variable warning

[Vincent Donnefort]

On Fri, Mar 13, 2026 at 10:49:16AM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@xxxxxxxx>
> 
> The 'cpu' variable is only used inside of an #ifdef block and causes
> a warning if there is no user:
> 
> arch/arm64/kvm/hyp_trace.c: In function 'kvm_hyp_trace_init':
> arch/arm64/kvm/hyp_trace.c:422:13: error: unused variable 'cpu' [-Werror=unused-variable]
>   422 |         int cpu;
>       |             ^~~
> 
> Change the #ifdef to an equivalent IS_ENABLED() check to avoid
> the warning.
> 
> Fixes: b22888917fa4 ("KVM: arm64: Sync boot clock with the nVHE/pKVM hyp")
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> ---
>  arch/arm64/kvm/hyp_trace.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/kvm/hyp_trace.c b/arch/arm64/kvm/hyp_trace.c
> index c1e28f6581ab..8b7f2bf2fba8 100644
> --- a/arch/arm64/kvm/hyp_trace.c
> +++ b/arch/arm64/kvm/hyp_trace.c
> @@ -424,17 +424,16 @@ int __init kvm_hyp_trace_init(void)
>  	if (is_kernel_in_hyp_mode())
>  		return 0;
>  
> -#ifdef CONFIG_ARM_ARCH_TIMER_OOL_WORKAROUND
>  	for_each_possible_cpu(cpu) {
>  		const struct arch_timer_erratum_workaround *wa =
>  			per_cpu(timer_unstable_counter_workaround, cpu);
>  
> -		if (wa && wa->read_cntvct_el0) {
> +		if (IS_ENABLED(CONFIG_ARM_ARCH_TIMER_OOL_WORKAROUND) &&
> +		    wa && wa->read_cntvct_el0) {
>  			pr_warn("hyp trace can't handle CNTVCT workaround '%s'\n", wa->desc);
>  			return -EOPNOTSUPP;
>  		}
>  	}
> -#endif
>  
>  	hyp_trace_init_events();

Reviewed-by: Vincent Donnefort <vdonnefort@xxxxxxxxxx>

>  
> -- 
> 2.39.5
>