Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755655Ab2BGN1v (ORCPT ); Tue, 7 Feb 2012 08:27:51 -0500 Received: from cantor2.suse.de ([195.135.220.15]:34581 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753969Ab2BGN1t (ORCPT ); Tue, 7 Feb 2012 08:27:49 -0500 Date: Tue, 7 Feb 2012 13:27:45 +0000 From: Mel Gorman To: Hillf Danton Cc: Andrew Morton , Linux-MM , Linux-Netdev , LKML , David Miller , Neil Brown , Peter Zijlstra Subject: Re: [PATCH 00/15] Swap-over-NBD without deadlocking V8 Message-ID: <20120207132745.GH5938@suse.de> References: <1328568978-17553-1-git-send-email-mgorman@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4661 Lines: 112 On Tue, Feb 07, 2012 at 08:45:18PM +0800, Hillf Danton wrote: > On Tue, Feb 7, 2012 at 6:56 AM, Mel Gorman wrote: > > > > The core issue is that network block devices do not use mempools like normal > > block devices do. As the host cannot control where they receive packets from, > > they cannot reliably work out in advance how much memory they might need. > > > > > > Patch 1 serialises access to min_free_kbytes. It's not strictly needed > > ? ? ? ?by this series but as the series cares about watermarks in > > ? ? ? ?general, it's a harmless fix. It could be merged independently. > > > > > Any light shed on tuning min_free_kbytes for every day work? > For every day work, leave min_free_kbytes as the default. > > > Patch 2 adds knowledge of the PFMEMALLOC reserves to SLAB and SLUB to > > ? ? ? ?preserve access to pages allocated under low memory situations > > ? ? ? ?to callers that are freeing memory. > > > > Patch 3 introduces __GFP_MEMALLOC to allow access to the PFMEMALLOC > > ? ? ? ?reserves without setting PFMEMALLOC. > > > > Patch 4 opens the possibility for softirqs to use PFMEMALLOC reserves > > ? ? ? ?for later use by network packet processing. > > > > Patch 5 ignores memory policies when ALLOC_NO_WATERMARKS is set. > > > > Patches 6-11 allows network processing to use PFMEMALLOC reserves when > > ? ? ? ?the socket has been marked as being used by the VM to clean > > ? ? ? ?pages. If packets are received and stored in pages that were > > ? ? ? ?allocated under low-memory situations and are unrelated to > > ? ? ? ?the VM, the packets are dropped. > > > > Patch 12 is a micro-optimisation to avoid a function call in the > > ? ? ? ?common case. > > > > Patch 13 tags NBD sockets as being SOCK_MEMALLOC so they can use > > ? ? ? ?PFMEMALLOC if necessary. > > > > If it is feasible to bypass hang by tuning min_mem_kbytes, No. Increasing or descreasing min_free_kbytes changes the timing but it will still hang. > things may > become simpler if NICs are also tagged. That would mean making changes to every driver and they do not necessarily know what higher level protocol like TCP they are transmitting. How is that simpler? What is the benefit? > Sock buffers, pre-allocated if > necessary just after NICs are turned on, are not handed back to kmem > cache but queued on local lists which are maintained by NIC driver, based > the on the info of min_mem_kbytes or similar, for tagged NICs. I think you are referring to doing something like SKB recycling within the driver. > Upside is no changes in VM core. Downsides? > That wouls indead requires driver-specific changes and new core infrastructure to deal with SKB recycling spreading the complexity over a wider range of code. If all the SKBs are in use for SOCK_MEMALLOC purposes for whatever reason and more cannot be allocated, it will still hang. So downsides are it would be equally if not more complex than this approach that it may still hang. > > Patch 14 notes that it is still possible for the PFMEMALLOC reserve > > ? ? ? ?to be depleted. To prevent this, direct reclaimers get > > ? ? ? ?throttled on a waitqueue if 50% of the PFMEMALLOC reserves are > > ? ? ? ?depleted. ?It is expected that kswapd and the direct reclaimers > > ? ? ? ?already running will clean enough pages for the low watermark > > ? ? ? ?to be reached and the throttled processes are woken up. > > > > Patch 15 adds a statistic to track how often processes get throttled > > > > > > For testing swap-over-NBD, a machine was booted with 2G of RAM with a > > swapfile backed by NBD. 8*NUM_CPU processes were started that create > > anonymous memory mappings and read them linearly in a loop. The total > > size of the mappings were 4*PHYSICAL_MEMORY to use swap heavily under > > memory pressure. Without the patches, the machine locks up within > > minutes and runs to completion with them applied. > > > > > > While testing, what happens if the network wire is plugged off over > three minutes? > I didn't test the scenario and I don't have a test machine available right now to try but it is up to the userspace NBD client to manage the reconnection. It is also up to the admin to prevent the NBD client being killed by something like the OOM killer and to have it mlocked to avoid the NBD client itself being swapped. NFS is able to handle this in-kernel but NBD may be more fragile. -- Mel Gorman 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/