public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

  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