From: Vincent Guittot <vincent.guittot@linaro.org>
To: Xing Zhengjun <zhengjun.xing@linux.intel.com>
Cc: Tao Zhou <ouwen210@hotmail.com>,
linux-kernel@vger.kernel.org, Hillf Danton <hdanton@sina.com>,
kernel test robot <rong.a.chen@intel.com>,
Peter Zijlstra <peterz@infradead.org>,
Mel Gorman <mgorman@suse.de>
Subject: Re: [LKP] [sched/fair] 6c8116c914: stress-ng.mmapfork.ops_per_sec -38.0% regression
Date: Tue, 30 Jun 2020 09:43:11 +0200 [thread overview]
Message-ID: <20200630074311.GA12788@vingu-book> (raw)
In-Reply-To: <d0faca7b-641a-e0e6-db89-443d88e2b3d8@linux.intel.com>
Hi Tao,
Le lundi 15 juin 2020 à 16:14:01 (+0800), Xing Zhengjun a écrit :
>
>
> On 6/15/2020 1:18 PM, Tao Zhou wrote:
...
> I apply the patch based on v5.7, the regression still existed.
Could you try the patch below ? This patch is not a real fix because it impacts performance of others benchmarks but it will at least narrow your problem.
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 9f78eb76f6fb..a4d8614b1854 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -8915,9 +8915,9 @@ find_idlest_group(struct sched_domain *sd, struct task_struct *p, int this_cpu)
* and consider staying local.
*/
- if ((sd->flags & SD_NUMA) &&
- ((idlest_sgs.avg_load + imbalance) >= local_sgs.avg_load))
- return NULL;
+// if ((sd->flags & SD_NUMA) &&
+// ((idlest_sgs.avg_load + imbalance) >= local_sgs.avg_load))
+// return NULL;
/*
* If the local group is less loaded than the selected
--
> =========================================================================================
> tbox_group/testcase/rootfs/kconfig/compiler/nr_threads/disk/sc_pid_max/testtime/class/cpufreq_governor/ucode:
>
> lkp-bdw-ep6/stress-ng/debian-x86_64-20191114.cgz/x86_64-rhel-7.6/gcc-7/100%/1HDD/4194304/1s/scheduler/performance/0xb000038
>
> commit:
> e94f80f6c49020008e6fa0f3d4b806b8595d17d8
> 6c8116c914b65be5e4d6f66d69c8142eb0648c22
> v5.7
> c7e6d37f60da32f808140b1b7dabcc3cde73c4cc (Tao's patch)
>
> e94f80f6c4902000 6c8116c914b65be5e4d6f66d69c v5.7
> c7e6d37f60da32f808140b1b7da
> ---------------- --------------------------- ---------------------------
> ---------------------------
> %stddev %change %stddev %change %stddev %change
> %stddev
> \ | \ | \
> | \
> 819250 ± 5% -10.1% 736616 ± 8% +41.2% 1156877 ± 3%
> +43.6% 1176246 ± 5% stress-ng.futex.ops
> 818985 ± 5% -10.1% 736460 ± 8% +41.2% 1156215 ± 3%
> +43.6% 1176055 ± 5% stress-ng.futex.ops_per_sec
> 1551 ± 3% -3.4% 1498 ± 5% -4.6% 1480 ± 5%
> -14.3% 1329 ± 11% stress-ng.inotify.ops
> 1547 ± 3% -3.5% 1492 ± 5% -4.8% 1472 ± 5%
> -14.3% 1326 ± 11% stress-ng.inotify.ops_per_sec
> 11292 ± 8% -2.8% 10974 ± 8% -9.4% 10225 ± 6%
> -10.1% 10146 ± 6% stress-ng.kill.ops
> 11317 ± 8% -2.6% 11023 ± 8% -9.1% 10285 ± 5%
> -10.3% 10154 ± 6% stress-ng.kill.ops_per_sec
> 28.20 ± 4% -35.4% 18.22 -33.4% 18.77
> -27.7% 20.40 ± 9% stress-ng.mmapfork.ops_per_sec
> 2999012 ± 21% -10.1% 2696954 ± 22% -88.5% 344447 ± 11%
> -87.8% 364932 stress-ng.tee.ops_per_sec
> 7882 ± 3% -5.4% 7458 ± 4% -2.0% 7724 ± 3%
> -2.2% 7709 ± 4% stress-ng.vforkmany.ops
> 7804 ± 3% -5.2% 7400 ± 4% -2.0% 7647 ± 3%
> -2.1% 7636 ± 4% stress-ng.vforkmany.ops_per_sec
> 46745421 ± 3% -8.1% 42938569 ± 3% -5.2% 44312072 ± 4%
> -2.3% 45648193 stress-ng.yield.ops
> 46734472 ± 3% -8.1% 42926316 ± 3% -5.2% 44290338 ± 4%
> -2.4% 45627571 stress-ng.yield.ops_per_sec
>
>
>
...
> --
> Zhengjun Xing
next prev parent reply other threads:[~2020-06-30 7:43 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-21 0:47 [sched/fair] 6c8116c914: stress-ng.mmapfork.ops_per_sec -38.0% regression kernel test robot
2020-04-24 8:15 ` [LKP] " Xing Zhengjun
2020-04-24 15:16 ` [LKP] " Vincent Guittot
[not found] ` <20200425012306.13516-1-hdanton@sina.com>
2020-04-25 8:42 ` [LKP] " Vincent Guittot
[not found] ` <20200426124208.8872-1-hdanton@sina.com>
2020-04-27 9:03 ` Vincent Guittot
[not found] ` <20200427113533.4688-1-hdanton@sina.com>
2020-04-27 12:46 ` Vincent Guittot
2020-06-12 7:59 ` Xing Zhengjun
[not found] ` <BL0PR14MB377940B17C0889D725FF78599A9C0@BL0PR14MB3779.namprd14.prod.outlook.com>
2020-06-15 8:14 ` Xing Zhengjun
2020-06-30 7:43 ` Vincent Guittot [this message]
[not found] ` <BL0PR14MB377920305EDF3047DCAF8B719A6F0@BL0PR14MB3779.namprd14.prod.outlook.com>
2020-06-30 14:22 ` Vincent Guittot
[not found] ` <BL0PR14MB37795DCF260BAF4EB2648BE89A650@BL0PR14MB3779.namprd14.prod.outlook.com>
2020-07-10 12:48 ` Vincent Guittot
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=20200630074311.GA12788@vingu-book \
--to=vincent.guittot@linaro.org \
--cc=hdanton@sina.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mgorman@suse.de \
--cc=ouwen210@hotmail.com \
--cc=peterz@infradead.org \
--cc=rong.a.chen@intel.com \
--cc=zhengjun.xing@linux.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