public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Hazem Mohamed Abuelfotoh <abuehaze@amazon.com>
To: <clm@meta.com>
Cc: <bsegall@google.com>, <clm@fb.com>, <dietmar.eggemann@arm.com>,
	<joseph.salisbury@oracle.com>, <juri.lelli@redhat.com>,
	<linux-kernel@vger.kernel.org>, <mgorman@suse.de>,
	<mingo@redhat.com>, <peterz@infradead.org>, <rostedt@goodmis.org>,
	<vincent.guittot@linaro.org>, <vschneid@redhat.com>,
	<abuehaze@amazon.com>, <cpru@amazon.com>
Subject: Re: [REGRESSION][v6.17-rc1]sched/fair: Bump sd->max_newidle_lb_cost when newidle balance fails
Date: Fri, 10 Oct 2025 13:09:32 +0000	[thread overview]
Message-ID: <20251010130932.14768-1-abuehaze@amazon.com> (raw)
In-Reply-To: <28340138-a00e-47bc-a36f-270a01ac83b4@meta.com>

>> Hi Chris,
>> 
>> During testing, we are seeing a ~6% performance regression with the 
>> upstream stable v6.12.43 kernel (And Oracle UEK 
>> 6.12.0-104.43.4.el9uek.x86_64 kernel) when running the Phoronix 
>> pts/apache benchmark with 100 concurrent requests [0].  The regression 
>> is seen with the following hardware:
>> 
>> PROCESSOR: Intel Xeon Platinum 8167M Core Count: 8 Thread Count: 16 
>> Extensions: SSE 4.2 + AVX512CD + AVX2 + AVX + RDRAND + FSGSBASE Cache 
>> Size: 16 MB Microcode: 0x1 Core Family: Cascade Lake
>> 
>> After performing a bisect, we found that the performance regression was 
>> introduced by the following commit:
>> 
>> Stable v6.12.43: fc4289233e4b ("sched/fair: Bump sd->max_newidle_lb_cost 
>> when newidle balance fails")
>> Mainline v6.17-rc1: 155213a2aed4 ("sched/fair: Bump 
>> sd->max_newidle_lb_cost when newidle balance fails")
>> 
>> Reverting this commit causes the performance regression to not exist.
>> 
>> I was hoping to get your feedback, since you are the patch author.  Do 
>> you think gathering any additional data will help diagnose this issue?

> Hi everyone,

> Peter, we've had a collection of regression reports based on this
> change, so it sounds like we need to make it less aggressive, or maybe
> we need to make the degrading of the cost number more aggressive?

> Joe (and everyone else who has hit this), can I talk you into trying the
> drgn from
> https://lore.kernel.org/lkml/2fbf24bc-e895-40de-9ff6-5c18b74b4300@meta.com/

> I'm curious if it degrades at all or just gets stuck up high.

Hi All,

We are also seeing 20-30% performance regression on Database workloads
specifically Cassandra & Mongodb across multiple hardware platforms. We
have seen the regression on v6.1.149 & v6.12.43 and we were able to
bisect the regression to 155213a2aed4 ("sched/fair: Bump 
> sd->max_newidle_lb_cost when newidle balance fails"). 

We were able to reproduce this regression on below AWS instance types:

- c7a.4xlarge (16 4th generation AMD EPYC processors + 32 GiB RAM)
- c7i.4xlarge (16 4th Generation Intel Xeon Scalable processors + 32GiB RAM)
- c7g.4xlarge (16 AWS Arm based Graviton3 processors + 32 GiB RAM)
- c8g.4xlarge (16 AWS Arm based Graviton4 processors + 32 GiB RAM)

We will try drgn from
https://lore.kernel.org/lkml/2fbf24bc-e895-40de-9ff6-5c18b74b4300@meta.com/
 and will let you know the results. Meanwhile and given the significant
impact, Should we revert this commit on latest mainline & on impacted stable 
branches to stop the bleeding until we have a permanent fix?

Thank you.

Hazem

  parent reply	other threads:[~2025-10-10 13:10 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-06 20:23 [REGRESSION][v6.17-rc1]sched/fair: Bump sd->max_newidle_lb_cost when newidle balance fails Joseph Salisbury
2025-10-06 21:23 ` Chris Mason
2025-10-07 11:34   ` Peter Zijlstra
2025-10-10  1:04     ` [External] : " Joseph Salisbury
2025-10-10 17:09       ` Peter Zijlstra
2025-10-17 17:01         ` Joseph Salisbury
2025-10-30  7:29           ` Adam Li
2025-10-31 21:16             ` [External] : " Joseph Salisbury
2025-11-04 18:11             ` Joseph Salisbury
2025-10-30  7:22         ` Adam Li
2025-10-30  9:30           ` Peter Zijlstra
2025-10-30 20:53             ` Dietmar Eggemann
2025-10-31  2:46             ` Adam Li
2025-10-10  1:14     ` Joseph Salisbury
2025-10-07 20:22   ` [External] : " Joseph Salisbury
2025-10-10 13:09   ` Hazem Mohamed Abuelfotoh [this message]
2025-10-27 18:36   ` Josh Don

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=20251010130932.14768-1-abuehaze@amazon.com \
    --to=abuehaze@amazon.com \
    --cc=bsegall@google.com \
    --cc=clm@fb.com \
    --cc=clm@meta.com \
    --cc=cpru@amazon.com \
    --cc=dietmar.eggemann@arm.com \
    --cc=joseph.salisbury@oracle.com \
    --cc=juri.lelli@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mgorman@suse.de \
    --cc=mingo@redhat.com \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=vincent.guittot@linaro.org \
    --cc=vschneid@redhat.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