> > > > +static void reset_id_reg(struct kvm_vcpu *vcpu, const struct sys_reg_desc *rd) > > > > > > Since not all ID registers will use this, then maybe name it > > > reset_sanitised_id_reg? > > > > Thank you for the suggestion. > > > > I named it 'reset_id_reg' according to the naming conventions of > > set_id_reg, get_id_reg, and access_id_reg which are used for the same > > set of ID registers (ID_SANITISED ones) as reset_id_reg. > > I would think it's better to use consistent names for all of them. > > So, I am a bit reluctant to change only the name of reset_id_reg. > > > > What do you think about the names of those other three functions ? > > I think I like the shorter names, so please disregard my suggestion. Thank you for the confirmation. I will keep those names as they are. Regards, Reiji