Messages in this thread | | | Date | Sat, 29 Nov 2008 11:24:15 -0500 | From | Rik van Riel <> | Subject | Re: [PATCH] vmscan: bail out of page reclaim after swap_cluster_max pages |
| |
KOSAKI Motohiro wrote:
> The result talk about three things. > > - rvr and mine patch increase direct reclaim imbalancing, indeed. > - However, background reclaim scanning is _very_ much than direct reclaim. > Then, direct reclaim imbalancing is ignorable on the big view. > rvr patch doesn't reintroduce zone imbalancing issue. > - rvr's priority==DEF_PRIORITY condition checking doesn't improve > zone balancing at all. > we can drop it. > > Again, I believe my patch improve vm scanning totally. > > Any comments?
Reclaiming is very easy when the workload is just page cache, because the application will be throttled when too many page cache pages are dirty.
When using mmap or memory hogs writing to swap, applications will not be throttled by the "too many dirty pages" logic, but may instead end up being throttled in the direct reclaim path instead.
At that point direct reclaim may become a lot more common, making the imbalance more significant.
I'll run a few tests.
> Andrew, I hope add this mesurement result to rvr bailing out patch description too.
So far the performance numbers you have measured are very encouraging and do indeed suggest that the priority==DEF_PRIORITY thing does not make a difference.
-- All rights reversed.
| |