From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleg Nesterov Subject: Re: [PATCH 09/14] resource limits: track highwater mark of locked memory Date: Fri, 15 Jul 2016 17:14:08 +0200 Message-ID: <20160715151408.GA32317@redhat.com> References: <1468578983-28229-1-git-send-email-toiwoton@gmail.com> <1468578983-28229-10-git-send-email-toiwoton@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1468578983-28229-10-git-send-email-toiwoton@gmail.com> Sender: owner-linux-mm@kvack.org To: Topi Miettinen Cc: linux-kernel@vger.kernel.org, Tony Luck , Fenghua Yu , Paolo Bonzini , Radim =?utf-8?B?S3LEjW3DocWZ?= , Alexander Graf , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Doug Ledford , Sean Hefty , Hal Rosenstock , Mike Marciniszyn , Dennis Dalessandro , Christian Benvenuti , Dave Goodell , Sudeep Dutt , Ashutosh Dixit , Alex Williamson , Alexei List-Id: linux-rdma@vger.kernel.org On 07/15, Topi Miettinen wrote: > > Track maximum size of locked memory, to be able to configure > RLIMIT_MEMLOCK resource limits. The information is available > with taskstats and cgroupstats netlink socket. So I personally still dislike the very idea of this series... but I won't argue if you convince maintainers. > @@ -2020,6 +2020,10 @@ static int acct_stack_growth(struct vm_area_struct *vma, unsigned long size, uns > return -ENOMEM; > > update_resource_highwatermark(RLIMIT_STACK, actual_size); > + if (vma->vm_flags & VM_LOCKED) > + update_resource_highwatermark(RLIMIT_MEMLOCK, > + (mm->locked_vm + grow) << > + PAGE_SHIFT); Btw this is not right. The same for the previous patch which tracks RLIMIT_STACK. The "current" task can debugger/etc. Yes, yes, this just reminds that the whole rlimit logic in this path is broken but still... Oleg. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vk0-f71.google.com (mail-vk0-f71.google.com [209.85.213.71]) by kanga.kvack.org (Postfix) with ESMTP id C70226B0005 for ; Fri, 15 Jul 2016 11:14:00 -0400 (EDT) Received: by mail-vk0-f71.google.com with SMTP id w127so4112288vkh.1 for ; Fri, 15 Jul 2016 08:14:00 -0700 (PDT) Received: from mx1.redhat.com (mx1.redhat.com. [209.132.183.28]) by mx.google.com with ESMTPS id b34si5535143qtb.136.2016.07.15.08.13.59 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 15 Jul 2016 08:14:00 -0700 (PDT) Date: Fri, 15 Jul 2016 17:14:08 +0200 From: Oleg Nesterov Subject: Re: [PATCH 09/14] resource limits: track highwater mark of locked memory Message-ID: <20160715151408.GA32317@redhat.com> References: <1468578983-28229-1-git-send-email-toiwoton@gmail.com> <1468578983-28229-10-git-send-email-toiwoton@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1468578983-28229-10-git-send-email-toiwoton@gmail.com> Sender: owner-linux-mm@kvack.org List-ID: To: Topi Miettinen Cc: linux-kernel@vger.kernel.org, Tony Luck , Fenghua Yu , Paolo Bonzini , Radim =?utf-8?B?S3LEjW3DocWZ?= , Alexander Graf , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Doug Ledford , Sean Hefty , Hal Rosenstock , Mike Marciniszyn , Dennis Dalessandro , Christian Benvenuti , Dave Goodell , Sudeep Dutt , Ashutosh Dixit , Alex Williamson , Alexei Starovoitov , Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Alexander Shishkin , Andrew Morton , Jiri Slaby , Konstantin Khlebnikov , Cyrill Gorcunov , Dave Hansen , Thomas Gleixner , Greg Kroah-Hartman , Dan Carpenter , Vlastimil Babka , Michal Hocko , "Kirill A. Shutemov" , Eric B Munson , Alexey Klimov , Andrea Arcangeli , Chen Gang , Andrey Ryabinin , David Rientjes , Hugh Dickins , Alexander Kuleshov , "open list:IA64 (Itanium) PLATFORM" , "open list:KERNEL VIRTUAL MACHINE (KVM)" , "open list:KERNEL VIRTUAL MACHINE (KVM) FOR POWERPC" , "open list:LINUX FOR POWERPC (32-BIT AND 64-BIT)" , "open list:INFINIBAND SUBSYSTEM" , "open list:BPF (Safe dynamic programs and tools)" , "open list:MEMORY MANAGEMENT" On 07/15, Topi Miettinen wrote: > > Track maximum size of locked memory, to be able to configure > RLIMIT_MEMLOCK resource limits. The information is available > with taskstats and cgroupstats netlink socket. So I personally still dislike the very idea of this series... but I won't argue if you convince maintainers. > @@ -2020,6 +2020,10 @@ static int acct_stack_growth(struct vm_area_struct *vma, unsigned long size, uns > return -ENOMEM; > > update_resource_highwatermark(RLIMIT_STACK, actual_size); > + if (vma->vm_flags & VM_LOCKED) > + update_resource_highwatermark(RLIMIT_MEMLOCK, > + (mm->locked_vm + grow) << > + PAGE_SHIFT); Btw this is not right. The same for the previous patch which tracks RLIMIT_STACK. The "current" task can debugger/etc. Yes, yes, this just reminds that the whole rlimit logic in this path is broken but still... Oleg. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3rrbgs0p3kzDqF6 for ; Sat, 16 Jul 2016 01:14:01 +1000 (AEST) Date: Fri, 15 Jul 2016 17:14:08 +0200 From: Oleg Nesterov To: Topi Miettinen Cc: linux-kernel@vger.kernel.org, Tony Luck , Fenghua Yu , Paolo Bonzini , Radim =?utf-8?B?S3LEjW3DocWZ?= , Alexander Graf , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Doug Ledford , Sean Hefty , Hal Rosenstock , Mike Marciniszyn , Dennis Dalessandro , Christian Benvenuti , Dave Goodell , Sudeep Dutt , Ashutosh Dixit , Alex Williamson , Alexei Starovoitov , Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Alexander Shishkin , Andrew Morton , Jiri Slaby , Konstantin Khlebnikov , Cyrill Gorcunov , Dave Hansen , Thomas Gleixner , Greg Kroah-Hartman , Dan Carpenter , Vlastimil Babka , Michal Hocko , "Kirill A. Shutemov" , Eric B Munson , Alexey Klimov , Andrea Arcangeli , Chen Gang , Andrey Ryabinin , David Rientjes , Hugh Dickins , Alexander Kuleshov , "open list:IA64 (Itanium) PLATFORM" , "open list:KERNEL VIRTUAL MACHINE (KVM)" , "open list:KERNEL VIRTUAL MACHINE (KVM) FOR POWERPC" , "open list:LINUX FOR POWERPC (32-BIT AND 64-BIT)" , "open list:INFINIBAND SUBSYSTEM" , "open list:BPF (Safe dynamic programs and tools)" , "open list:MEMORY MANAGEMENT" Subject: Re: [PATCH 09/14] resource limits: track highwater mark of locked memory Message-ID: <20160715151408.GA32317@redhat.com> References: <1468578983-28229-1-git-send-email-toiwoton@gmail.com> <1468578983-28229-10-git-send-email-toiwoton@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1468578983-28229-10-git-send-email-toiwoton@gmail.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 07/15, Topi Miettinen wrote: > > Track maximum size of locked memory, to be able to configure > RLIMIT_MEMLOCK resource limits. The information is available > with taskstats and cgroupstats netlink socket. So I personally still dislike the very idea of this series... but I won't argue if you convince maintainers. > @@ -2020,6 +2020,10 @@ static int acct_stack_growth(struct vm_area_struct *vma, unsigned long size, uns > return -ENOMEM; > > update_resource_highwatermark(RLIMIT_STACK, actual_size); > + if (vma->vm_flags & VM_LOCKED) > + update_resource_highwatermark(RLIMIT_MEMLOCK, > + (mm->locked_vm + grow) << > + PAGE_SHIFT); Btw this is not right. The same for the previous patch which tracks RLIMIT_STACK. The "current" task can debugger/etc. Yes, yes, this just reminds that the whole rlimit logic in this path is broken but still... Oleg.