I wrote: > /* If not our own pmap, mark whatever's on the CPU now as lazy. */ > KASSERT(ci->ci_tlbstate == TLBSTATE_VALID); > >> mypmap = vm_map_pmap(&l->l_proc->p_vmspace->vm_map); > if (ci->ci_pmap == vm_map_pmap(&l->l_proc->p_vmspace->vm_map)) { > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > shouldn't the mypmap be used the line below instead of the second vm_map_pmap > call?
This is harmless as vm_map_pmap() is just an accessor macro and does not map anything. Martin