From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755458AbbA0KlS (ORCPT ); Tue, 27 Jan 2015 05:41:18 -0500 Received: from cantor2.suse.de ([195.135.220.15]:44792 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751626AbbA0KlP (ORCPT ); Tue, 27 Jan 2015 05:41:15 -0500 Date: Tue, 27 Jan 2015 11:41:12 +0100 From: Michal Hocko To: Andrew Morton Cc: Christoph Lameter , Vinayak Menon , linux-mm@kvack.org, linux-kernel@vger.kernel.org, hannes@cmpxchg.org, vdavydov@parallels.com, mgorman@suse.de, minchan@kernel.org Subject: Re: [PATCH v2] mm: vmscan: fix the page state calculation in too_many_isolated Message-ID: <20150127104112.GA19880@dhcp22.suse.cz> References: <1421235419-30736-1-git-send-email-vinmenon@codeaurora.org> <20150114165036.GI4706@dhcp22.suse.cz> <54B7F7C4.2070105@codeaurora.org> <20150116154922.GB4650@dhcp22.suse.cz> <54BA7D3A.40100@codeaurora.org> <20150126172832.GC22681@dhcp22.suse.cz> <20150126141159.cfb8357352e044f5d6f66369@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150126141159.cfb8357352e044f5d6f66369@linux-foundation.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon 26-01-15 14:11:59, Andrew Morton wrote: [...] > So do we drop > mm-vmscan-fix-the-page-state-calculation-in-too_many_isolated.patch? I would like to see a confirmation from Vinayak that this really helped in his test case first and only then drop the above patch. I really think that we shouldn't spread hacks around the code just workaround inefficiency in the vmstat code. We already have two places which need a special treatment and who knows how many more will show up later. Even with this patch applied we have other issues related to the overloaded workqueues as mentioned earlier but those should be fixed by a separate fixe(s). -- Michal Hocko SUSE Labs