From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750949Ab2DGEWF (ORCPT ); Sat, 7 Apr 2012 00:22:05 -0400 Received: from mail-iy0-f174.google.com ([209.85.210.174]:63643 "EHLO mail-iy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750766Ab2DGEV4 (ORCPT ); Sat, 7 Apr 2012 00:21:56 -0400 Date: Fri, 6 Apr 2012 21:21:38 -0700 (PDT) From: Hugh Dickins X-X-Sender: hugh@eggly.anvils To: Konstantin Khlebnikov cc: Oleg Nesterov , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Roland Dreier , Andrew Morton , Linus Torvalds Subject: Re: [PATCH RFC] mm: account VMA before forced-COW via /proc/pid/mem In-Reply-To: <4F7D5859.5050106@openvz.org> Message-ID: References: <20120402153631.5101.44091.stgit@zurg> <20120403143752.GA5150@redhat.com> <4F7C1B67.6030300@openvz.org> <20120404154148.GA7105@redhat.com> <4F7D5859.5050106@openvz.org> User-Agent: Alpine 2.00 (LSU 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 5 Apr 2012, Konstantin Khlebnikov wrote: > Oleg Nesterov wrote: > > On 04/04, Konstantin Khlebnikov wrote: > > > Oleg Nesterov wrote: > > > > On 04/02, Konstantin Khlebnikov wrote: > > > > > > > > > > Currently kernel does not account read-only private mappings into > > > > > memory commitment. > > > > > But these mappings can be force-COW-ed in get_user_pages(). > > > > > > > > Heh. tail -n3 Documentation/vm/overcommit-accounting > > > > may be you should update it then. > > > > > > I just wonder how fragile this accounting... > > > > I meant, this patch could also remove this "TODO" from the docs. > > Actually I dug into this code for killing VM_ACCOUNT vma flag. > Currently we cannot do this only because asymmetry in mprotect_fixup(): > it account vma on read-only -> writable conversion, but keep on backward > operation. > Probably we can kill this asymmetry, and after that we can recognize > accountable vma > by its others flags state, so we don't need special VM_ACCOUNT for this. (I believe the VM_ACCOUNT flag will need to stay.) But this is just a quick note to say that I'm not ignoring you: I have a strong interest in this, but only now found time to look through the thread and ponder, and I'm not yet ready to decide. I've long detested that behaviour of GUP write,force, and my strong preference would be not to layer more strangeness upon strangeness, but limit the damage by making GUP write,force fail in that case, instead of inserting a PageAnon page into a VM_SHARED mapping. I think it's unlikely that it will cause a regression in real life (it already fails if you did not open the mmap'ed file for writing), but it would be a user-visible change in behaviour, and I've research to do before arriving at a conclusion. Hugh