[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251128093129.GD3245006@noisy.programming.kicks-ass.net>
Date: Fri, 28 Nov 2025 10:31:29 +0100
From: Peter Zijlstra <peterz@...radead.org>
To: Huang Shijie <shijie@...amperecomputing.com>
Cc: mingo@...hat.com, juri.lelli@...hat.com, vincent.guittot@...aro.org,
patches@...erecomputing.com, cl@...ux.com,
Shubhang@...amperecomputing.com, dietmar.eggemann@....com,
rostedt@...dmis.org, bsegall@...gle.com, mgorman@...e.de,
linux-kernel@...r.kernel.org, vschneid@...hat.com,
vineethr@...ux.ibm.com, kprateek.nayak@....com
Subject: Re: [PATCH v4 1/2] sched/fair: set rq->idle_stamp at the end of the
sched_balance_newidle
On Fri, Nov 28, 2025 at 03:54:53PM +0800, Huang Shijie wrote:
> Save the idle_stamp at the beginning of sched_balance_newidle(),
> if it cannot pull any task, set it for rq->idle_stamp.
>
This changelog tells me what the patch does, something I can see from
reading the patch itself. However, it completely fails at the purpose of
a changelog, which is to elucidate the reader as to the reasons for
doing so.
Powered by blists - more mailing lists