From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759403Ab2FHBF0 (ORCPT ); Thu, 7 Jun 2012 21:05:26 -0400 Received: from ud10.udmedia.de ([194.117.254.50]:52339 "EHLO mail.ud10.udmedia.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757319Ab2FHBFZ (ORCPT ); Thu, 7 Jun 2012 21:05:25 -0400 Date: Fri, 8 Jun 2012 03:05:20 +0200 From: Markus Trippelsdorf To: Linus Torvalds Cc: akpm@linux-foundation.org, Linux Kernel Mailing List , linux-mm , khlebnikov@openvz.org, hughd@google.com, kamezawa.hiroyu@jp.fujitsu.com, oleg@redhat.com, stable@vger.kernel.org Subject: Re: [patch 12/12] mm: correctly synchronize rss-counters at exit/exec Message-ID: <20120608010520.GA25317@x4> References: <20120607212114.E4F5AA02F8@akpm.mtv.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2012.06.07 at 17:25 -0700, Linus Torvalds wrote: > Ugh, looking more at the patch, I'm getting more and more convinces > that it is pure and utter garbage. > > It does "sync_mm_rss(mm);" in mmput(), _after_ it has done the > possibly final mmdrop(). WTF? > > This is crap, guys. Seriously. Stop playing russian rulette with this > code. I think we need to revert *all* of the crazy rss games, unless > Konstantin can show us some truly obviously correct fix. > > Sadly, I merged and pushed out the crap before I had rebooted and > noticed this problem, so now it's in the wild. Can somebody please > take a look at this asap? You've somehow merged the wrong patch. The correct version can be found here: http://marc.info/?l=linux-kernel&m=133848759505805 -- Markus