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?