Received-SPF: pass (google.com: domain of linux-kernel+bounces-47393-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <2448f18f-2f3a-4158-89c4-91415bca9eb1@intel.com>
Date: Wed, 31 Jan 2024 15:47:08 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [patch v2 14/30] x86/cpu/topology: Rework possible CPU management
Content-Language: en-US
To: Thomas Gleixner <tglx@linutronix.de>, LKML <linux-kernel@vger.kernel.org>
CC: <x86@kernel.org>, Tom Lendacky <thomas.lendacky@amd.com>, Andrew Cooper
	<andrew.cooper3@citrix.com>, Arjan van de Ven <arjan@linux.intel.com>, "Huang
 Rui" <ray.huang@amd.com>, Juergen Gross <jgross@suse.com>, Dimitri Sivanich
	<dimitri.sivanich@hpe.com>, K Prateek Nayak <kprateek.nayak@amd.com>, "Kan
 Liang" <kan.liang@linux.intel.com>, Zhang Rui <rui.zhang@intel.com>, "Paul E.
 McKenney" <paulmck@kernel.org>, Feng Tang <feng.tang@intel.com>, "Andy
 Shevchenko" <andy@infradead.org>, Michael Kelley <mhklinux@outlook.com>,
	"Peter Zijlstra (Intel)" <peterz@infradead.org>
References: <20240118123127.055361964@linutronix.de>
 <20240118123649.343911720@linutronix.de>
From: Sohil Mehta <sohil.mehta@intel.com>
In-Reply-To: <20240118123649.343911720@linutronix.de>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M3Y5R0gvTG1Fb1RtVzl1WENSa3FKNDRqTHJMRUR3Rml2VTZiWEIyRUpmVjlo?=
 =?utf-8?B?aHJ5WHBLbnAvMy9xVEtRa2hmREpzdWt1NXpyTTZvb0pJeERWblFKRzFoejVo?=
 =?utf-8?B?M1pUNHV0WXFRNUZqZTU0L1pYMmlZRDU4dVdUL25xWTJkelNnWE1raTNibkdL?=
 =?utf-8?B?U3NPUzVhU0tLYk5DVll5K2JnMWkyNGcySHFGYkc4eGFTVE45aVEwUXFaMEsy?=
 =?utf-8?B?bWMyb3paWnZVaXpDbXA4Z0IwdFdBcnVlYTZESFNMRTUvOWhWWmdueGdTS0M1?=
 =?utf-8?B?QkJqNjVWdVpOMWl3VS9rN2hiMDRnS3dmWUx3d21sUlAvOUZ4c0NIR1hpMWF1?=
 =?utf-8?B?Y0hMbGxlWi9xQ3VSbVllcHMzdExmM2QrUHpTeGZZYUZFb3hBNDBCeENmN1NB?=
 =?utf-8?B?NitWUW8yTmhsek9UTUxPSWdBNFovRW1UbUY3WVJ1ekRNekNVRmd4U2ZBR1R5?=
 =?utf-8?B?Q1pIS1FvK0tndEtnOFpZRHJMVCswRGViNWhKaEhvQkRDT2YzRkx4SDg0ZTRJ?=
 =?utf-8?B?WEdvQlV1K0x3Yzg2V21kQnhsSisyclBkWGhoUTdFQ0xhM2JkNlRQUVN6OHNy?=
 =?utf-8?B?QTlOU0Y4MmFiZ0EyM3llOU90YnJtZnQ2UGx3Yk0xdGxDL09ocDFJY2tzZ2dz?=
 =?utf-8?B?WW5RUll0aTZmNHA5VHZKZktHcnk4TURiTTFYVUc0Z2p3QjlYV1EzWUI5ejJQ?=
 =?utf-8?B?Wnc2UytFelZyZXI3OFJlV3d2WjhNVldxLzRscm1peSt5enhwR0hUS0pZZ3Mw?=
 =?utf-8?B?cldSb3crZy9rdmhVdWJyRTNkQ0VlUDdNZ0lxL1cvWmszRjBieVp3NE5zb3Nu?=
 =?utf-8?B?RWdLeWZWVnoxMm5XcHliSWF4THN6Z3pHVEVXYlRKZ3VqeVVKc2pZNFUydjh3?=
 =?utf-8?B?ejhDbzBGcUg1dnpmSHpPbEFIS1ZlSlFlVGhUZ1pKNENNV05Qd1N5cHVwdW1k?=
 =?utf-8?B?M1hXTVJhcDJEdXRYS1pzcjRpcHhDeXFwbTlTb0Y0N1BnVFJBZlY0SFdrSHJL?=
 =?utf-8?B?aDZpUG1nckN0bFFlRFdXdGxhYTdjODF0UnVHOVNrbnJ5NnVsTUlvS1hEbWVv?=
 =?utf-8?B?TXRxL3RxNi96aGhUWjdwL0VYQ094M3R0bVhORE4yWXIvRWNrMStvejdXaXF2?=
 =?utf-8?B?Nmd3TmpyZTJ5OHB4L1FvNHVwZDluVFRwSFk5eWdtZ1h3cVVRU0tOVENvcUJR?=
 =?utf-8?B?ajNFcXZDU0lrK1FPc2JGWGp0T0tnREM0M3l3WVBKUWkxM2lxaFRZa01GdXVp?=
 =?utf-8?B?UG80VWI5aHJqUzc3RHg5RFkvQzdpRElXVzFibmQ2TFF2OHBBdDhqRDhtTDdN?=
 =?utf-8?B?VjlFOEZsWW9BUjdZNS85VUFGa2pQeEVQdjVYOXMvMytGTnZjQXBYSllFclRY?=
 =?utf-8?B?aldPRTdsOXFiWlFaREN4Y1FNR01wQjZpZHU3NFZRdmdpWnNtRFlVdkYrZzFk?=
 =?utf-8?B?d0Nyd3ZHMXUyLzZsZjJLWk40RWhGVjdvdVJSK2NzMVdvckVhbUIzRVFqMjRV?=
 =?utf-8?B?TlRKSDkzU0tXQm9qQ3pYRGFCWVdJd0J4YXRmbUxNQStVNktacXpTaGhSUGxF?=
 =?utf-8?B?NStOSEc1VDlGUEtVbVFwRkFBNjI0NmJSTjY5Wm5WeC8zTnNxcjd5VW53Z0JK?=
 =?utf-8?B?TGZVMDI4R0VMQjFBVFYzaGxLdCtYVDM1SkxHbUxVMTB3Yi9aNi8zOWswOW0v?=
 =?utf-8?B?YzE2YjArZDJVM2tSMHBmS2xsVCtaemQ3ZDVzZUk3b3J6Skd2VnB4WCtDY3dD?=
 =?utf-8?B?emhCZ2x2ZUsrbE11Q2FxU0ZwaTd2WmZKUFl2dENoTE1YTXI4QjdZYmlFWGFT?=
 =?utf-8?B?VjRpdDJyTXdpb2pWanIwb0ZjUXhXaVlQcGYzb0d6S0tSSlpUR3ZPTG1OSG9W?=
 =?utf-8?B?UkdIRTg5d3VZZm9NUGZ5RDZxTWJNNTVJUG9NNlBDdmhqM29YKzJ0NkQra0pa?=
 =?utf-8?B?Yk5JWnhlNFZacDdkc3Y0N3NiRHRlblZ0SnFBL0pHaHMrcjFRemM0WHh0dHlG?=
 =?utf-8?B?MkVYbjFrNVhydTBsVFZhSTA1aitzbTlaNlFYS0xpMGprcWROTm43cE5OTXNB?=
 =?utf-8?B?SGpLUzBKLzlUbm1aSUJXSFlzR05FeUFJWWNFT1RKLzI5NE1VWjliUHJMMnI3?=
 =?utf-8?Q?d+mc0XoDmyqP4jTh/h0n+9UKw?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 75f91369-4189-48c9-e7ce-08dc22b6f134
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3320.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2024 23:47:10.7055
 (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: hyVYjvjVAwxNCPTSdQLQ1I4OzNEW7nJmmU2TnLa37SFvuypegAJlbcuD8HbILgZD9Pa7UptoARpOcrpw8GnwWA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR11MB7864
X-OriginatorOrg: intel.com

On 1/23/2024 5:11 AM, Thomas Gleixner wrote:

> +	pr_info("Allowing %u present CPUs plus %u hotplug CPUs\n", assigned, disabled);
> +	if (topo_info.nr_rejected_cpus)
> +		pr_info("Rejected CPUs %u\n", topo_info.nr_rejected_cpus);
> +

I encountered the same issue that Micheal mentions in the other thread.
This is how the messages show up in a 40-cpu system with:

nr_cpus=2 (correct)
-------------------
CPU topo: Allowing 2 present CPUs plus 0 hotplug CPUs
CPU topo: Rejected CPUs 38

nr_cpus=1 (incorrect)
---------------------
CPU topo: Allowing 1 present CPUs plus 0 hotplug CPUs
CPU topo: Rejected CPUs 40

Sohil