linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Uladzislau Rezki <urezki@gmail.com>
To: kernel test robot <oliver.sang@intel.com>
Cc: Uladzislau Rezki <urezki@gmail.com>,
	oe-lkp@lists.linux.dev, lkp@intel.com,
	Linux Memory Management List <linux-mm@kvack.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Baoquan He <bhe@redhat.com>, Christoph Hellwig <hch@lst.de>,
	Dave Chinner <david@fromorbit.com>,
	Joel Fernandes <joel@joelfernandes.org>,
	Kazuhito Hagio <k-hagio-ab@nec.com>,
	"Liam R. Howlett" <Liam.Howlett@oracle.com>,
	Lorenzo Stoakes <lstoakes@gmail.com>,
	Matthew Wilcox <willy@infradead.org>,
	Oleksiy Avramchenko <oleksiy.avramchenko@sony.com>,
	"Paul E. McKenney" <paulmck@kernel.org>,
	ying.huang@intel.com, feng.tang@intel.com, fengwei.yin@intel.com
Subject: Re: [linux-next:master] [mm]  8f33a2ff30: stress-ng.resched.ops_per_sec -10.3% regression
Date: Thu, 29 Feb 2024 19:21:32 +0100	[thread overview]
Message-ID: <ZeDLLMAix0-zAe7J@pc636> (raw)
In-Reply-To: <202402292306.8520763a-oliver.sang@intel.com>

Hello.

> 
> 
> Hello,
> 
> kernel test robot noticed a -10.3% regression of stress-ng.resched.ops_per_sec on:
> 
> 
> commit: 8f33a2ff307248c3e55a7696f60b3658b28edb57 ("mm: vmalloc: set nr_nodes based on CPUs in a system")
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
> 
> testcase: stress-ng
> test machine: 64 threads 2 sockets Intel(R) Xeon(R) Gold 6346 CPU @ 3.10GHz (Ice Lake) with 256G memory
> parameters:
> 
> 	nr_threads: 100%
> 	testtime: 60s
> 	test: resched
> 	cpufreq_governor: performance
> 
> 
> In addition to that, the commit also has significant impact on the following tests:
> 
> +------------------+-------------------------------------------------------------------------------------------+
> | testcase: change | stress-ng: stress-ng.pthread.ops_per_sec 23.0% improvement                                |
> | test machine     | 64 threads 2 sockets Intel(R) Xeon(R) Gold 6346 CPU @ 3.10GHz (Ice Lake) with 256G memory |
> | test parameters  | cpufreq_governor=performance                                                              |
> |                  | nr_threads=100%                                                                           |
> |                  | test=pthread                                                                              |
> |                  | testtime=60s                                                                              |
> +------------------+-------------------------------------------------------------------------------------------+
> | testcase: change | stress-ng: stress-ng.fstat.ops_per_sec 14.2% improvement                                  |
> | test machine     | 64 threads 2 sockets Intel(R) Xeon(R) Gold 6346 CPU @ 3.10GHz (Ice Lake) with 256G memory |
> | test parameters  | cpufreq_governor=performance                                                              |
> |                  | disk=1HDD                                                                                 |
> |                  | fs=xfs                                                                                    |
> |                  | nr_threads=100%                                                                           |
> |                  | test=fstat                                                                                |
> |                  | testtime=60s                                                                              |
> +------------------+-------------------------------------------------------------------------------------------+
> 
This is good if i understand this correctly.

> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <oliver.sang@intel.com>
> | Closes: https://lore.kernel.org/oe-lkp/202402292306.8520763a-oliver.sang@intel.com
> 
> 
> Details are as below:
> -------------------------------------------------------------------------------------------------->
> 
> 
> The kernel config and materials to reproduce are available at:
> https://download.01.org/0day-ci/archive/20240229/202402292306.8520763a-oliver.sang@intel.com
> 
> =========================================================================================
> compiler/cpufreq_governor/kconfig/nr_threads/rootfs/tbox_group/test/testcase/testtime:
>   gcc-12/performance/x86_64-rhel-8.3/100%/debian-12-x86_64-20240206.cgz/lkp-icl-2sp7/resched/stress-ng/60s
> 
> commit: 
>   8e1d743f2c ("mm: vmalloc: support multiple nodes in vmallocinfo")
>   8f33a2ff30 ("mm: vmalloc: set nr_nodes based on CPUs in a system")
> 
> 8e1d743f2c2671aa 8f33a2ff307248c3e55a7696f60 
> ---------------- --------------------------- 
>          %stddev     %change         %stddev
>              \          |                \  
>       7.48            -0.8        6.73        mpstat.cpu.all.nice%
>   10439977           -10.4%    9351864        vmstat.system.cs
>   14670714 ±  3%     +18.1%   17330709 ±  5%  numa-numastat.node0.local_node
>   14688319 ±  3%     +18.1%   17348214 ±  5%  numa-numastat.node0.numa_hit
>   14538034 ±  3%     +15.7%   16824234 ±  4%  numa-numastat.node1.local_node
>   14556613 ±  3%     +15.6%   16834659 ±  4%  numa-numastat.node1.numa_hit
>   14685240 ±  3%     +18.0%   17334251 ±  5%  numa-vmstat.node0.numa_hit
>   14667635 ±  3%     +18.1%   17316745 ±  5%  numa-vmstat.node0.numa_local
>   14551744 ±  3%     +15.6%   16815047 ±  4%  numa-vmstat.node1.numa_hit
>   14533165 ±  3%     +15.6%   16804623 ±  4%  numa-vmstat.node1.numa_local
>  9.153e+08           -10.3%  8.208e+08        stress-ng.resched.ops
>   15220752           -10.3%   13651349        stress-ng.resched.ops_per_sec
>  6.584e+08           -10.8%  5.871e+08        stress-ng.time.involuntary_context_switches
>
This is not. I am working on it to figure out what happens.

--
Uladzislau Rezki


  reply	other threads:[~2024-02-29 18:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-29 16:01 kernel test robot
2024-02-29 18:21 ` Uladzislau Rezki [this message]
2024-03-04  9:06 ` Uladzislau Rezki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZeDLLMAix0-zAe7J@pc636 \
    --to=urezki@gmail.com \
    --cc=Liam.Howlett@oracle.com \
    --cc=akpm@linux-foundation.org \
    --cc=bhe@redhat.com \
    --cc=david@fromorbit.com \
    --cc=feng.tang@intel.com \
    --cc=fengwei.yin@intel.com \
    --cc=hch@lst.de \
    --cc=joel@joelfernandes.org \
    --cc=k-hagio-ab@nec.com \
    --cc=linux-mm@kvack.org \
    --cc=lkp@intel.com \
    --cc=lstoakes@gmail.com \
    --cc=oe-lkp@lists.linux.dev \
    --cc=oleksiy.avramchenko@sony.com \
    --cc=oliver.sang@intel.com \
    --cc=paulmck@kernel.org \
    --cc=willy@infradead.org \
    --cc=ying.huang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox