On 10/6/18 2:45 PM, Emilio G. Cota wrote: > @@ -753,6 +763,7 @@ void tlb_set_page_with_attrs(CPUState *cpu, target_ulong > vaddr, > } > > copy_tlb_helper_locked(te, &tn); > + env->tlb_desc[mmu_idx].used++; > qemu_spin_unlock(&env->tlb_lock); > }
So.. you're computing what? Total TLB occupancy? Perhaps a better name than "used"? Otherwise the patch looks plausible. r~