Re: [LKP] [timer] 17cdf560f27: +4.0% boot-meminfo.VmallocUsed

2015-04-03 Thread Peter Zijlstra
On Fri, Apr 03, 2015 at 09:03:48AM +0800, huang ying wrote:
> On Thu, Apr 2, 2015 at 3:54 PM, Peter Zijlstra  wrote:
> 
> > On Thu, Apr 02, 2015 at 09:06:09AM +0800, Huang Ying wrote:
> > > FYI, we noticed the below changes on
> > >
> > > git://git.kernel.org/pub/scm/linux/kernel/git/vireshk/linux.git
> > timer/alloc_tvec_statically
> > > commit 17cdf560f2727f687ab159707d0aa591f8a2f82d ("timer: Allocate
> > per-cpu tvec_base's statically")
> > >
> > >
> > > testbox/testcase/testparams: lkp-sbx04/boot/performance-1
> > >
> > > c5e77f5216abdd1d  17cdf560f2727f687ab159707d
> > >   --
> > >  %stddev %change %stddev
> > >  \  |\
> > > 410996 ±  0%  +4.0% 427412 ±  0%  boot-meminfo.VmallocUsed
> > >
> > > testbox/testcase/testparams: wsm/boot/performance-1
> > >
> > > c5e77f5216abdd1d  17cdf560f2727f687ab159707d
> > >   --
> > >  85712 ±  0%  +2.4%  87772 ±  0%  boot-meminfo.VmallocUsed
> > >
> > > testbox/testcase/testparams: lkp-snb01/boot/performance-1
> > >
> > > c5e77f5216abdd1d  17cdf560f2727f687ab159707d
> > >   --
> > > 447001 ±  0%  -1.0% 442558 ±  0%  boot-slabinfo.num_objs
> > > 125616 ±  5% -13.5% 108720 ±  4%  boot-meminfo.DirectMap4k
> > > 347196 ±  0%  +4.7% 363612 ±  0%  boot-meminfo.VmallocUsed
> > >
> >
> > well, duh.. it changes a dynamic allocation into a static per-cpu one.
> > What did you expect?
> >
> 
> Just FYI.  This may be the expected behavior.  If so, you have a third
> party test report.

Ah ok. I thought you thought it was a bug/regression.

Thanks
--
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/


Re: [LKP] [timer] 17cdf560f27: +4.0% boot-meminfo.VmallocUsed

2015-04-03 Thread Peter Zijlstra
On Fri, Apr 03, 2015 at 09:03:48AM +0800, huang ying wrote:
 On Thu, Apr 2, 2015 at 3:54 PM, Peter Zijlstra pet...@infradead.org wrote:
 
  On Thu, Apr 02, 2015 at 09:06:09AM +0800, Huang Ying wrote:
   FYI, we noticed the below changes on
  
   git://git.kernel.org/pub/scm/linux/kernel/git/vireshk/linux.git
  timer/alloc_tvec_statically
   commit 17cdf560f2727f687ab159707d0aa591f8a2f82d (timer: Allocate
  per-cpu tvec_base's statically)
  
  
   testbox/testcase/testparams: lkp-sbx04/boot/performance-1
  
   c5e77f5216abdd1d  17cdf560f2727f687ab159707d
     --
%stddev %change %stddev
\  |\
   410996 ±  0%  +4.0% 427412 ±  0%  boot-meminfo.VmallocUsed
  
   testbox/testcase/testparams: wsm/boot/performance-1
  
   c5e77f5216abdd1d  17cdf560f2727f687ab159707d
     --
85712 ±  0%  +2.4%  87772 ±  0%  boot-meminfo.VmallocUsed
  
   testbox/testcase/testparams: lkp-snb01/boot/performance-1
  
   c5e77f5216abdd1d  17cdf560f2727f687ab159707d
     --
   447001 ±  0%  -1.0% 442558 ±  0%  boot-slabinfo.num_objs
   125616 ±  5% -13.5% 108720 ±  4%  boot-meminfo.DirectMap4k
   347196 ±  0%  +4.7% 363612 ±  0%  boot-meminfo.VmallocUsed
  
 
  well, duh.. it changes a dynamic allocation into a static per-cpu one.
  What did you expect?
 
 
 Just FYI.  This may be the expected behavior.  If so, you have a third
 party test report.

Ah ok. I thought you thought it was a bug/regression.

Thanks
--
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/


Re: [LKP] [timer] 17cdf560f27: +4.0% boot-meminfo.VmallocUsed

2015-04-02 Thread Peter Zijlstra
On Thu, Apr 02, 2015 at 09:06:09AM +0800, Huang Ying wrote:
> FYI, we noticed the below changes on
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/vireshk/linux.git 
> timer/alloc_tvec_statically
> commit 17cdf560f2727f687ab159707d0aa591f8a2f82d ("timer: Allocate per-cpu 
> tvec_base's statically")
> 
> 
> testbox/testcase/testparams: lkp-sbx04/boot/performance-1
> 
> c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
>   --  
>  %stddev %change %stddev
>  \  |\  
> 410996 ±  0%  +4.0% 427412 ±  0%  boot-meminfo.VmallocUsed
> 
> testbox/testcase/testparams: wsm/boot/performance-1
> 
> c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
>   --  
>  85712 ±  0%  +2.4%  87772 ±  0%  boot-meminfo.VmallocUsed
> 
> testbox/testcase/testparams: lkp-snb01/boot/performance-1
> 
> c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
>   --  
> 447001 ±  0%  -1.0% 442558 ±  0%  boot-slabinfo.num_objs
> 125616 ±  5% -13.5% 108720 ±  4%  boot-meminfo.DirectMap4k
> 347196 ±  0%  +4.7% 363612 ±  0%  boot-meminfo.VmallocUsed
> 

well, duh.. it changes a dynamic allocation into a static per-cpu one.
What did you expect?
--
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/


Re: [LKP] [timer] 17cdf560f27: +4.0% boot-meminfo.VmallocUsed

2015-04-02 Thread Peter Zijlstra
On Thu, Apr 02, 2015 at 09:06:09AM +0800, Huang Ying wrote:
 FYI, we noticed the below changes on
 
 git://git.kernel.org/pub/scm/linux/kernel/git/vireshk/linux.git 
 timer/alloc_tvec_statically
 commit 17cdf560f2727f687ab159707d0aa591f8a2f82d (timer: Allocate per-cpu 
 tvec_base's statically)
 
 
 testbox/testcase/testparams: lkp-sbx04/boot/performance-1
 
 c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
   --  
  %stddev %change %stddev
  \  |\  
 410996 ±  0%  +4.0% 427412 ±  0%  boot-meminfo.VmallocUsed
 
 testbox/testcase/testparams: wsm/boot/performance-1
 
 c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
   --  
  85712 ±  0%  +2.4%  87772 ±  0%  boot-meminfo.VmallocUsed
 
 testbox/testcase/testparams: lkp-snb01/boot/performance-1
 
 c5e77f5216abdd1d  17cdf560f2727f687ab159707d  
   --  
 447001 ±  0%  -1.0% 442558 ±  0%  boot-slabinfo.num_objs
 125616 ±  5% -13.5% 108720 ±  4%  boot-meminfo.DirectMap4k
 347196 ±  0%  +4.7% 363612 ±  0%  boot-meminfo.VmallocUsed
 

well, duh.. it changes a dynamic allocation into a static per-cpu one.
What did you expect?
--
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/