Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755373Ab3GQAdP (ORCPT ); Tue, 16 Jul 2013 20:33:15 -0400 Received: from mail-oa0-f48.google.com ([209.85.219.48]:44198 "EHLO mail-oa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755256Ab3GQAdO (ORCPT ); Tue, 16 Jul 2013 20:33:14 -0400 MIME-Version: 1.0 In-Reply-To: <1373901620-2021-4-git-send-email-mgorman@suse.de> References: <1373901620-2021-1-git-send-email-mgorman@suse.de> <1373901620-2021-4-git-send-email-mgorman@suse.de> Date: Wed, 17 Jul 2013 08:33:13 +0800 Message-ID: Subject: Re: [PATCH 03/18] mm: numa: Account for THP numa hinting faults on the correct node From: Hillf Danton To: Mel Gorman Cc: Peter Zijlstra , Srikar Dronamraju , Ingo Molnar , Andrea Arcangeli , Johannes Weiner , Linux-MM , LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2102 Lines: 53 On Mon, Jul 15, 2013 at 11:20 PM, Mel Gorman wrote: > THP NUMA hinting fault on pages that are not migrated are being > accounted for incorrectly. Currently the fault will be counted as if the > task was running on a node local to the page which is not necessarily > true. > Can you please run test again without this correction and check the difference? > Signed-off-by: Mel Gorman > --- > mm/huge_memory.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/mm/huge_memory.c b/mm/huge_memory.c > index e2f7f5aa..e4a79fa 100644 > --- a/mm/huge_memory.c > +++ b/mm/huge_memory.c > @@ -1293,7 +1293,7 @@ int do_huge_pmd_numa_page(struct mm_struct *mm, struct vm_area_struct *vma, > struct page *page; > unsigned long haddr = addr & HPAGE_PMD_MASK; > int target_nid; > - int current_nid = -1; > + int src_nid = -1; > bool migrated; > > spin_lock(&mm->page_table_lock); > @@ -1302,9 +1302,9 @@ int do_huge_pmd_numa_page(struct mm_struct *mm, struct vm_area_struct *vma, > > page = pmd_page(pmd); > get_page(page); > - current_nid = page_to_nid(page); > + src_nid = numa_node_id(); > count_vm_numa_event(NUMA_HINT_FAULTS); > - if (current_nid == numa_node_id()) > + if (src_nid == page_to_nid(page)) > count_vm_numa_event(NUMA_HINT_FAULTS_LOCAL); > > target_nid = mpol_misplaced(page, vma, haddr); > @@ -1346,8 +1346,8 @@ clear_pmdnuma: > update_mmu_cache_pmd(vma, addr, pmdp); > out_unlock: > spin_unlock(&mm->page_table_lock); > - if (current_nid != -1) > - task_numa_fault(current_nid, HPAGE_PMD_NR, false); > + if (src_nid != -1) > + task_numa_fault(src_nid, HPAGE_PMD_NR, false); > return 0; > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/