From: Vincent Guittot <vincent.guittot@linaro.org>
To: Xing Zhengjun <zhengjun.xing@linux.intel.com>
Cc: Hillf Danton <hdanton@sina.com>,
kernel test robot <rong.a.chen@intel.com>,
Ingo Molnar <mingo@kernel.org>,
Mel Gorman <mgorman@techsingularity.net>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Juri Lelli <juri.lelli@redhat.com>,
Steven Rostedt <rostedt@goodmis.org>,
Valentin Schneider <valentin.schneider@arm.com>,
Phil Auld <pauld@redhat.com>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [LKP] [sched/fair] 070f5e860e: reaim.jobs_per_min -10.5% regression
Date: Tue, 16 Jun 2020 08:54:32 +0200 [thread overview]
Message-ID: <20200616065432.GA18401@vingu-book> (raw)
In-Reply-To: <d60343e9-b3a0-bdc7-84f4-e8c912f92c46@linux.intel.com>
Hi Xing,
Le mardi 16 juin 2020 à 11:17:16 (+0800), Xing Zhengjun a écrit :
>
>
> On 6/15/2020 4:10 PM, Vincent Guittot wrote:
> > Hi Xing,
> >
> > Le lundi 15 juin 2020 à 15:26:59 (+0800), Xing Zhengjun a écrit :
> > >
> > >
> > > On 6/12/2020 7:06 PM, Hillf Danton wrote:
> > > >
> > > > On Fri, 12 Jun 2020 14:36:49 +0800 Xing Zhengjun wrote:
> >
...
> >
>
> I apply the patch based on v5.7, the test result is as the following:
TBH, I didn't expect that the results would still be bad, so i wonder if the threshold are
the root problem.
Could you run tests with the patch below that removes condition with runnable_avg ?
I just want to make sure that those 2 conditions are the root cause.
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index da3e5b54715b..f5774d0af059 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -8210,10 +8210,6 @@ group_has_capacity(unsigned int imbalance_pct, struct sg_lb_stats *sgs)
if (sgs->sum_nr_running < sgs->group_weight)
return true;
- if ((sgs->group_capacity * imbalance_pct) <
- (sgs->group_runnable * 100))
- return false;
-
if ((sgs->group_capacity * 100) >
(sgs->group_util * imbalance_pct))
return true;
@@ -8239,10 +8235,6 @@ group_is_overloaded(unsigned int imbalance_pct, struct sg_lb_stats *sgs)
(sgs->group_util * imbalance_pct))
return true;
- if ((sgs->group_capacity * imbalance_pct) <
- (sgs->group_runnable * 100))
- return true;
-
return false;
}
Thanks.
Vincent
>
> =========================================================================================
> tbox_group/testcase/rootfs/kconfig/compiler/runtime/nr_task/debug-setup/test/cpufreq_governor/ucode:
>
> lkp-ivb-d04/reaim/debian-x86_64-20191114.cgz/x86_64-rhel-7.6/gcc-7/300s/100%/test/five_sec/performance/0x21
>
> commit:
> 9f68395333ad7f5bfe2f83473fed363d4229f11c
> 070f5e860ee2bf588c99ef7b4c202451faa48236
> v5.7
> 3e1643da53f3fc7414cfa3ad2a16ab2a164b7f4d (the test patch)
>
> 9f68395333ad7f5b 070f5e860ee2bf588c99ef7b4c2 v5.7
> 3e1643da53f3fc7414cfa3ad2a1
> ---------------- --------------------------- ---------------------------
> ---------------------------
> %stddev %change %stddev %change %stddev %change
> %stddev
> \ | \ | \
> | \
> 0.69 -10.3% 0.62 -9.1% 0.62
> -7.1% 0.64 reaim.child_systime
> 0.62 -1.0% 0.61 +0.5% 0.62
> +1.3% 0.63 reaim.child_utime
> 66870 -10.0% 60187 -7.6% 61787
> -6.1% 62807 reaim.jobs_per_min
> 16717 -10.0% 15046 -7.6% 15446
> -6.1% 15701 reaim.jobs_per_min_child
> 97.84 -1.1% 96.75 -0.4% 97.43
> -0.5% 97.34 reaim.jti
> 72000 -10.8% 64216 -8.3% 66000
> -5.7% 67885 reaim.max_jobs_per_min
> 0.36 +10.6% 0.40 +7.8% 0.39
> +6.9% 0.38 reaim.parent_time
> 1.58 ± 2% +71.0% 2.70 ± 2% +26.9% 2.01 ± 2%
> +32.5% 2.09 ± 6% reaim.std_dev_percent
> 0.00 ± 5% +110.4% 0.01 ± 3% +48.8% 0.01 ± 7%
> +61.7% 0.01 ± 8% reaim.std_dev_time
> 50800 -2.4% 49600 -1.6% 50000
> -1.3% 50133 reaim.workload
>
>
> > >
> > > =========================================================================================
> > > tbox_group/testcase/rootfs/kconfig/compiler/runtime/nr_task/debug-setup/test/cpufreq_governor/ucode:
> > >
> > > lkp-ivb-d04/reaim/debian-x86_64-20191114.cgz/x86_64-rhel-7.6/gcc-7/300s/100%/test/five_sec/performance/0x21
> > >
> > > commit:
> > > 9f68395333ad7f5bfe2f83473fed363d4229f11c
> > > 070f5e860ee2bf588c99ef7b4c202451faa48236
> > > v5.7
> > > 6b33257768b8dd3982054885ea310871be2cfe0b (Hillf's patch)
> > >
> > > 9f68395333ad7f5b 070f5e860ee2bf588c99ef7b4c2 v5.7
> > > 6b33257768b8dd3982054885ea3
> > > ---------------- --------------------------- ---------------------------
> > > ---------------------------
> > > %stddev %change %stddev %change %stddev %change
> > > %stddev
> > > \ | \ | \
> > > | \
> > > 0.69 -10.3% 0.62 -9.1% 0.62
> > > -10.1% 0.62 reaim.child_systime
> > > 0.62 -1.0% 0.61 +0.5% 0.62
> > > +0.3% 0.62 reaim.child_utime
> > > 66870 -10.0% 60187 -7.6% 61787
> > > -8.3% 61305 reaim.jobs_per_min
> > > 16717 -10.0% 15046 -7.6% 15446
> > > -8.3% 15326 reaim.jobs_per_min_child
> > > 97.84 -1.1% 96.75 -0.4% 97.43
> > > -0.5% 97.37 reaim.jti
> > > 72000 -10.8% 64216 -8.3% 66000
> > > -8.3% 66000 reaim.max_jobs_per_min
> > > 0.36 +10.6% 0.40 +7.8% 0.39
> > > +9.4% 0.39 reaim.parent_time
> > > 1.58 ± 2% +71.0% 2.70 ± 2% +26.9% 2.01 ± 2%
> > > +33.2% 2.11 reaim.std_dev_percent
> > > 0.00 ± 5% +110.4% 0.01 ± 3% +48.8% 0.01 ± 7%
> > > +65.3% 0.01 ± 3% reaim.std_dev_time
> > > 50800 -2.4% 49600 -1.6% 50000
> > > -1.8% 49866 reaim.workload
> > >
> > >
> > >
> > > --
> > > Zhengjun Xing
>
> --
> Zhengjun Xing
next prev parent reply other threads:[~2020-06-16 6:54 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-19 2:38 [sched/fair] 070f5e860e: reaim.jobs_per_min -10.5% regression kernel test robot
2020-03-19 2:38 ` kernel test robot
2020-06-12 6:36 ` Xing Zhengjun
2020-06-12 6:36 ` [LKP] " Xing Zhengjun
2020-06-12 7:07 ` Vincent Guittot
2020-06-12 7:07 ` [LKP] " Vincent Guittot
2020-06-12 15:19 ` Vincent Guittot
2020-06-12 15:19 ` [LKP] " Vincent Guittot
2020-06-15 7:37 ` Xing Zhengjun
2020-06-15 7:37 ` [LKP] " Xing Zhengjun
[not found] ` <20200612110616.20264-1-hdanton@sina.com>
2020-06-12 15:23 ` Vincent Guittot
2020-06-15 7:26 ` Xing Zhengjun
2020-06-15 8:10 ` Vincent Guittot
2020-06-16 3:17 ` Xing Zhengjun
2020-06-16 6:54 ` Vincent Guittot [this message]
2020-06-17 0:30 ` Xing Zhengjun
2020-06-17 14:57 ` Vincent Guittot
2020-06-17 16:25 ` Vincent Guittot
2020-06-18 2:45 ` Xing Zhengjun
2020-06-18 12:35 ` Vincent Guittot
2020-06-19 5:01 ` Xing Zhengjun
[not found] ` <20200618082406.8292-1-hdanton@sina.com>
2020-06-19 5:10 ` Xing Zhengjun
2020-06-19 4:55 ` Xing Zhengjun
2020-06-19 7:15 ` Vincent Guittot
2020-06-24 9:04 ` Vincent Guittot
[not found] ` <20200615151030.6480-1-hdanton@sina.com>
2020-06-16 3:24 ` Xing Zhengjun
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=20200616065432.GA18401@vingu-book \
--to=vincent.guittot@linaro.org \
--cc=a.p.zijlstra@chello.nl \
--cc=hdanton@sina.com \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mgorman@techsingularity.net \
--cc=mingo@kernel.org \
--cc=pauld@redhat.com \
--cc=rong.a.chen@intel.com \
--cc=rostedt@goodmis.org \
--cc=valentin.schneider@arm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.