Hi Joonsoo,

On Mon, Jan 06, 2014 at 09:30:52AM +0900, Joonsoo Kim wrote:
> On Sun, Jan 05, 2014 at 05:04:56PM +0800, fengguang...@intel.com wrote:
> > Hi Joonsoo,
> > 
> > We noticed the below changes for commit 23f0d2093c ("sched: Factor out
> > code to should_we_balance()") in test vm-scalability/300s-lru-file-readtwice
> 
> Hello, Fengguang.
> 
> There was a mistake in this patch and there was a fix and it was already 
> merged
> into mainline.
> 
> Could you test again with the commit (b0cff9d sched: Fix load balancing
> performance regression in should_we_balance())?

Yes, b0cff9d completely restores the performance. Sorry for the noise!

Thanks,
Fengguang

> > 
> >         95a79b805b935f4  23f0d2093c789e612185180c4
> >         ---------------  -------------------------
> > ==>           4.45 ~ 5%   +1777.7%      83.60 ~ 5%  vm-scalability.stddev
> > ==>       14966511 ~ 0%     -12.6%   13084545 ~ 2%  
> > vm-scalability.throughput
> >                 38 ~ 9%    +406.3%        193 ~ 7%  
> > proc-vmstat.kswapd_low_wmark_hit_quickly
> >             610823 ~ 0%     -41.4%     357990 ~ 0%  softirqs.SCHED
> >          5.424e+08 ~ 0%     -38.5%  3.338e+08 ~ 6%  proc-vmstat.pgdeactivate
> >           4.68e+08 ~ 0%     -37.5%  2.924e+08 ~ 6%  
> > proc-vmstat.pgrefill_normal
> >          5.549e+08 ~ 0%     -37.1%  3.491e+08 ~ 6%  proc-vmstat.pgactivate
> >           14938509 ~ 1%     +27.0%   18974176 ~ 1%  vmstat.memory.free
> >             978771 ~ 1%     +23.9%    1212704 ~ 3%  
> > numa-vmstat.node2.nr_free_pages
> >            3747434 ~ 0%     +21.7%    4560196 ~ 2%  
> > proc-vmstat.nr_free_pages
> > ==>      1.353e+08 ~ 0%     +18.8%  1.607e+08 ~ 0%  proc-vmstat.numa_foreign
> >          1.353e+08 ~ 0%     +18.8%  1.607e+08 ~ 0%  proc-vmstat.numa_miss
> >          1.353e+08 ~ 0%     +18.8%  1.607e+08 ~ 0%  proc-vmstat.numa_other
> >            3936842 ~ 1%     +22.2%    4812045 ~ 4%  
> > numa-meminfo.node2.MemFree
> >           21803812 ~ 0%     +17.7%   25661536 ~ 4%  
> > numa-vmstat.node3.numa_foreign
> >           73701524 ~ 0%     +15.0%   84769542 ~ 0%  
> > proc-vmstat.pgscan_direct_dma32
> >           73700683 ~ 0%     +15.0%   84768687 ~ 0%  
> > proc-vmstat.pgsteal_direct_dma32
> >          3.101e+08 ~ 0%     +11.2%  3.448e+08 ~ 0%  
> > proc-vmstat.pgsteal_direct_normal
> >          3.103e+08 ~ 0%     +11.2%  3.449e+08 ~ 0%  
> > proc-vmstat.pgscan_direct_normal
> >           45613907 ~ 0%     +12.6%   51342974 ~ 3%  
> > numa-vmstat.node0.numa_other
> >             795639 ~ 0%     -48.6%     409113 ~13%  
> > time.voluntary_context_switches
> >                375 ~ 0%      +6.1%        398 ~ 0%  time.elapsed_time
> >               9427 ~ 0%      -5.8%       8880 ~ 0%  
> > time.percent_of_cpu_this_job_got
> > 
> > The test case basically does
> > 
> > for i in `seq 1 $nr_cpu`
> > do      
> >         create_sparse_file huge-$i
> >         dd if=huge-$i of=/dev/null &
> >         dd if=huge-$i of=/dev/null &
> > done
> > 
> > where nr_cpu=120 (test box is a 4-socket ivybridge system).
> > 
> > The change looks stable, each point below is a sample run:
> > 
> >                                vm-scalability.stddev
> > 
> >    120 
> > ++-------------------------------------------------------------------+
> >        |                                                                    
> > |
> >    100 ++             *            *                                        
> > |
> >        | *.***        :    **      :     *    *  *     *     *              
> > |
> >        **     * *.** * :  *  :*.*  :: .* :   : * :*   * :  .* :   .*   * 
> > .**|
> >     80 ++      *    *  *. :  *   *: ** : ::  :  *  :.*  * *   * ** :   :*   
> > *
> >        |                 *        *     : ***      *     *     *    :**     
> > |
> >     60 ++                               *                           *       
> > |
> >        |                                                                    
> > |
> >     40 ++                                                                   
> > |
> >        |                                                                    
> > |
> >        |                                                                    
> > |
> >     20 ++                                                                   
> > |
> >        | O  OO OO OOO  O OO  O                                              
> > |
> >      0 
> > OO--O--O------OO----OO-----------------------------------------------+
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majord...@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to