From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
To: kosaki.motohiro@jp.fujitsu.com
Cc: LKML <linux-kernel@vger.kernel.org>,
linux-mm <linux-mm@kvack.org>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 0/5] page reclaim throttle v7
Date: Thu, 5 Jun 2008 11:06:37 +0900 [thread overview]
Message-ID: <20080605110637.d50af953.kamezawa.hiroyu@jp.fujitsu.com> (raw)
In-Reply-To: <20080605021211.871673550@jp.fujitsu.com>
On Thu, 05 Jun 2008 11:12:11 +0900
kosaki.motohiro@jp.fujitsu.com wrote:
> Hi
>
> I post latest version of page reclaim patch series.
>
> This patch is holding up very well under usex stress test
> over 24+ hours :)
>
>
> Against: 2.6.26-rc2-mm1
>
I like this series and I'd like to support this under memcg when
this goes to mainline. (it seems better to test this for a while
before adding some memcg-related changes.)
Then, please give me inputs.
What do you think do I have to do for supporting this in memcg ?
Handling the case of scan_global_lru(sc)==false is enough ?
Thanks,
-Kame
>
> changelog
> ========================================
> v6 -> v7
> o rebase to 2.6.26-rc2-mm1
> o get_vm_stat: make cpu-unplug safety.
> o mark vm_max_nr_task_per_zone __read_mostly.
> o add check __GFP_FS, __GFP_IO for avoid deadlock.
> o fixed compile error on x86_64.
>
> v5 -> v6
> o rebase to 2.6.25-mm1
> o use PGFREE statics instead wall time.
> o separate function type change patch and introduce throttle patch.
>
> v4 -> v5
> o rebase to 2.6.25-rc8-mm1
>
> v3 -> v4:
> o fixed recursive shrink_zone problem.
> o add last_checked variable in shrink_zone for
> prevent corner case regression.
>
> v2 -> v3:
> o use wake_up() instead wake_up_all()
> o max reclaimers can be changed Kconfig option and sysctl.
> o some cleanups
>
> v1 -> v2:
> o make per zone throttle
>
>
>
> background
> =====================================
> current VM implementation doesn't has limit of # of parallel reclaim.
> when heavy workload, it bring to 2 bad things
> - heavy lock contention
> - unnecessary swap out
>
> at end of last year, KAMEZAWA Hiroyuki proposed the patch of page
> reclaim throttle and explain it improve reclaim time.
> http://marc.info/?l=linux-mm&m=119667465917215&w=2
>
> but unfortunately it works only memcgroup reclaim.
> since, I implement it again for support global reclaim and mesure it.
>
>
> benefit
> =====================================
> <<1. fix the bug of incorrect OOM killer>>
>
> if do following commanc, sometimes OOM killer happened.
> (OOM happend about 10%)
>
> $ ./hackbench 125 process 1000
>
> because following bad scenario is happend.
>
> 1. memory shortage happend.
> 2. many task call shrink_zone at the same time.
> 3. thus, All page are isolated from LRU at the same time.
> 4. the last task can't isolate any page from LRU.
> 5. it cause reclaim failure.
> 6. it cause OOM killer.
>
> my patch is directly solution for that problem.
>
>
> <<2. performance improvement>>
> I mesure RvR Split LRU series + page reclaim throttle series performance by hackbench.
>
> result number mean seconds (i.e. smaller is better)
>
>
> + split_lru improvement
> num_group 2.6.26-rc2-mm1 + throttle ratio
> -----------------------------------------------------------------
> 100 28.383 28.247
> 110 31.237 30.83
> 120 33.282 33.473
> 130 36.530 37.356
> 140 101.041 44.873 >200%
> 150 795.020 96.265 >800%
>
>
> Why this patch imrove performance?
>
> vanilla kernel get unstable performance at swap happend because
> unnecessary swap out happend freqently.
> this patch doesn't improvement best case, but be able to prevent worst case.
> thus, The average performance of hackbench increase largely.
>
> --
> 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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
>
--
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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2008-06-05 2:06 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-05 2:12 kosaki.motohiro
2008-06-05 2:06 ` KAMEZAWA Hiroyuki [this message]
2008-06-05 2:23 ` KOSAKI Motohiro
2008-06-05 2:12 ` [PATCH 1/5] fix incorrect variable type of do_try_to_free_pages() kosaki.motohiro
2008-06-05 1:26 ` KOSAKI Motohiro
2008-06-05 2:12 ` [PATCH 2/5] introduce get_vm_event() kosaki.motohiro
2008-06-05 1:29 ` KOSAKI Motohiro
2008-06-05 2:12 ` [PATCH 3/5] change return type of shrink_zone() kosaki.motohiro
2008-06-05 2:12 ` [PATCH 4/5] add throttle to shrink_zone() kosaki.motohiro
2008-06-08 20:12 ` Andrew Morton
2008-06-09 0:38 ` KOSAKI Motohiro
2008-06-05 2:12 ` [PATCH 5/5] introduce sysctl of throttle kosaki.motohiro
2008-06-08 20:09 ` Andrew Morton
2008-06-09 0:37 ` KOSAKI Motohiro
2008-06-08 20:10 ` Andrew Morton
2008-06-09 0:34 ` KOSAKI Motohiro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080605110637.d50af953.kamezawa.hiroyu@jp.fujitsu.com \
--to=kamezawa.hiroyu@jp.fujitsu.com \
--cc=akpm@linux-foundation.org \
--cc=kosaki.motohiro@jp.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox