lkml.org 
[lkml]   [2012]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[RFC PATCH] mm: don't treat anonymous pages as dirtyable pages
> 5) reset counters and stress it more.
>
> # usemem 1G --sleep 1000&
> # free
> total used free shared buffers cached
> Mem: 6801 6758 42 0 0 994
> -/+ buffers/cache: 5764 1036
> Swap: 51106 235 50870
>
> It's now obviously slow, it now takes seconds or even 10+ seconds to switch to
> the other windows:
>
> 765.30 A System Monitor
> 769.72 A Dictionary
> 772.01 A Home
> 790.79 A Desktop Help
> 795.47 A *Unsaved Document 1 - gedit
> 813.01 A ALC888.svg (1/11)
> 819.24 A Restore Session - Iceweasel
> 827.23 A Klondike
> 853.57 A urxvt
> 862.49 A xeyes
> 868.67 A Xpdf: /usr/share/doc/shared-mime-info/shared-mime-info-spec.pdf
> 869.47 A snb:/home/wfg - ZSH
>
> And it seems that the slowness is caused by huge number of pageout()s:
>
> /debug/vm/nr_reclaim_throttle_clean:0
> /debug/vm/nr_reclaim_throttle_kswapd:0
> /debug/vm/nr_reclaim_throttle_recent_write:0
> /debug/vm/nr_reclaim_throttle_write:307
> /debug/vm/nr_congestion_wait:0
> /debug/vm/nr_reclaim_wait_congested:0
> /debug/vm/nr_reclaim_wait_writeback:0
> /debug/vm/nr_migrate_wait_writeback:0
> nr_vmscan_write 175085
> allocstall 669671

The heavy swapping is a big problem. This patch is found to
effectively eliminate it :-)

---
Subject: mm: don't treat anonymous pages as dirtyable pages

Assume a mem=1GB desktop (swap enabled) with 800MB anonymous pages and
200MB file pages. When the user starts a heavy dirtier task, the file
LRU lists may be mostly filled with dirty pages since the global dirty
limit is calculated as

(anon+file) * 20% = 1GB * 20% = 200MB

This makes the file LRU lists hard to reclaim, which in turn increases
the scan rate of the anon LRU lists and lead to a lot of swapping. This
is probably one big reason why some desktop users see bad responsiveness
during heavy file copies once the swap is enabled.

The heavy swapping could mostly be avoided by calculating the global
dirty limit as

file * 20% = 200MB * 20% = 40MB

The side effect would be that users feel longer file copy time because
the copy task is throttled earlier than before. However typical users
should be much more sensible to interactive performance rather than the
copy task which may well be leaved in the background.

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
include/linux/vmstat.h | 1 -
mm/page-writeback.c | 10 ++++++----
mm/vmscan.c | 14 --------------
3 files changed, 6 insertions(+), 19 deletions(-)

--- linux.orig/include/linux/vmstat.h 2012-03-02 13:55:28.569749568 +0800
+++ linux/include/linux/vmstat.h 2012-03-02 13:56:06.585750471 +0800
@@ -139,7 +139,6 @@ static inline unsigned long zone_page_st
return x;
}

-extern unsigned long global_reclaimable_pages(void);
extern unsigned long zone_reclaimable_pages(struct zone *zone);

#ifdef CONFIG_NUMA
--- linux.orig/mm/page-writeback.c 2012-03-02 13:55:28.549749567 +0800
+++ linux/mm/page-writeback.c 2012-03-02 13:56:26.257750938 +0800
@@ -181,8 +181,7 @@ static unsigned long highmem_dirtyable_m
struct zone *z =
&NODE_DATA(node)->node_zones[ZONE_HIGHMEM];

- x += zone_page_state(z, NR_FREE_PAGES) +
- zone_reclaimable_pages(z) - z->dirty_balance_reserve;
+ x += zone_dirtyable_memory(z);
}
/*
* Make sure that the number of highmem pages is never larger
@@ -206,7 +205,9 @@ unsigned long global_dirtyable_memory(vo
{
unsigned long x;

- x = global_page_state(NR_FREE_PAGES) + global_reclaimable_pages() -
+ x = global_page_state(NR_FREE_PAGES) +
+ global_page_state(NR_ACTIVE_FILE) +
+ global_page_state(NR_INACTIVE_FILE) -
dirty_balance_reserve;

if (!vm_highmem_is_dirtyable)
@@ -275,7 +276,8 @@ unsigned long zone_dirtyable_memory(stru
* care about vm_highmem_is_dirtyable here.
*/
return zone_page_state(zone, NR_FREE_PAGES) +
- zone_reclaimable_pages(zone) -
+ zone_page_state(zone, NR_ACTIVE_FILE) +
+ zone_page_state(zone, NR_INACTIVE_FILE) -
zone->dirty_balance_reserve;
}

--- linux.orig/mm/vmscan.c 2012-03-02 13:55:28.561749567 +0800
+++ linux/mm/vmscan.c 2012-03-02 13:56:06.585750471 +0800
@@ -3315,20 +3315,6 @@ void wakeup_kswapd(struct zone *zone, in
* - mapped pages, which may require several travels to be reclaimed
* - dirty pages, which is not "instantly" reclaimable
*/
-unsigned long global_reclaimable_pages(void)
-{
- int nr;
-
- nr = global_page_state(NR_ACTIVE_FILE) +
- global_page_state(NR_INACTIVE_FILE);
-
- if (nr_swap_pages > 0)
- nr += global_page_state(NR_ACTIVE_ANON) +
- global_page_state(NR_INACTIVE_ANON);
-
- return nr;
-}
-
unsigned long zone_reclaimable_pages(struct zone *zone)
{
int nr;

\
 
 \ /
  Last update: 2012-03-02 08:03    [W:0.254 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site