Received-SPF: pass (google.com: domain of linux-kernel+bounces-4643-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: "Wang, Xiaoming" <xiaoming.wang@intel.com>
To: Thomas Gleixner <tglx@linutronix.de>, "peterz@infradead.org"
	<peterz@infradead.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>
CC: "Wang, Xiaoming" <xiaoming.wang@intel.com>
Subject: RE: [PATCH] kernel/cpu: The number of CPUHP_AP_ONLINE_DYN (30) is too
 small that stops the new dynamic states hotplug.
Thread-Topic: [PATCH] kernel/cpu: The number of CPUHP_AP_ONLINE_DYN (30) is
 too small that stops the new dynamic states hotplug.
Thread-Index: AQHaLKaGkoT+yzskUkKe6mEiubwNYLCqw56AgAUpVQA=
Date: Tue, 19 Dec 2023 03:30:05 +0000
Message-ID: <DM6PR11MB31646E5034B747314510C9E6EC97A@DM6PR11MB3164.namprd11.prod.outlook.com>
References: <20231212025235.775986-1-xiaoming.wang@intel.com>
 <87o7ermf0c.ffs@tglx>
In-Reply-To: <87o7ermf0c.ffs@tglx>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?nhgDWO/fvgP4NibACbJUuUdwWPOdOxkOO/aWN0NVt33M/m0covWuL0jzefS6?=
 =?us-ascii?Q?XnOrzczA0It454yZEHfqTUQhGGdGaLQyf4oc8bfdzLub5D5UFLWL9Ld1vNQD?=
 =?us-ascii?Q?E2ch08bzm0jd5ra9UNqWmeVrIEECtwMFZYCWQcUe4Qtma1fNUw/7K+cSEHH2?=
 =?us-ascii?Q?BzHx05Y3dW9g7Y7Jhl4uzx4LGJS0TTI87BBtxrSTVcrZFc3Hll2OV6qZnrH3?=
 =?us-ascii?Q?JT9+qZab4sENdGK17cujx90ofWyVAJ/S2d5DX97urCEpGoEjWjWZVttf5f9Y?=
 =?us-ascii?Q?jKwwmDElZFqFdIFRwTESTGOtrGSPBAmjUY+Cgd+2LprSyGHRCzFuKgGAlZSn?=
 =?us-ascii?Q?XbojicHZuRfbdheQoRy+7wdPjjZMDBF7DZwZZ2OLVQ+p8lNCLf5mbDaZjMtj?=
 =?us-ascii?Q?5qkx55gQScyo1Biw/xSZ4uZbOLEmBbQLkMbV3lRALO5A312ewazwVSEHLy+v?=
 =?us-ascii?Q?yDGxB5wG7ITsLO0eqcY4GFjnlvqwr7gV2BKVBuokq5v+1g1aZD8uEr75dPHz?=
 =?us-ascii?Q?G87ZPk/G0SDin6gTHvb/O8OzHjiHpnCArBDByzre4FEzaFnGPhv9zvFXU7Lr?=
 =?us-ascii?Q?kX8FHSIq6XiV27uR9OVq2n0KtmbrLusqS/PkK3C2hYkXNORvVfQtDRYe5tY6?=
 =?us-ascii?Q?oMwKprufSst3qRKPeM8moAhGeDs9NUNMhIe+i00KnWJmrqA+NWh5dKzL3v6G?=
 =?us-ascii?Q?rUAbMa4nuhsl04C28O6XBPveS2PQ6NC+3bN0THRWat9dju8rxzfGD1vqxmV1?=
 =?us-ascii?Q?0z+cIWdiXr0EhU9dFJLeJVFEi2vCm22op38CNStvYNnTEDo+uMCowFx5eJAZ?=
 =?us-ascii?Q?XbjyM6pbMk3Pl9+FUC1AnbU5HBUydLUguT11dvObI4F4nsRIjxcEH2QY9Vj7?=
 =?us-ascii?Q?uTg0K2Uj0VxQXZ/3v+HQLTlPspQ6DrePfF+2N04x0Nx8OyTdpEWq4MZL44WB?=
 =?us-ascii?Q?KUsZ33BiZ0RlSbbC7mmUV3QPbfU0GsuxAulTnRoYj5W5mJDbojOOBqt8gX9B?=
 =?us-ascii?Q?OPCH2bb4yRwFUtq5l4RbHpa8xorAIwTA0vhQojU2NvlaBGl0Kea4bmK+4t1Z?=
 =?us-ascii?Q?ns6I/J2CvdnRgCUKlV9qYWO4FcUPBBRTzKShN6ViRMQx0TraXNcYeuXv72C6?=
 =?us-ascii?Q?grK2WHAlxde2vThvFPXm6OLQf6HiqZ4bGXHgihfN9M5Xab8KEpvvstVInAbp?=
 =?us-ascii?Q?lAB0+o7IHghVEzBT2c7p7TU8+wAJmtMu8joZlYpCWIJmKRkYKIQSR6O2SzcJ?=
 =?us-ascii?Q?lZd7JtcjdoOKO/WXs9wdEhEf/z6utmCMW+8FTaqYOg/y5kR1PaVrA2L7ULze?=
 =?us-ascii?Q?grLIOdomYILqdnjcA5euvsBR20Ph6RsrIy3bFT1cfQ6XPBEI5EfVlGrz6IPy?=
 =?us-ascii?Q?oRnK3fl/3VSj2efQKa+jaLJW3OzXAIRNae/Mg2ykBQBooVi+gamCKfl5sp9s?=
 =?us-ascii?Q?6H4jx6Rt0c+HQD7ZdCWJNQnxKIg9I9AiANLrKiHjnnFLt0OjQ6M7V8VO3q4Y?=
 =?us-ascii?Q?IK/gydAVhMSGsxKeFVD0TKy5uUaMofbWCIL91n35UbaxTb2hi9qW8Y0BqTfJ?=
 =?us-ascii?Q?GNd8+WaW2jCDVvrYuS6tnQIle/mxZCH+AFqTNrYC?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3164.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9f2e0963-2c78-47c4-9548-08dc0042cb2e
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2023 03:30:05.6063
 (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: qRQSHiyrSAgdJrQpm84nnxIlDYRftaluastooMYrDdlDaYYCqSe0szCX6NP2YO0Rk53QXwh9epvPHHFMOZuISw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB5179
X-OriginatorOrg: intel.com

Thank you for your response Thomas!

You get my opinion exactly.
I will modify the patch under your guide.

> 191: ia64/sep5:online
I do not know what this dynamical state mean either.
Maybe this is an irregular naming driver for X86.

I list them by cat /sys/devices/system/cpu/hotplug/states | grep online
And find the dynamical state is over 30.
So I think it need to enlarge the room for dynamic states.
Do you think this make sense?

-----Original Message-----
From: Thomas Gleixner <tglx@linutronix.de>=20
Sent: Saturday, December 16, 2023 3:40 AM
To: Wang, Xiaoming <xiaoming.wang@intel.com>; peterz@infradead.org; linux-k=
ernel@vger.kernel.org
Cc: Wang, Xiaoming <xiaoming.wang@intel.com>
Subject: Re: [PATCH] kernel/cpu: The number of CPUHP_AP_ONLINE_DYN (30) is =
too small that stops the new dynamic states hotplug.

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#p=
atch-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 s=
tates is not an explanation and completely useless as it even contains an e=
ntry which do not exist:

> 191: ia64/sep5:online

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

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

Thanks
Xiaoming
  =20