On Mon, Dec 21, 2020, Paolo Bonzini wrote:
> On 18/12/20 10:10, Vitaly Kuznetsov wrote:
> > > - int root = vcpu->arch.mmu->shadow_root_level;
> > > - int leaf;
> > > - int level;
> > > + int root, leaf, level;
> > > bool reserved = false;
> > Personal taste: I would've renamed 'root' to 'root_level' (to be
> > consistent with get_walk()/kvm_tdp_mmu_get_walk()) and 'level' to
> > e.g. 'l' as it's only being used as an interator ('i' would also do).
>
> Maybe agree on the former, not really on the latter. :)
Same here. I kept 'root' to reduce code churn, even though I'd probably have
used 'root_level' if I were writing from scratch.
