Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754723AbbKXRCn (ORCPT ); Tue, 24 Nov 2015 12:02:43 -0500 Received: from mail-wm0-f41.google.com ([74.125.82.41]:33342 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753609AbbKXRCm (ORCPT ); Tue, 24 Nov 2015 12:02:42 -0500 Date: Tue, 24 Nov 2015 18:02:39 +0100 From: Michal Hocko To: Johannes Weiner Cc: David Rientjes , Vlastimil Babka , Andrew Morton , Andrea Arcangeli , Mel Gorman , linux-mm@kvack.org, LKML Subject: Re: [PATCH] mm, oom: Give __GFP_NOFAIL allocations access to memory reserves Message-ID: <20151124170239.GA13492@dhcp22.suse.cz> References: <1447249697-13380-1-git-send-email-mhocko@kernel.org> <5651BB43.8030102@suse.cz> <20151123092925.GB21050@dhcp22.suse.cz> <5652DFCE.3010201@suse.cz> <20151123101345.GF21050@dhcp22.suse.cz> <20151124094708.GA29472@dhcp22.suse.cz> <20151124162604.GB9598@cmpxchg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151124162604.GB9598@cmpxchg.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1995 Lines: 42 On Tue 24-11-15 11:26:04, Johannes Weiner wrote: > On Tue, Nov 24, 2015 at 10:47:09AM +0100, Michal Hocko wrote: > > Besides that there is no other reliable warning that we are getting > > _really_ short on memory unlike when the allocation failure is > > allowed. OOM killer report might be missing because there was no actual > > killing happening. > > This is why I would like to see that warning generalized, and not just > for __GFP_NOFAIL. We have allocations other than explicit __GFP_NOFAIL > that will loop forever in the allocator, Yes but does it make sense to warn for all of them? Wouldn't it be sufficient to warn about those which cannot allocate anything even though they are doing ALLOC_NO_WATERMARKS? We could still hint the administrator to increase min_free_kbytes for his particular workload. Such a situation should be really exceptional and warn_once should be sufficient. > and when this deadlocks the > machine all we see is other tasks hanging, but not the culprit. If we > were to get a backtrace of some task in the allocator that is known to > hold locks, suddenly all the other hung tasks will make sense, and it > will clearly distinguish such an allocator deadlock from other issues. Tetsuo was suggesting a more sophisticated infrastructure for tracking allocations [1] which take too long without making progress. I haven't seen his patch because I was too busy with other stuff but maybe this is what you would like to see? Anyway I would like to make some progress on this patch. Do you think that it would be acceptable in the current form without the warning or you preffer a different way? [1] http://lkml.kernel.org/r/201510182105.AGA00839.FHVFFStLQOMOOJ%40I-love.SAKURA.ne.jp -- Michal Hocko SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/