On 25/10/19 04:01, Miaohe Lin wrote: > - if (r < 0) > - goto out; > - changed = memcmp(pdpte, vcpu->arch.walk_mmu->pdptrs, sizeof(pdpte)) != 0; > -out: > + if (r >= 0) > + changed = memcmp(pdpte, vcpu->arch.walk_mmu->pdptrs, > + sizeof(pdpte)) != 0; > > return changed; Even better: if (r < 0) return true; return memcmp(...) != 0; Paolo