On 05/04/2016 23:41, Hugh Dickins wrote: > +/* > + * We are holding kvm->mmu_lock, serializing against mmu notifiers. > + * We have a ref on page. > ... > +static bool is_huge_tmpfs(struct kvm_vcpu *vcpu, > + unsigned long address, struct page *page)
vcpu is only used to access vcpu->kvm->mm. If it's still possible to give a sensible rule for locking, I wouldn't mind if is_huge_tmpfs took the mm directly and was moved out of KVM. Otherwise, it would be quite easy for people touch mm code to miss it. Apart from this, both patches look good. Paolo
