Messages in this thread | | | From | Qais Yousef <> | Subject | [PATCH 0/2] sched: Optionally skip uclamp logic in fast path | Date | Thu, 18 Jun 2020 20:55:23 +0100 |
| |
This series attempts to address the report that uclamp logic could be expensive sometimes and shows a regression in netperf UDP_STREAM under certain conditions.
The first patch is a fix for how struct uclamp_rq is initialized which is required by the 2nd patch which contains the real 'fix'.
Worth noting that the root cause of the overhead is believed to be system specific or related to potential certain code/data layout issues, leading to worse I/D $ performance.
Different systems exhibited different behaviors and the regression did disappear in certain kernel version while attempting to reporoduce.
More info can be found here:
https://lore.kernel.org/lkml/20200616110824.dgkkbyapn3io6wik@e107158-lin/
Having the static key seemed the best thing to do to ensure the effect of uclamp is minimized for kernels that compile it in but don't have a userspace that uses it, which will allow distros to distribute uclamp capable kernels by default without having to compromise on performance for some systems that could be affected.
Thanks
-- Qais Yousef
Cc: Juri Lelli <juri.lelli@redhat.com> Cc: Vincent Guittot <vincent.guittot@linaro.org> Cc: Dietmar Eggemann <dietmar.eggemann@arm.com> Cc: Steven Rostedt <rostedt@goodmis.org> Cc: Ben Segall <bsegall@google.com> Cc: Mel Gorman <mgorman@suse.de> CC: Patrick Bellasi <patrick.bellasi@matbug.net> Cc: Chris Redpath <chrid.redpath@arm.com> Cc: Lukasz Luba <lukasz.luba@arm.com> Cc: linux-kernel@vger.kernel.org
Qais Yousef (2): sched/uclamp: Fix initialization of strut uclamp_rq sched/uclamp: Protect uclamp fast path code with static key
kernel/sched/core.c | 73 +++++++++++++++++++++++++++++++++++++++------ 1 file changed, 64 insertions(+), 9 deletions(-)
-- 2.17.1
| |