From: Xing Zhengjun <zhengjun.xing@linux.intel.com>
To: Vincent Guittot <vincent.guittot@linaro.org>
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: Wed, 17 Jun 2020 08:30:21 +0800 [thread overview]
Message-ID: <3de4937b-1dcd-6d2e-836f-fed1c295dd7c@linux.intel.com> (raw)
In-Reply-To: <20200616065432.GA18401@vingu-book>
On 6/16/2020 2:54 PM, Vincent Guittot wrote:
>
> 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
>
I apply the patch based on v5.7, the test result is as the following:
=========================================================================================
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
63a5d0fbb5ec62f5148c251c01e709b8358cd0ee (the test patch)
9f68395333ad7f5b 070f5e860ee2bf588c99ef7b4c2 v5.7
63a5d0fbb5ec62f5148c251c01e
---------------- --------------------------- ---------------------------
---------------------------
%stddev %change %stddev %change
%stddev %change %stddev
\ | \ | \
| \
0.69 -10.3% 0.62 -9.1% 0.62
+1.0% 0.69 reaim.child_systime
0.62 -1.0% 0.61 +0.5% 0.62
-0.1% 0.62 reaim.child_utime
66870 -10.0% 60187 -7.6% 61787
+1.1% 67636 reaim.jobs_per_min
16717 -10.0% 15046 -7.6% 15446
+1.1% 16909 reaim.jobs_per_min_child
97.84 -1.1% 96.75 -0.4% 97.43
+0.3% 98.09 reaim.jti
72000 -10.8% 64216 -8.3% 66000
+0.0% 72000 reaim.max_jobs_per_min
0.36 +10.6% 0.40 +7.8% 0.39
-1.1% 0.36 reaim.parent_time
1.58 ± 2% +71.0% 2.70 ± 2% +26.9% 2.01 ±
2% -11.9% 1.39 ± 4% reaim.std_dev_percent
0.00 ± 5% +110.4% 0.01 ± 3% +48.8% 0.01 ±
7% -27.3% 0.00 ± 15% reaim.std_dev_time
50800 -2.4% 49600 -1.6% 50000
+0.0% 50800 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
>> 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
--
Zhengjun Xing
next prev parent reply other threads:[~2020-06-17 0:30 UTC|newest]
Thread overview: 21+ 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-06-12 6:36 ` [LKP] " Xing Zhengjun
2020-06-12 7:07 ` Vincent Guittot
2020-06-12 15:19 ` Vincent Guittot
2020-06-15 7:37 ` 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
2020-06-17 0:30 ` Xing Zhengjun [this message]
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=3de4937b-1dcd-6d2e-836f-fed1c295dd7c@linux.intel.com \
--to=zhengjun.xing@linux.intel.com \
--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=vincent.guittot@linaro.org \
/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