From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f197.google.com (mail-pf0-f197.google.com [209.85.192.197]) by kanga.kvack.org (Postfix) with ESMTP id 6065F6B0033 for ; Wed, 13 Sep 2017 20:53:10 -0400 (EDT) Received: by mail-pf0-f197.google.com with SMTP id q75so2656023pfl.1 for ; Wed, 13 Sep 2017 17:53:10 -0700 (PDT) Received: from mga02.intel.com (mga02.intel.com. [134.134.136.20]) by mx.google.com with ESMTPS id t66si9793496pgc.220.2017.09.13.17.53.08 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 13 Sep 2017 17:53:09 -0700 (PDT) From: "Huang\, Ying" Subject: Re: [PATCH -mm -v4 3/5] mm, swap: VMA based swap readahead References: <20170807054038.1843-1-ying.huang@intel.com> <20170807054038.1843-4-ying.huang@intel.com> <20170913014019.GB29422@bbox> <20170913140229.8a6cad6f017fa3ea8b53cefc@linux-foundation.org> Date: Thu, 14 Sep 2017 08:53:04 +0800 In-Reply-To: <20170913140229.8a6cad6f017fa3ea8b53cefc@linux-foundation.org> (Andrew Morton's message of "Wed, 13 Sep 2017 14:02:29 -0700") Message-ID: <87lglim77z.fsf@yhuang-dev.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ascii Sender: owner-linux-mm@kvack.org List-ID: To: Andrew Morton Cc: Minchan Kim , "Huang, Ying" , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Johannes Weiner , Rik van Riel , Shaohua Li , Hugh Dickins , Fengguang Wu , Tim Chen , Dave Hansen Hi, Andrew, Andrew Morton writes: > On Wed, 13 Sep 2017 10:40:19 +0900 Minchan Kim wrote: > >> Every zram users like low-end android device has used 0 page-cluster >> to disable swap readahead because it has no seek cost and works as >> synchronous IO operation so if we do readahead multiple pages, >> swap falut latency would be (4K * readahead window size). IOW, >> readahead is meaningful only if it doesn't bother faulted page's >> latency. >> >> However, this patch introduces additional knob /sys/kernel/mm/swap/ >> vma_ra_max_order as well as page-cluster. It means existing users >> has used disabled swap readahead doesn't work until they should be >> aware of new knob and modification of their script/code to disable >> vma_ra_max_order as well as page-cluster. >> >> I say it's a *regression* and wanted to fix it but Huang's opinion >> is that it's not a functional regression so userspace should be fixed >> by themselves. >> Please look into detail of discussion in >> http://lkml.kernel.org/r/%3C1505183833-4739-4-git-send-email-minchan@kernel.org%3E > > hm, tricky problem. I do agree that linking the physical and virtual > readahead schemes in the proposed fashion is unfortunate. I also agree > that breaking existing setups (a bit) is also unfortunate. > > Would it help if, when page-cluster is written to zero, we do > > printk_once("physical readahead disabled, virtual readahead still > enabled. Disable virtual readhead via > /sys/kernel/mm/swap/vma_ra_max_order"). > > Or something like that. It's pretty lame, but it should help alert the > zram-readahead-disabling people to the issue? This sounds good for me. Hi, Minchan, what do you think about this? I think for low-end android device, the end-user may have no opportunity to upgrade to the latest kernel, the device vendor should care about this. For desktop users, the warning proposed by Andrew may help to remind them for the new knob. Best Regards, Huang, Ying -- 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