Date:   Tue, 14 Mar 2023 12:09:37 +0800
From:   Chen Yu <yu.c.chen@intel.com>
To:     K Prateek Nayak <kprateek.nayak@amd.com>
CC:     Peter Zijlstra <peterz@infradead.org>,
        Vincent Guittot <vincent.guittot@linaro.org>,
        Ingo Molnar <mingo@redhat.com>,
        Juri Lelli <juri.lelli@redhat.com>,
        Mel Gorman <mgorman@techsingularity.net>,
        Tim Chen <tim.c.chen@intel.com>,
        Dietmar Eggemann <dietmar.eggemann@arm.com>,
        "Steven Rostedt" <rostedt@goodmis.org>,
        Ben Segall <bsegall@google.com>,
        Abel Wu <wuyun.abel@bytedance.com>,
        Yicong Yang <yangyicong@hisilicon.com>,
        "Gautham R . Shenoy" <gautham.shenoy@amd.com>,
        Honglei Wang <wanghonglei@didichuxing.com>,
        Len Brown <len.brown@intel.com>,
        Chen Yu <yu.chen.surf@gmail.com>,
        Tianchen Ding <dtcccc@linux.alibaba.com>,
        "Joel Fernandes" <joel@joelfernandes.org>,
        Josh Don <joshdon@google.com>,
        "Hillf Danton" <hdanton@sina.com>,
        kernel test robot <yujie.liu@intel.com>,
        "Arjan Van De Ven" <arjan.van.de.ven@intel.com>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v6 0/2] sched/fair: Wake short task on current CPU
