Re: [Patch v2 3/5] KVM: x86/mmu: Optimize SPTE change for aging gfn range

From: Vipin Sharma
Date: Tue Feb 07 2023 - 12:49:09 EST


On Mon, Feb 6, 2023 at 2:17 PM Ben Gardon <bgardon@xxxxxxxxxx> wrote:
>
> On Fri, Feb 3, 2023 at 11:28 AM Vipin Sharma <vipinsh@xxxxxxxxxx> wrote:

> > - * __tdp_mmu_set_spte - Set a TDP MMU SPTE and handle the associated bookkeeping
> > + * _tdp_mmu_set_spte - Set a TDP MMU SPTE and handle the associated bookkeeping
>
> Nit: Not sure how we got to the point of having a single and double
> underscore of the function, but what do you think about just calling
> this one tdp_mmu_set_spte and the other tdp_mmu_iter_set_spte?

I like your idea. I also don't like the single and double underscore
prefix on functions.