On Thu, Jul 21, 2016 at 04:10:50PM +0900, Joonsoo Kim wrote:
> > @@ -1993,6 +1994,32 @@ static bool inactive_list_is_low(struct lruvec *lruvec, bool file)
> > inactive = lruvec_lru_size(lruvec, file * LRU_FILE);
> > active = lruvec_lru_size(lruvec, file * LRU_FILE + LRU_ACTIVE);
> >
> > + /*
> > + * For global reclaim on zone-constrained allocations, it is necessary
> > + * to check if rotations are required for lowmem to be reclaimed. This
> > + * calculates the inactive/active pages available in eligible zones.
> > + */
> > + if (global_reclaim(sc)) {
> > + struct pglist_data *pgdat = lruvec_pgdat(lruvec);
> > + int zid;
> > +
> > + for (zid = sc->reclaim_idx; zid < MAX_NR_ZONES; zid++) {
>
> Should be changed to "zid = sc->reclaim_idx + 1"
>
You're right, well spotted!
--
Mel Gorman
SUSE Labs
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>