lkml.org 
[lkml]   [2023]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] kernel/cpu: The number of CPUHP_AP_ONLINE_DYN (30) is too small that stops the new dynamic states hotplug.
Date
On Tue, Dec 12 2023 at 10:52, wangxiaoming321 wrote:

The subject line has a two issues:

1) the prefix is wrong. See
https://www.kernel.org/doc/html/latest/process/maintainer-tip.html#patch-subject

You find further information about changelogs there and in other
parts of the documentation

2) The sentence is too long and incomprehensible.

> 144: perf/x86/cstate:online
...
> 195: online

I have no idea what this number salad is trying to tell me. Changelogs
need to explain the problem they are trying to solve. This listing of
hotplug states is not an explanation and completely useless as it even
contains an entry which do not exist:

> 191: ia64/sep5:online

Even if it would exist a ia64 hotplug callback is hardly relevant on x86,
right?

Also the fixed entries e.g. workqueue and RCU/tree are irrelevant for
what you are trying to describe.

So what you really want to say is something like this:

cpu/hotplug: Increase the number of dynamic states

The dynamically allocatable hotplug state space can be exhausted by
the existing drivers and infrastructure which install CPU hotplug
states dynamically. That prevents new drivers and infrastructure from
installing dynamically allocated states.

Increase the size of the CPUHP_AP_ONLINE_DYN state by 10 to make
room.

At least that's what I oracled out of your "changelog". Hmm?

Thanks,

tglx



\
 
 \ /
  Last update: 2023-12-15 20:40    [W:0.047 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site