From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ua0-f200.google.com (mail-ua0-f200.google.com [209.85.217.200]) by kanga.kvack.org (Postfix) with ESMTP id C58D16B0253 for ; Sun, 14 Aug 2016 06:34:12 -0400 (EDT) Received: by mail-ua0-f200.google.com with SMTP id u13so68525803uau.2 for ; Sun, 14 Aug 2016 03:34:12 -0700 (PDT) Received: from mail-wm0-f68.google.com (mail-wm0-f68.google.com. [74.125.82.68]) by mx.google.com with ESMTPS id ch18si14898127wjb.75.2016.08.14.03.34.11 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 14 Aug 2016 03:34:11 -0700 (PDT) Received: by mail-wm0-f68.google.com with SMTP id o80so5950661wme.0 for ; Sun, 14 Aug 2016 03:34:11 -0700 (PDT) Date: Sun, 14 Aug 2016 12:34:09 +0200 From: Michal Hocko Subject: Re: [dm-devel] [RFC PATCH 2/2] mm, mempool: do not throttle PF_LESS_THROTTLE tasks Message-ID: <20160814103409.GC9248@dhcp22.suse.cz> References: <20160725083247.GD9401@dhcp22.suse.cz> <87lh0n4ufs.fsf@notabene.neil.brown.name> <20160727182411.GE21859@dhcp22.suse.cz> <87eg6e4vhc.fsf@notabene.neil.brown.name> <20160728071711.GB31860@dhcp22.suse.cz> <20160803143419.GC1490@dhcp22.suse.cz> <20160812123242.GH3639@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Mikulas Patocka Cc: Mel Gorman , NeilBrown , Tetsuo Handa , LKML , linux-mm@kvack.org, "dm-devel@redhat.com David Rientjes" , Ondrej Kozina , Andrew Morton On Sat 13-08-16 13:34:29, Mikulas Patocka wrote: > > > On Fri, 12 Aug 2016, Michal Hocko wrote: > > > On Thu 04-08-16 14:49:41, Mikulas Patocka wrote: > > > > > On Wed, 3 Aug 2016, Michal Hocko wrote: > > > > > > > But the device congestion is not the only condition required for the > > > > throttling. The pgdat has also be marked congested which means that the > > > > LRU page scanner bumped into dirty/writeback/pg_reclaim pages at the > > > > tail of the LRU. That should only happen if we are rotating LRUs too > > > > quickly. AFAIU the reclaim shouldn't allow free ticket scanning in that > > > > situation. > > > > > > The obvious problem here is that mempool allocations should sleep in > > > mempool_alloc() on &pool->wait (until someone returns some entries into > > > the mempool), they should not sleep inside the page allocator. > > > > I agree that mempool_alloc should _primarily_ sleep on their own > > throttling mechanism. I am not questioning that. I am just saying that > > the page allocator has its own throttling which it relies on and that > > cannot be just ignored because that might have other undesirable side > > effects. So if the right approach is really to never throttle certain > > requests then we have to bail out from a congested nodes/zones as soon > > as the congestion is detected. > > > > Now, I would like to see that something like that is _really_ necessary. > > Currently, it is not a problem - device mapper reports the device as > congested only if the underlying physical disks are congested. > > But once we change it so that device mapper reports congested state on its > own (when it has too many bios in progress), this starts being a problem. OK, can we wait until it starts becoming a real problem and solve it appropriately then? I will repost the patch which removes thottle_vm_pageout in the meantime as it doesn't seem to be needed anymore. -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org