From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753823AbbFIUCG (ORCPT ); Tue, 9 Jun 2015 16:02:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49609 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753681AbbFIUBz (ORCPT ); Tue, 9 Jun 2015 16:01:55 -0400 Message-ID: <5577462A.40402@redhat.com> Date: Tue, 09 Jun 2015 16:01:46 -0400 From: Rik van Riel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Mel Gorman , Andrew Morton CC: Hugh Dickins , Minchan Kim , Dave Hansen , Andi Kleen , H Peter Anvin , Ingo Molnar , Linus Torvalds , Thomas Gleixner , Peter Zijlstra , Linux-MM , LKML Subject: Re: [PATCH 2/4] mm: Send one IPI per CPU to TLB flush all entries after unmapping pages References: <1433871118-15207-1-git-send-email-mgorman@suse.de> <1433871118-15207-3-git-send-email-mgorman@suse.de> In-Reply-To: <1433871118-15207-3-git-send-email-mgorman@suse.de> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/09/2015 01:31 PM, Mel Gorman wrote: > An IPI is sent to flush remote TLBs when a page is unmapped that was > potentially accesssed by other CPUs. There are many circumstances where > this happens but the obvious one is kswapd reclaiming pages belonging to > a running process as kswapd and the task are likely running on separate CPUs. > It's still a noticeable improvement with vmstat showing interrupts went > from roughly 500K per second to 45K per second. > > The patch will have no impact on workloads with no memory pressure or > have relatively few mapped pages. It will have an unpredictable impact > on the workload running on the CPU being flushed as it'll depend on how > many TLB entries need to be refilled and how long that takes. Worst case, > the TLB will be completely cleared of active entries when the target PFNs > were not resident at all. > > Signed-off-by: Mel Gorman Reviewed-by: Rik van Riel -- All rights reversed