Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <08f1bbda-7d41-9b1b-c345-54f4c62ff976@intel.com>
Date:   Wed, 26 Jul 2023 16:27:47 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [patch 00/29] x86/cpu: Rework the topology evaluation
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>,
        "James E.J. Bottomley" <jejb@linux.ibm.com>,
        Dick Kennedy <dick.kennedy@broadcom.com>,
        James Smart <james.smart@broadcom.com>,
        "Martin K. Petersen" <martin.petersen@oracle.com>,
        <linux-scsi@vger.kernel.org>, <linux-hwmon@vger.kernel.org>,
        Jean Delvare <jdelvare@suse.com>,
        Huang Rui <ray.huang@amd.com>,
        Guenter Roeck <linux@roeck-us.net>,
        Steve Wahl <steve.wahl@hpe.com>,
        Mike Travis <mike.travis@hpe.com>,
        Dimitri Sivanich <dimitri.sivanich@hpe.com>,
        Russ Anderson <russ.anderson@hpe.com>
References: <20230724155329.474037902@linutronix.de>
 <545cff6e-09ad-afc3-5d5f-bf5a435885b9@intel.com> <87mszis3hd.ffs@tglx>
From:   Sohil Mehta <sohil.mehta@intel.com>
In-Reply-To: <87mszis3hd.ffs@tglx>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YTRYU0xaWnh1ZytVYkMzeDh6OG9xV2pnRnJ1K0d5SGcxdERrcG41b2Y4a1Mv?=
 =?utf-8?B?WWRTMC9Gc3ZzOUNsVy94VTZTeDVQOW82OUcrTFA2eEdiWEI3RUNMdnYzS0NX?=
 =?utf-8?B?NW0xR2t6QjRiQTh5RGpzWVFUU3ZHOFZPOHVvT3FBR1UxRmViZVVxRnB2WndB?=
 =?utf-8?B?UVhPUFNaRFd3WE5XS2swcnNabXMwNTA5bnNVSmh5YjJGc2QvS1UrNGlrU2VU?=
 =?utf-8?B?eGtDN0QrRk9qZE1yVzNMOG02VTZnaTVVMmhSekNHQzhIanE4Ri9PMjRRWjZz?=
 =?utf-8?B?VjVNOEtZa0NaaXJWWlZaMWd3YzlaMCtxbVlrZVZFM1dOejdaY0J4NlFKRlh2?=
 =?utf-8?B?N3EwWUZWWVI4WkZRb05rNGRVcmdhVjd0K1ZiV1ExVzFlOUEvYnY5ZlB6eDZj?=
 =?utf-8?B?K3MxeDh4UDh0ZGpDKzAwU0pTaWF4Ukx2V2Nra3RVMjM1bU1LcFVXVFd2b3lN?=
 =?utf-8?B?TjNqaldTdkFLdEduQ0k3Z2VUNWs3b3BBcUF4d0RUaS9ZTThZanJVZVVzWk92?=
 =?utf-8?B?VXNTY3VwVE1RMEVndXBydWNEQ2xuVVl2UmJJZG53YkZHbFhBZjJDalhQVk50?=
 =?utf-8?B?QUtqazdpVHFjSEp5aWRhdGQxUzZpZjBpK1V2c0YrdnhHdFRPblVtV3VsZ1Iz?=
 =?utf-8?B?dFN3Mzc5ZDhpdEtJUGNLZCtHNVlEeE05SGxDcEpHN3lTVVJnTitrdTJMOXRt?=
 =?utf-8?B?TFJXTzI5R2pIY1RiK1F5S2k3MUpMVGZpMEVRV3hob1JyUGxkK1lFSEJjU3pD?=
 =?utf-8?B?UGx6NGRWOEZpcDhtSEpZZWhjSVJZdWVzd01QcHRVTXo4UnFOMHFqWGV0T1Y4?=
 =?utf-8?B?VXFyd2FxdGJvL25vK1I2OVlTY3hIbzBCZlB1R2tVT2hnZk1ML2xwYnQ2Q0pQ?=
 =?utf-8?B?RFEzWTk5b1BpeDFPZ1E2WXFmREtFb0N5U2JNaUl6MGQ5alZ2WGdkQVVSd0lv?=
 =?utf-8?B?dG9zemhYMFlpV0FnZldobWpocldQT1FOQW5iVG11YVZWOFZ1aVNsMzE3TXdX?=
 =?utf-8?B?Y2s0Q0F1OUxRNzJlVTRVeWpSODN4c2dac285bys4VVBvR242emdLWVVXcEJj?=
 =?utf-8?B?bWJMNHczMzVpK2Q5bVlNR3haTmhLb2lkNTZuNVl2bzhEL3JwRzZyR29OTEp6?=
 =?utf-8?B?b2o1Zk1mVURZSlJqSHpneTRYOXp2dENKYVd1VmRkYlZmTGJPa0FjRmtBZlhl?=
 =?utf-8?B?cDhXbCtRSkdiNzNORFVQR0VpQTQwNWVyTmFTdUFPejRweEd2aDBTWHZzKzNS?=
 =?utf-8?B?ZnQydkNkL3laSFB4NDJQb3JqSllhRmlpeHB2bjVhTUQwUkE5akFoSW5JcTN5?=
 =?utf-8?B?OStjRGNYNHRPYS9OeEVRRW5MdVJTa3R5ME1qQ3hJbGp2OWVQd2NnTnFqYS83?=
 =?utf-8?B?QXgyM2FYSU5MU0JScW8yN3diU1dyMTFsSW1kL3RvL0lBQ0d4Z2d5NERleUF1?=
 =?utf-8?B?ZnU2ZHZOSTNGaHpLU2s2MmF2STNGeW4rRnppNHpiZ2dwK1VXQkJUSmRxUXVo?=
 =?utf-8?B?MnVHQnhCSnlSZGprOGtSeVBRelZFME8zY05NQUxyWGNGcmRwL3cwSWw2bkpH?=
 =?utf-8?B?WERCUFZJZ2pkVThBMnAwelJBOGFNckVSWmFaazBZQXNmM1IzMVRRUGxUL3Rn?=
 =?utf-8?B?ekF3eDNsUnFOTW8rVkxuVnFYK21uaStTbnRPRzRnM1JJWjlWVGtxQmFVL01Y?=
 =?utf-8?B?cTdpc3lLUjI3cmtFQkkxYm9GcHVWY2wxcGd6dWpacm5ReUY5SWJiN1cyRzBs?=
 =?utf-8?B?WDlUVHQrelU1Z2ZOTFVFYk5RNFZBcnhtcUIrMUlDSC9UQTE1ZDRia1RYQzJG?=
 =?utf-8?B?VGp4NkpVT0dkYXc0eURHT0hxZE5LdlVvbkhQRWQwNWtEck4xR29XZWFQUFBT?=
 =?utf-8?B?alBFTHM0K0hVVVIycFZ3cWVWQWpqcEd4M3B4dVpPLzVQalcydms1TVBSVWtJ?=
 =?utf-8?B?YVppald3ZklLVE1jenhSVG1HYjFzL1JKdFJQclNZZjZWMTV6RWdFSGVML1Zh?=
 =?utf-8?B?NmUyeVVVUHZuUnRCd2dMYks2MUxFUGJqc0ZMY3BpOTE4VVNmZVNLTDJOeE5N?=
 =?utf-8?B?TjlOVm5ycmxhbXJncHRkaTNqN2p1SHhBTGlJNDlRVFNZYWdZRkUwcC9vekU4?=
 =?utf-8?Q?Zzs8QKZL2BPoMtm6fSlxcG+f2?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4eb9667f-4a72-47c3-969b-08db8e2fed01
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3320.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2023 23:27:49.5243
 (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: M0pADvGoX3kvTpuTKgCjL8T3khPTYctE5PM8CTnVWbAueXTaiuHrJYVVUINFwfqiQPKUXZysvZxGx2MMiq5BDg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5062
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 7/26/2023 3:38 PM, Thomas Gleixner wrote:
> There is a real and unrelated snafu vs. that logical package and logical
> die management which I discovered today. I missed the fact that this
> cruft abuses cpuinfo as permanent storage, which breaks CPU
> offline/online as the online operation reinitializes the topology
> information.
> 
> I pushed out a fixed version to
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git x86/topology
> 
> earlier. It doesn't have a tag yet.
> 

Thanks, I'll try this out and experiment with CPU hotplug as well.

-Sohil