On Thu, Nov 04, 2010 at 08:50:58AM +0800, Shaohua Li wrote: > nr_dirty and nr_congested are increased only when page is dirty. So if all pages > are clean, both them will be zero. In this case, we should not mark the zone > congested. > > Signed-off-by: Shaohua Li <shaohua.li@xxxxxxxxx> Acked-by: Mel Gorman <mel@xxxxxxxxx> > > diff --git a/mm/vmscan.c b/mm/vmscan.c > index b8a6fdc..d31d7ce 100644 > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -913,7 +913,7 @@ keep_lumpy: > * back off and wait for congestion to clear because further reclaim > * will encounter the same problem > */ > - if (nr_dirty == nr_congested) > + if (nr_dirty == nr_congested && nr_dirty != 0) > zone_set_flag(zone, ZONE_CONGESTED); > > free_page_list(&free_pages); > > -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>