From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759168AbaCSLEk (ORCPT ); Wed, 19 Mar 2014 07:04:40 -0400 Received: from cantor2.suse.de ([195.135.220.15]:56856 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759105AbaCSLEj (ORCPT ); Wed, 19 Mar 2014 07:04:39 -0400 Date: Wed, 19 Mar 2014 12:04:36 +0100 From: Jan Kara To: Hugh Dickins Cc: Linus Torvalds , Dave Jones , Cyrill Gorcunov , Sasha Levin , Andrew Morton , Linux Kernel , linux-mm , Joonsoo Kim , Bob Liu , Konstantin Khlebnikov Subject: Re: bad rss-counter message in 3.14rc5 Message-ID: <20140319110436.GF26358@quack.suse.cz> References: <20140311142817.GA26517@redhat.com> <20140311143750.GE32390@moon> <20140311171045.GA4693@redhat.com> <20140311173603.GG32390@moon> <20140311173917.GB4693@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue 18-03-14 19:37:01, Hugh Dickins wrote: > On Tue, 18 Mar 2014, Linus Torvalds wrote: > > On Tue, Mar 18, 2014 at 7:06 PM, Hugh Dickins wrote: > > > > > > I'd love that, if we can get away with it now: depends very > > > much on whether we then turn out to break userspace or not. > > > > Right. I suspect we can, though, but it's one of those "we can try it > > and see". Remind me early in the 3.15 merge window, and we can just > > turn the "force" case into an error case and see if anybody hollers. > > Super, I'll do that, thanks. > > For 3.15, and probably 3.16 too, we should keep in place whatever > partial accommodations we have for the case (such as allowing for > anon and swap in fremap's zap_pte), in case we do need to revert; > but clean those away later on. (Not many, I think: it was mainly > a guilty secret that VM accounting didn't really hold together.) Different drivers actually use the 'force' argument of get_user_pages() a lot on userspace provided buffers (AFAIU because they want to tell the kernel HW is going to write to that memory so they want to prepare for it). It is hard to imagine someone will use this for MAP_SHARED pages (or what that would be supposed to achieve) but sometimes userspace is surprisingly inventive... Just something to be aware of... Honza -- Jan Kara SUSE Labs, CR