Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1245685

Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[] mapping

From Thomas Gleixner <tglx@linutronix.de>
Newsgroups linux.kernel
Subject Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[] mapping
Date 2015-10-13 15:00 +0200
Message-ID <qj7lp-728-19@gated-at.bofh.it> (permalink)
References (1 earlier) <qfe26-46r-17@gated-at.bofh.it> <qhHW2-6eZ-47@gated-at.bofh.it> <qiIwG-4vm-15@gated-at.bofh.it> <qiIwG-4vm-13@gated-at.bofh.it> <qj4dR-2vA-37@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, 13 Oct 2015, Jiang Liu wrote:
> On 2015/10/12 18:25, Thomas Gleixner wrote:
> > On Mon, 12 Oct 2015, Daniel J Blueman wrote:
> >> Another approach which may be suitable without changing SRAT parsing to be
> >> after the memory allocator is up, is to exploit the associativity of the
> >> bottom APIC ID bits.
> > 
> > What's the problem with moving (SRAT/ACPI/whatever) APIC parsing after
> > the memory allocator is up and available?
> Hi Thomas,
> 	The work flow is as below at boot:
> 1) figure out memory NUMA topology info by walking ACPI table or probing
>    AMD northbirdge.
> 2) initialize memory allocation based on memory NUMA topology.
> 
> And to make code simple, it also scan CPU NUMA topology in step 1, so
> we could avoid walking ACPI tables twice. On the other hand, there are
> several subsystems having code pattern as follow before booting APs.
> up:
> for_each_possible_cpu(cpu)
> 	alloc_page_node(size, cpu_to_node(cpu))
> So it's a little hard to find a suitable hook point to delay CPU NUMA
> topology scanning after memory allocator is ready.

Not really. The memory allocator is available very early and long
before smp_prepare_cpus(). So we should try hard to move it after that
point, even if that means that we need to walk the tables twice.

Alternatively, use a bootmem allocation and convert it to a radix tree
when the allocator is up.

Thanks,

	tglx


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[]  mapping Jiang Liu <jiang.liu@linux.intel.com> - 2015-10-09 17:40 +0200
  Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[]  mapping Thomas Gleixner <tglx@linutronix.de> - 2015-10-12 12:30 +0200
    Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[]  mapping Jiang Liu <jiang.liu@linux.intel.com> - 2015-10-13 11:40 +0200
      Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[]  mapping Thomas Gleixner <tglx@linutronix.de> - 2015-10-13 15:00 +0200
  Re: [PATCH 3/3] x86/apic: Use smaller array for __apicid_to_node[]  mapping Daniel J Blueman <daniel@numascale.com> - 2015-10-12 12:40 +0200

csiph-web