On Tue, Mar 11, 2025 at 03:51:39PM +0800, Yicong Yang wrote: > From: Yicong Yang <yangyic...@hisilicon.com> > > The core CPU control framework supports runtime SMT control which > is not yet supported on arm64. Besides the general vulnerabilities > concerns we want this runtime control on our arm64 server for: > > - better single CPU performance in some cases > - saving overall power consumption > > This patchset implements it in the following aspects: > > - Provides a default topology_is_primary_thread() > - support retrieve SMT thread number on OF based system > - support retrieve SMT thread number on ACPI based system > - select HOTPLUG_SMT for arm64 > > Tests has been done on our ACPI based arm64 server and on ACPI/OF > based QEMU VMs. > > Change since v11: > - Remove the check and warning for heterogeneous platform as suggested and > discussed
IIUC, the ask was not to remove it completely but to allow single threaded and same number of threads in all the multi-threaded CPUs. Anyways that is not a must, we can just stash max_smt_thread_num value before updating with max and see if it changes from one value to another with value != 1. It can be done later when we want to warn such systems if they appear in the future and we can't support them. For now, it looks fine. -- Regards, Sudeep