Sending to wider audience. I originally sent this to only the people Andi had cc'd.
Andi posted two patches to clean up the logical package ID, and I posted a bug fix for systems which boot with less than than the maximum core count. If I take his changes into account the code base ends up with a significant cleanup. I am reposting his patches here. I boot tested this using max_cpus and nr_cpus, as well as booting on systems which panic without the fix in 3/3. ---8<--- Cleanup the logical package ID code by storing the logical package ID in the cpuinfo_x86 struct and calculating the maximum logical package ID after all the CPUs have been enumerated. Signed-off-by: Prarit Bhargava <pra...@redhat.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Ingo Molnar <mi...@redhat.com> Cc: "H. Peter Anvin" <h...@zytor.com> Cc: x...@kernel.org Cc: Peter Zijlstra <pet...@infradead.org> Cc: Andi Kleen <a...@linux.intel.com> Cc: Dave Hansen <dave.han...@intel.com> Cc: Piotr Luc <piotr....@intel.com> Cc: Kan Liang <kan.li...@intel.com> Cc: Borislav Petkov <b...@suse.de> Cc: Stephane Eranian <eran...@google.com> Cc: Prarit Bhargava <pra...@redhat.com> Cc: Arvind Yadav <arvind.yadav...@gmail.com> Cc: Andy Lutomirski <l...@kernel.org> Cc: Christian Borntraeger <borntrae...@de.ibm.com> Cc: "Kirill A. Shutemov" <kirill.shute...@linux.intel.com> Cc: Tom Lendacky <thomas.lenda...@amd.com> Cc: He Chen <he.c...@linux.intel.com> Cc: Mathias Krause <mini...@googlemail.com> Cc: Tim Chen <tim.c.c...@linux.intel.com> Cc: Vitaly Kuznetsov <vkuzn...@redhat.com> Andi Kleen (2): perf/x86/intel/uncore: Cache logical pkg id in uncore driver x86/topology: Avoid wasting 128k for package id array Prarit Bhargava (1): x86/smpboot: Fix __max_logical_packages estimate arch/x86/events/intel/uncore.c | 1 + arch/x86/events/intel/uncore.h | 1 + arch/x86/events/intel/uncore_snbep.c | 2 +- arch/x86/include/asm/processor.h | 6 +- arch/x86/kernel/smpboot.c | 113 +++++++++++------------------------ 5 files changed, 42 insertions(+), 81 deletions(-) -- 1.8.5.5
