Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965926Ab3E2M4l (ORCPT ); Wed, 29 May 2013 08:56:41 -0400 Received: from us01smtp3.synopsys.com ([198.182.44.81]:37023 "EHLO hermes.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965690Ab3E2M4j (ORCPT ); Wed, 29 May 2013 08:56:39 -0400 From: Vineet Gupta To: , CC: Vineet Gupta , Andrew Morton , Mel Gorman , Hugh Dickins , Rik van Riel , David Rientjes , Peter Zijlstra , "linux-arch@vger.kernel.org" , Catalin Marinas , Max Filippov Subject: [PATCH] mm: Fix the TLB range flushed when __tlb_remove_page() runs out of slots Date: Wed, 29 May 2013 18:26:13 +0530 Message-ID: <1369832173-15088-1-git-send-email-vgupta@synopsys.com> X-Mailer: git-send-email 1.7.10.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.12.197.232] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1810 Lines: 62 zap_pte_range loops from @addr to @end. In the middle, if it runs out of batching slots, TLB entries needs to be flushed for @start to @interim, NOT @interim to @end. Since ARC port doesn't use page free batching I can't test it myself but this seems like the right thing to do. Observed this when working on a fix for the issue at thread: http://www.spinics.net/lists/linux-arch/msg21736.html Signed-off-by: Vineet Gupta Cc: Andrew Morton Cc: Mel Gorman Cc: Hugh Dickins Cc: Rik van Riel Cc: David Rientjes Cc: Peter Zijlstra Cc: linux-mm@kvack.org Cc: linux-arch@vger.kernel.org Cc: Catalin Marinas Cc: Max Filippov --- mm/memory.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/mm/memory.c b/mm/memory.c index 6dc1882..d9d5fd9 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -1110,6 +1110,7 @@ static unsigned long zap_pte_range(struct mmu_gather *tlb, spinlock_t *ptl; pte_t *start_pte; pte_t *pte; + unsigned long range_start = addr; again: init_rss_vec(rss); @@ -1215,12 +1216,14 @@ again: force_flush = 0; #ifdef HAVE_GENERIC_MMU_GATHER - tlb->start = addr; - tlb->end = end; + tlb->start = range_start; + tlb->end = addr; #endif tlb_flush_mmu(tlb); - if (addr != end) + if (addr != end) { + range_start = addr; goto again; + } } return addr; -- 1.7.10.4 -- 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/