Re: [PATCH v4 06/18] x86/apic: Add update_vector callback for Secure AVIC

From: Thomas Gleixner
Date: Thu Apr 17 2025 - 06:52:39 EST


On Thu, Apr 17 2025 at 14:46, Neeraj Upadhyay wrote:
> +
> +static inline void update_vector(unsigned int cpu, unsigned int offset,
> + unsigned int vector, bool set)
> +{
> + unsigned long *reg = get_reg_bitmap(cpu, offset);
> + unsigned int bit = get_vec_bit(vector);
> +
> + if (set)
> + set_bit(bit, reg);
> + else
> + clear_bit(bit, reg);
> +}

> +static void savic_update_vector(unsigned int cpu, unsigned int vector, bool set)
> +{
> + update_vector(cpu, SAVIC_ALLOWED_IRR, vector, set);

This indirection is required because otherwise the code is too simple to
follow?