On Mon 23-11-15 13:24:47, Johannes Weiner wrote: > On Mon, Nov 23, 2015 at 10:41:06AM +0100, Michal Hocko wrote: > > @@ -3197,8 +3197,10 @@ __alloc_pages_slowpath(gfp_t gfp_mask, unsigned int order, > > unsigned long target; > > > > reclaimable = zone_reclaimable_pages(zone) + > > - zone_page_state(zone, NR_ISOLATED_FILE) + > > - zone_page_state(zone, NR_ISOLATED_ANON); > > + zone_page_state(zone, NR_ISOLATED_FILE); > > + if (get_nr_swap_pages() > 0) > > + reclaimable += zone_page_state(zone, NR_ISOLATED_ANON); > > Can you include the isolated counts in zone_reclaimable_pages()? OK, this makes sense. NR_ISOLATED_* should be a temporary condition after which pages either get back to the LRU or they get migrated to a different location thus freed. I will spin this intot a separate patch. Thanks! -- Michal Hocko 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>