On Tue, Sep 07, 2010 at 10:32:44AM +0900, KAMEZAWA Hiroyuki wrote: > From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> > > next_active_pageblock() is for finding next _used_ freeblock. It skips > several blocks when it finds there are a chunk of free pages lager than > pageblock. But it has 2 bugs. > > 1. We have no lock. page_order(page) - pageblock_order can be minus. > 2. pageblocks_stride += is wrong. it should skip page_order(p) of pages. > > Changelog: 2010/09/07 > - fix range check of order returned by page_order(). > > Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> Acked-by: Mel Gorman <mel@xxxxxxxxx> -- 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/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>