Re: [PATCH] mm: fix pgalloc_stall on unpopulated zone

2016-07-14 Thread Minchan Kim
On Thu, Jul 14, 2016 at 09:51:53AM +0100, Mel Gorman wrote: > On Thu, Jul 14, 2016 at 10:11:19AM +0900, Minchan Kim wrote: > > > The patch means that the vmstat accounting and tracepoint data is also > > > out of sync. One thing I wanted to be able to do was > > > > > > 1. Observe that there are a

Re: [PATCH] mm: fix pgalloc_stall on unpopulated zone

2016-07-14 Thread Mel Gorman
On Thu, Jul 14, 2016 at 10:11:19AM +0900, Minchan Kim wrote: > > The patch means that the vmstat accounting and tracepoint data is also > > out of sync. One thing I wanted to be able to do was > > > > 1. Observe that there are alloc stalls on DMA32 or some other low zone > > 2. Activate mm_vmscan_

Re: [PATCH] mm: fix pgalloc_stall on unpopulated zone

2016-07-13 Thread Minchan Kim
On Wed, Jul 13, 2016 at 10:25:04AM +0100, Mel Gorman wrote: > On Wed, Jul 13, 2016 at 11:24:13AM +0900, Minchan Kim wrote: > > If we use sc->reclaim_idx for accounting pgstall, it can increase > > the count on unpopulated zone, for example, movable zone(but > > my system doesn't have movable zone)

Re: [PATCH] mm: fix pgalloc_stall on unpopulated zone

2016-07-13 Thread Mel Gorman
On Wed, Jul 13, 2016 at 11:24:13AM +0900, Minchan Kim wrote: > If we use sc->reclaim_idx for accounting pgstall, it can increase > the count on unpopulated zone, for example, movable zone(but > my system doesn't have movable zone) if allocation request were > GFP_HIGHUSER_MOVABLE. It doesn't make n