From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751431AbdEaQTp (ORCPT ); Wed, 31 May 2017 12:19:45 -0400 Received: from mx2.suse.de ([195.135.220.15]:54874 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751357AbdEaQTm (ORCPT ); Wed, 31 May 2017 12:19:42 -0400 Date: Wed, 31 May 2017 09:19:19 -0700 From: Davidlohr Bueso To: Laurent Dufour Cc: linux-mm@kvack.org, akpm@linux-foundation.org, Jan Kara , "Kirill A . Shutemov" , Michal Hocko , Peter Zijlstra , Mel Gorman , Andi Kleen , haren@linux.vnet.ibm.com, aneesh.kumar@linux.vnet.ibm.com, khandual@linux.vnet.ibm.com, paulmck@linux.vnet.ibm.com, linux-kernel@vger.kernel.org Subject: Re: [RFC v2 10/10] mm: Introduce CONFIG_MEM_RANGE_LOCK Message-ID: <20170531161919.GC28615@linux-80c1.suse> Mail-Followup-To: Laurent Dufour , linux-mm@kvack.org, akpm@linux-foundation.org, Jan Kara , "Kirill A . Shutemov" , Michal Hocko , Peter Zijlstra , Mel Gorman , Andi Kleen , haren@linux.vnet.ibm.com, aneesh.kumar@linux.vnet.ibm.com, khandual@linux.vnet.ibm.com, paulmck@linux.vnet.ibm.com, linux-kernel@vger.kernel.org References: <1495624801-8063-1-git-send-email-ldufour@linux.vnet.ibm.com> <1495624801-8063-11-git-send-email-ldufour@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <1495624801-8063-11-git-send-email-ldufour@linux.vnet.ibm.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 24 May 2017, Laurent Dufour wrote: >A new configuration variable is introduced to activate the use of >range lock instead of semaphore to protect per process memory layout. > >This range lock is replacing the use of a semaphore for mmap_sem. > >Currently only available for X86_64 and PPC64 architectures. > >By default this option is turned off and requires the EXPERT mode >since it is not yet complete. Just fyi I find this option quite useful for dev and debugging purposes, however it should not exist once any of this is seriously considered for merging. The reason being is that fundamentally such internals should not be exposed to configuration options. We either get it right for everybody, or we don't. I'm currently running lots of tests to see the overhead in real workloads on different boxes. While I hope that my artificial testing somewhat resembles some patterns, this will be the real deal. Thanks, Davidlohr