Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753886AbbK3NCF (ORCPT ); Mon, 30 Nov 2015 08:02:05 -0500 Received: from mail-wm0-f41.google.com ([74.125.82.41]:34856 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752067AbbK3NCD (ORCPT ); Mon, 30 Nov 2015 08:02:03 -0500 Date: Mon, 30 Nov 2015 14:02:00 +0100 From: Michal Hocko To: "Huang, Ying" Cc: Mel Gorman , lkp@01.org, LKML , Andrew Morton , Rik van Riel , Vitaly Wool , David Rientjes , Christoph Lameter , Johannes Weiner , Vlastimil Babka , Linus Torvalds , Will Deacon Subject: Re: [lkp] [mm, page_alloc] d0164adc89: -100.0% fsmark.app_overhead Message-ID: <20151130130200.GA21950@dhcp22.suse.cz> References: <87ziy1a89f.fsf@yhuang-dev.intel.com> <20151126132511.GG14880@techsingularity.net> <87oaegmeer.fsf@yhuang-dev.intel.com> <20151127100647.GH14880@techsingularity.net> <87h9k4kzcv.fsf@yhuang-dev.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87h9k4kzcv.fsf@yhuang-dev.intel.com> 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: 3104 Lines: 69 [Let's CC Will - see the question at the end of the email please] This seems to be a similar allocation failure reported http://lkml.kernel.org/r/87oafjpnb1.fsf%40yhuang-dev.intel.com where I failed to see the important point, more on that below. On Mon 30-11-15 10:14:24, Huang, Ying wrote: > Mel Gorman writes: > > > On Fri, Nov 27, 2015 at 09:14:52AM +0800, Huang, Ying wrote: > >> Hi, Mel, > >> > >> Mel Gorman writes: > >> > >> > On Thu, Nov 26, 2015 at 08:56:12AM +0800, kernel test robot wrote: > >> >> FYI, we noticed the below changes on > >> >> > >> >> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > >> >> commit d0164adc89f6bb374d304ffcc375c6d2652fe67d ("mm, page_alloc: > >> >> distinguish between being unable to sleep, unwilling to sleep and > >> >> avoiding waking kswapd") > >> >> > >> >> Note: the testing machine is a virtual machine with only 1G memory. > >> >> > >> > > >> > I'm not actually seeing any problem here. Is this a positive report or > >> > am I missing something obvious? > >> > >> Sorry the email subject is generated automatically and I forget to > >> change it to some meaningful stuff before sending out. From the testing > >> result, we found the commit make the OOM possibility increased from 0% > >> to 100% on this machine with small memory. I also added proc-vmstat > >> information data too to help diagnose it. > >> > > > > There is no reference to OOM possibility in the email that I can see. Can > > you give examples of the OOM messages that shows the problem sites? It was > > suspected that there may be some callers that were accidentally depending > > on access to emergency reserves. If so, either they need to be fixed (if > > the case is extremely rare) or a small reserve will have to be created > > for callers that are not high priority but still cannot reclaim. __virtblk_add_req calls virtqueue_add_sgs(vq, sgs, num_out, num_in, vbr, GFP_ATOMIC) alloc_indirect(gfp) gfp &= ~(__GFP_HIGHMEM | __GFP_HIGH) So this is true __GFP_ATOMIC, we just drop __GFP_HIGH so it doesn't get access to more reserves. It still does ALLOC_HARDER. So I think the real issue is somewhere else when something should have triggered kswapd and it doesn't do that anymore. I have tried to find that offender the last time but didn't manage to find any. Btw. I completely miss why b92b1b89a33c ("virtio: force vring descriptors to be allocated from lowmem") had to clear __GFP_HIGH. Will do you remember why you have dropped that flag as well? Also I do not seem to find any user of alloc_indirect which would do __GFP_HIGHMEM. All of them are either GFP_KERNEL or GFP_ATOMIC. So either I am missing something or this is not really needed. Maybe the situation was different back in 2012. -- 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/