Message-ID: <ZA/zgYzmFxht0Kbz@chenyu5-mobl1>
References: <cover.1677069490.git.yu.c.chen@intel.com>
 <28659855-c1a4-ab61-433e-a5bccb96fcbd@amd.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <28659855-c1a4-ab61-433e-a5bccb96fcbd@amd.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?kEx5YTRKqlOvP6Qr8U4GXCtl5JeNdP7s7ArRYEn0atbSLW2ZKI2CxktkNX4m?=
 =?us-ascii?Q?G+qaZZ5LnqL1nlV5QG5i3FlzsEBQt3GMqVhTAZrc+nN3uUBj9Fi9FNy/TB+9?=
 =?us-ascii?Q?HXOlH0GrDHE0N3Ug6z6oJX/Mjw2isX/oBmvOiByKxaPi3I+1Prv4NLjO7DTF?=
 =?us-ascii?Q?WVGslEfoNaN6Y+5+Ztp8xDmhdJrZtnQuy/owmXlKFzfph/V4KcmhiXUkDkqx?=
 =?us-ascii?Q?wK43VmoeqtL4Zv+VK8GGar5vFhLYTBBlwZPF/F1Om/hpLFhRcIOpdCCSs+7+?=
 =?us-ascii?Q?MqB0ezrmH18b4+tiCjdVRXw2R7TMzphGm/AwI+3u+zUwBXaJerA2GJyX4mF2?=
 =?us-ascii?Q?JAQTfi+m+9oIMjkJSV/mmVeSNmclUmBMoUD8jurKoERW55OHr6F1qH5KbC/a?=
 =?us-ascii?Q?jLyZlsgcoVxFPXSsJSIv5GOrN8SDd/LkFX6HpKmgH2CSPjFtxaysnSQjnEWd?=
 =?us-ascii?Q?K/6i0d8fDHPC3uqa3x5UDhGFGfZT5/c+cPisezWuvKpaayKZc/jGlOccmveH?=
 =?us-ascii?Q?oQ9wpPG6St5QOAd+eTjQSED5gZ3bp+OQ08UFog8a+Knpaz7c7OLxhf+OBBnp?=
 =?us-ascii?Q?RvwxyrOYznujNGKFTjSLsF0KHfVMsVLB6p+mTiVUIbYzBinwUXUiwleaM3an?=
 =?us-ascii?Q?1r8AHfpMbCShB3B6YTN47ukp2owaGaZf8rZlmjzxl57C/bgUnLWNgQQBQ1v3?=
 =?us-ascii?Q?c2sSODXc3o+weDYfN2ptRt17eHLLonFeqp9bd7cRdwqH33lshV264l6tXqMK?=
 =?us-ascii?Q?d9jVJ8eXEsCCbn64VYxe1ScNBdW5Jt8/nrI9CfyLsKIA6gVe4ZZDy0cdX1dm?=
 =?us-ascii?Q?lDUm7oWz6iLcd9h2qNHt1ii41Qn4uaytQqs1ECk1yx1M8Hi1S8aZoECrdUun?=
 =?us-ascii?Q?hOTzjldIwYH7TEtGVYQmLLa/UZq6TLkdpSd6fD9Igx1jrtU9ztAN+fjnEJgO?=
 =?us-ascii?Q?yOqLhsTRxxz5FGW+EkR9yWEg5pYASlS3FSe4/Qe+JSTXC7OHjeCmJGVs09M8?=
 =?us-ascii?Q?z+udA3Dbh2Nr1hqGFQzDr5KbW4enxm/PY64y7ek4p0u+1qgZGBmCpCVX4lMu?=
 =?us-ascii?Q?JbIXckMFCg72EE7zB4hWNBtSFEwUMCHoiNhiM7sSJhvQ4EV44eAnT4Xzb0h3?=
 =?us-ascii?Q?wmMbqRj2jfiMf7TOcCU2th+FXazilQHu7/zNjkTdjQKydgX++0dhzvdT0t1g?=
 =?us-ascii?Q?hUX+zQhbsvIWnfA/rAei7OIuVay6uDWmYmT5RxKUtoEjQVjMLVIvViiOVOYj?=
 =?us-ascii?Q?UNNVgG0Rp3aBNTC7OjCqFO0c1FzxW2Mop8qapSJUtzxvFjjjAwuNwUjO43gU?=
 =?us-ascii?Q?zju6BmfStDhfd5vEZHJtCkjxXCsbikO4gvLzOq7Y4fiIwyRVRfzWkanNHXT7?=
 =?us-ascii?Q?Pj4RXXezrzNOwiRGG7LwRSOjeWFhjbiwqQIuDacVurUL7KR1Df8DQ9tqcjks?=
 =?us-ascii?Q?M74cS9rUaM/qGokYnHL6+OJF20QsTT/OiLXGVszP6xmfAF8afU0UApmTRZbJ?=
 =?us-ascii?Q?I1knB+zb8KcvUCZKO1OU+c15CroxjfMZWYoRka5QsQADIMXSJssqkDMNdgTL?=
 =?us-ascii?Q?pMsjqR3x3dEvAOjc+UbKf5IjMO9OA2CI5HxIgJKn?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 3736c4fb-57ee-4117-5b7f-08db2441f8d9
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6206.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Mar 2023 04:09:57.2038
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: nvEt3VUkXSEWhgWkTtCsvXx8/AfHTtEpD8+z0ZdLO1EgTF2Kgo8MGvWOLF3dSgzbd6N/duQoFkAV0qjBmiJBpg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB6218
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2023-03-14 at 08:43:17 +0530, K Prateek Nayak wrote:
> Hello Chenyu,
> 
> I did not observe any regression when testing v6 :)
> Most of the benchmark results are comparable to the tip with minor
> gains in some benchmarks with single sender and single receiver.
> 
> With the introduction of wakee_flips condition in is_short_task(),
> most benchmarks that have multiple tasks interacting are now
> comparable to tip. There may be other avenues for optimization
> using SIS_SHORT as Abel pointed but this series is a good start in
> that direction. 
> 
> Tested-by: K Prateek Nayak <kprateek.nayak@amd.com>
Thanks Prateek!

thanks,
Chenyu