lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 29 Oct 2010 11:23:07 +0900 (JST) From: KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com> To: Andrea Arcangeli <aarcange@...hat.com> Cc: kosaki.motohiro@...fujitsu.com, Mel Gorman <mel@....ul.ie>, LKML <linux-kernel@...r.kernel.org>, linux-mm <linux-mm@...ck.org>, Andrew Morton <akpm@...ux-foundation.org>, Johannes Weiner <hannes@...xchg.org>, Wu Fengguang <fengguang.wu@...el.com>, Minchan Kim <minchan.kim@...il.com>, Rik van Riel <riel@...hat.com> Subject: Re: [PATCH 4/7] vmscan: narrowing synchrounous lumply reclaim condition > On Thu, Oct 28, 2010 at 05:00:57PM +0900, KOSAKI Motohiro wrote: > > Hi > > > > > My tree uses compaction in a fine way inside kswapd too and tons of > > > systems are running without lumpy and floods of order 9 allocations > > > with only compaction (in direct reclaim and kswapd) without the > > > slighest problem. Furthermore I extended compaction for all > > > allocations not just that PAGE_ALLOC_COSTLY_ORDER (maybe I already > > > removed all PAGE_ALLOC_COSTLY_ORDER checks?). There's no good reason > > > not to use compaction for every allocation including 1,2,3, and things > > > works fine this way. > > > > Interesting. I parsed this you have compaction improvement. If so, > > can you please post them? Generically, 1) improve the feature 2) remove > > unused one is safety order. In the other hand, reverse order seems to has > > regression risk. > > THP is way higher priority than the compaction improvements, so the > compaction improvements are not at the top of the queue: > > http://git.kernel.org/?p=linux/kernel/git/andrea/aa.git;a=shortlog > > http://git.kernel.org/?p=linux/kernel/git/andrea/aa.git;a=commitdiff;h=d8f02410d718725a7daaf192af33abc41dcfae16;hp=39c4a61fedc5f5bf0c95a60483ac0acea1a9a757 > > At the top of the queue there is the lumpy_reclaim removal as that's > higher priority than THP. Umm... If THP vs lumpy confliction is most big matter, I'd prefer automatical lumpy disabling when THP enabled rather than completely removing. It is lower risk. And, After finish to improve compaction, I expect we will be able to discuss remove thing. If my parse is correct, You have tested "improved-compaction + no-lumpy + THP" combination, but nobody have tested "current-compaction + no-lumpy". IOW, I only say I dislike a regression. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists