Re: [PATCH v2 0/3] Separate NUMA statistics from zone statistics

2017-08-25 Thread Jesper Dangaard Brouer 
On Fri, 25 Aug 2017 09:04:37 +0100
Mel Gorman  wrote:

> On Thu, Aug 24, 2017 at 05:59:58PM +0800, Kemi Wang wrote:
> > Each page allocation updates a set of per-zone statistics with a call to
> > zone_statistics(). As discussed in 2017 MM summit, these are a substantial
> > source of overhead in the page allocator and are very rarely consumed. This
> > significant overhead in cache bouncing caused by zone counters (NUMA
> > associated counters) update in parallel in multi-threaded page allocation
> > (pointed out by Dave Hansen).
> >   
> 
> For the series;
> 
> Acked-by: Mel Gorman 
> 

I'm very happy to see these issues being worked on, from our MM-summit
interactions. I would like to provide/have a:

Reported-by: Jesper Dangaard Brouer 

As I'm not sure an acked-by from me have any value/merit here ;-)
-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  LinkedIn: http://www.linkedin.com/in/brouer

Re: [PATCH v2 0/3] Separate NUMA statistics from zone statistics

2017-08-25 Thread Jesper Dangaard Brouer 
On Fri, 25 Aug 2017 09:04:37 +0100
Mel Gorman  wrote:

> On Thu, Aug 24, 2017 at 05:59:58PM +0800, Kemi Wang wrote:
> > Each page allocation updates a set of per-zone statistics with a call to
> > zone_statistics(). As discussed in 2017 MM summit, these are a substantial
> > source of overhead in the page allocator and are very rarely consumed. This
> > significant overhead in cache bouncing caused by zone counters (NUMA
> > associated counters) update in parallel in multi-threaded page allocation
> > (pointed out by Dave Hansen).
> >   
> 
> For the series;
> 
> Acked-by: Mel Gorman 
> 

I'm very happy to see these issues being worked on, from our MM-summit
interactions. I would like to provide/have a:

Reported-by: Jesper Dangaard Brouer 

As I'm not sure an acked-by from me have any value/merit here ;-)
-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Principal Kernel Engineer at Red Hat
  LinkedIn: http://www.linkedin.com/in/brouer

Re: [PATCH v2 0/3] Separate NUMA statistics from zone statistics

2017-08-25 Thread Mel Gorman 
On Thu, Aug 24, 2017 at 05:59:58PM +0800, Kemi Wang wrote:
> Each page allocation updates a set of per-zone statistics with a call to
> zone_statistics(). As discussed in 2017 MM summit, these are a substantial
> source of overhead in the page allocator and are very rarely consumed. This
> significant overhead in cache bouncing caused by zone counters (NUMA
> associated counters) update in parallel in multi-threaded page allocation
> (pointed out by Dave Hansen).
> 

For the series;

Acked-by: Mel Gorman 

-- 
Mel Gorman
SUSE Labs

Re: [PATCH v2 0/3] Separate NUMA statistics from zone statistics

2017-08-25 Thread Mel Gorman 
On Thu, Aug 24, 2017 at 05:59:58PM +0800, Kemi Wang wrote:
> Each page allocation updates a set of per-zone statistics with a call to
> zone_statistics(). As discussed in 2017 MM summit, these are a substantial
> source of overhead in the page allocator and are very rarely consumed. This
> significant overhead in cache bouncing caused by zone counters (NUMA
> associated counters) update in parallel in multi-threaded page allocation
> (pointed out by Dave Hansen).
> 

For the series;

Acked-by: Mel Gorman 

-- 
Mel Gorman
SUSE Labs