From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752266AbeCZVaS (ORCPT ); Mon, 26 Mar 2018 17:30:18 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:47790 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751739AbeCZVaQ (ORCPT ); Mon, 26 Mar 2018 17:30:16 -0400 From: jglisse@redhat.com To: linux-mm@kvack.org Cc: Andrew Morton , linux-kernel@vger.kernel.org, Ralph Campbell , =?UTF-8?q?J=C3=A9r=C3=B4me=20Glisse?= , Evgeny Baskakov , Mark Hairgrove , John Hubbard Subject: [PATCH 1/2] mm/hmm: do not ignore specific pte fault flag in hmm_vma_fault() Date: Mon, 26 Mar 2018 17:30:08 -0400 Message-Id: <20180326213009.2460-2-jglisse@redhat.com> In-Reply-To: <20180326213009.2460-1-jglisse@redhat.com> References: <20180326213009.2460-1-jglisse@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Ralph Campbell Save requested fault flags from caller supplied pfns array before overwriting it with the special none value. Without this we would not fault on all cases requested by caller, leading to caller calling us in a loop unless something else did change the CPU page table. Signed-off-by: Ralph Campbell Signed-off-by: Jérôme Glisse Cc: Evgeny Baskakov Cc: Mark Hairgrove Cc: John Hubbard --- mm/hmm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/mm/hmm.c b/mm/hmm.c index e4742f6f1e05..ba912da1c1a1 100644 --- a/mm/hmm.c +++ b/mm/hmm.c @@ -498,10 +498,11 @@ static int hmm_vma_handle_pte(struct mm_walk *walk, unsigned long addr, bool fault, write_fault; uint64_t cpu_flags; pte_t pte = *ptep; + uint64_t orig_pfn = *pfn; *pfn = range->values[HMM_PFN_NONE]; cpu_flags = pte_to_hmm_pfn_flags(range, pte); - hmm_pte_need_fault(hmm_vma_walk, *pfn, cpu_flags, + hmm_pte_need_fault(hmm_vma_walk, orig_pfn, cpu_flags, &fault, &write_fault); if (pte_none(pte)) { @@ -528,7 +529,7 @@ static int hmm_vma_handle_pte(struct mm_walk *walk, unsigned long addr, range->flags[HMM_PFN_DEVICE_PRIVATE]; cpu_flags |= is_write_device_private_entry(entry) ? range->flags[HMM_PFN_WRITE] : 0; - hmm_pte_need_fault(hmm_vma_walk, *pfn, cpu_flags, + hmm_pte_need_fault(hmm_vma_walk, orig_pfn, cpu_flags, &fault, &write_fault); if (fault || write_fault) goto fault; -- 2.14.3