Re: [PATCH 2/3 v3] x86/topology: Avoid wasting 128k for package id array

2017-10-23 Thread Prarit Bhargava
On 10/20/2017 05:03 AM, Thomas Gleixner wrote: > On Thu, 19 Oct 2017, Prarit Bhargava wrote: >> static void remove_siblinginfo(int cpu) >> { >> -int sibling; >> +int phys_pkg_id, sibling; >> struct cpuinfo_x86 *c = &cpu_data(cpu); >> >> for_each_cpu(sibling, topology_core_cp

Re: [PATCH 2/3 v3] x86/topology: Avoid wasting 128k for package id array

2017-10-20 Thread Thomas Gleixner
On Thu, 19 Oct 2017, Prarit Bhargava wrote: > static void remove_siblinginfo(int cpu) > { > - int sibling; > + int phys_pkg_id, sibling; > struct cpuinfo_x86 *c = &cpu_data(cpu); > > for_each_cpu(sibling, topology_core_cpumask(cpu)) { > @@ -1529,6 +1526,12 @@ static void rem

[PATCH 2/3 v3] x86/topology: Avoid wasting 128k for package id array

2017-10-19 Thread Prarit Bhargava
From: Andi Kleen I was looking at large early boot allocations and noticed that since (1f12e32f x86/topology: Create logical package id) every 64bit system allocates a 128k array to convert logical package ids. This happens because the array is sized for MAX_LOCAL_APIC and that is always 32k on