mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-21 08:19:28 +00:00
Revert "[POWERPC] Fake NUMA emulation for PowerPC"
This reverts commit 5c3f5892a2
,
basically because it changes behaviour even when no fake NUMA
information is specified on the kernel command line.
Firstly, it changes the nid, thus destroying the real NUMA
information. Secondly, it also changes behaviour in that if a node
ends up with no memory in it because of the memory limit, we used to
set it online and now we don't.
Also, in the non-NUMA case with no fake NUMA information, we do
node_set_online once for each LMB now, whereas previously we only did
it once. I don't know if that is actually a problem, but it does seem
unnecessary.
Signed-off-by: Paul Mackerras <paulus@samba.org>
This commit is contained in:
parent
75e06e2d7d
commit
55852bed57
1 changed files with 5 additions and 54 deletions
|
@ -24,8 +24,6 @@
|
||||||
|
|
||||||
static int numa_enabled = 1;
|
static int numa_enabled = 1;
|
||||||
|
|
||||||
static char *cmdline __initdata;
|
|
||||||
|
|
||||||
static int numa_debug;
|
static int numa_debug;
|
||||||
#define dbg(args...) if (numa_debug) { printk(KERN_INFO args); }
|
#define dbg(args...) if (numa_debug) { printk(KERN_INFO args); }
|
||||||
|
|
||||||
|
@ -41,43 +39,6 @@ static bootmem_data_t __initdata plat_node_bdata[MAX_NUMNODES];
|
||||||
static int min_common_depth;
|
static int min_common_depth;
|
||||||
static int n_mem_addr_cells, n_mem_size_cells;
|
static int n_mem_addr_cells, n_mem_size_cells;
|
||||||
|
|
||||||
static int __cpuinit fake_numa_create_new_node(unsigned long end_pfn,
|
|
||||||
unsigned int *nid)
|
|
||||||
{
|
|
||||||
unsigned long long mem;
|
|
||||||
char *p = cmdline;
|
|
||||||
static unsigned int fake_nid = 0;
|
|
||||||
static unsigned long long curr_boundary = 0;
|
|
||||||
|
|
||||||
*nid = fake_nid;
|
|
||||||
if (!p)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
mem = memparse(p, &p);
|
|
||||||
if (!mem)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
if (mem < curr_boundary)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
curr_boundary = mem;
|
|
||||||
|
|
||||||
if ((end_pfn << PAGE_SHIFT) > mem) {
|
|
||||||
/*
|
|
||||||
* Skip commas and spaces
|
|
||||||
*/
|
|
||||||
while (*p == ',' || *p == ' ' || *p == '\t')
|
|
||||||
p++;
|
|
||||||
|
|
||||||
cmdline = p;
|
|
||||||
fake_nid++;
|
|
||||||
*nid = fake_nid;
|
|
||||||
dbg("created new fake_node with id %d\n", fake_nid);
|
|
||||||
return 1;
|
|
||||||
}
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
static void __cpuinit map_cpu_to_node(int cpu, int node)
|
static void __cpuinit map_cpu_to_node(int cpu, int node)
|
||||||
{
|
{
|
||||||
numa_cpu_lookup_table[cpu] = node;
|
numa_cpu_lookup_table[cpu] = node;
|
||||||
|
@ -383,14 +344,12 @@ static void __init parse_drconf_memory(struct device_node *memory)
|
||||||
if (nid == 0xffff || nid >= MAX_NUMNODES)
|
if (nid == 0xffff || nid >= MAX_NUMNODES)
|
||||||
nid = default_nid;
|
nid = default_nid;
|
||||||
}
|
}
|
||||||
|
node_set_online(nid);
|
||||||
|
|
||||||
size = numa_enforce_memory_limit(start, lmb_size);
|
size = numa_enforce_memory_limit(start, lmb_size);
|
||||||
if (!size)
|
if (!size)
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
fake_numa_create_new_node(((start + size) >> PAGE_SHIFT), &nid);
|
|
||||||
node_set_online(nid);
|
|
||||||
|
|
||||||
add_active_range(nid, start >> PAGE_SHIFT,
|
add_active_range(nid, start >> PAGE_SHIFT,
|
||||||
(start >> PAGE_SHIFT) + (size >> PAGE_SHIFT));
|
(start >> PAGE_SHIFT) + (size >> PAGE_SHIFT));
|
||||||
}
|
}
|
||||||
|
@ -470,6 +429,7 @@ static int __init parse_numa_properties(void)
|
||||||
nid = of_node_to_nid_single(memory);
|
nid = of_node_to_nid_single(memory);
|
||||||
if (nid < 0)
|
if (nid < 0)
|
||||||
nid = default_nid;
|
nid = default_nid;
|
||||||
|
node_set_online(nid);
|
||||||
|
|
||||||
if (!(size = numa_enforce_memory_limit(start, size))) {
|
if (!(size = numa_enforce_memory_limit(start, size))) {
|
||||||
if (--ranges)
|
if (--ranges)
|
||||||
|
@ -478,9 +438,6 @@ static int __init parse_numa_properties(void)
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
fake_numa_create_new_node(((start + size) >> PAGE_SHIFT), &nid);
|
|
||||||
node_set_online(nid);
|
|
||||||
|
|
||||||
add_active_range(nid, start >> PAGE_SHIFT,
|
add_active_range(nid, start >> PAGE_SHIFT,
|
||||||
(start >> PAGE_SHIFT) + (size >> PAGE_SHIFT));
|
(start >> PAGE_SHIFT) + (size >> PAGE_SHIFT));
|
||||||
|
|
||||||
|
@ -504,7 +461,7 @@ static void __init setup_nonnuma(void)
|
||||||
unsigned long top_of_ram = lmb_end_of_DRAM();
|
unsigned long top_of_ram = lmb_end_of_DRAM();
|
||||||
unsigned long total_ram = lmb_phys_mem_size();
|
unsigned long total_ram = lmb_phys_mem_size();
|
||||||
unsigned long start_pfn, end_pfn;
|
unsigned long start_pfn, end_pfn;
|
||||||
unsigned int i, nid = 0;
|
unsigned int i;
|
||||||
|
|
||||||
printk(KERN_DEBUG "Top of RAM: 0x%lx, Total RAM: 0x%lx\n",
|
printk(KERN_DEBUG "Top of RAM: 0x%lx, Total RAM: 0x%lx\n",
|
||||||
top_of_ram, total_ram);
|
top_of_ram, total_ram);
|
||||||
|
@ -514,11 +471,9 @@ static void __init setup_nonnuma(void)
|
||||||
for (i = 0; i < lmb.memory.cnt; ++i) {
|
for (i = 0; i < lmb.memory.cnt; ++i) {
|
||||||
start_pfn = lmb.memory.region[i].base >> PAGE_SHIFT;
|
start_pfn = lmb.memory.region[i].base >> PAGE_SHIFT;
|
||||||
end_pfn = start_pfn + lmb_size_pages(&lmb.memory, i);
|
end_pfn = start_pfn + lmb_size_pages(&lmb.memory, i);
|
||||||
|
add_active_range(0, start_pfn, end_pfn);
|
||||||
fake_numa_create_new_node(end_pfn, &nid);
|
|
||||||
add_active_range(nid, start_pfn, end_pfn);
|
|
||||||
node_set_online(nid);
|
|
||||||
}
|
}
|
||||||
|
node_set_online(0);
|
||||||
}
|
}
|
||||||
|
|
||||||
void __init dump_numa_cpu_topology(void)
|
void __init dump_numa_cpu_topology(void)
|
||||||
|
@ -747,10 +702,6 @@ static int __init early_numa(char *p)
|
||||||
if (strstr(p, "debug"))
|
if (strstr(p, "debug"))
|
||||||
numa_debug = 1;
|
numa_debug = 1;
|
||||||
|
|
||||||
p = strstr(p, "fake=");
|
|
||||||
if (p)
|
|
||||||
cmdline = p + strlen("fake=");
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
early_param("numa", early_numa);
|
early_param("numa", early_numa);
|
||||||
|
|
Loading…
Reference in a new issue