redonkable
/
remarkable-linux
1
0
Fork 0
Code Releases Activity

[PATCH] x86, x86_64: dual core proc-cpuinfo and sibling-map fix 

- broken sibling_map setup in x86_64

- grouping all the core and HT related cpuinfo fields.
  We are reasonably sure that adding new cpuinfo fields after "siblings" field,
  will not cause any app failure. Thats because today's /proc/cpuinfo
  format is completely different on x86, x86_64 and we haven't heard of any
  x86 app breakage because of this issue. Grouping these fields will 
  result in more or less common format on all architectures (ia64, x86 and 
  x86_64) and will cause less confusion.

Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
wifi-calibration
Siddha, Suresh B 2005-04-16 15:25:20 -07:00 committed by Linus Torvalds
parent a8ab26fe5b
commit d31ddaa172
3 changed files with 6 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
arch/i386/kernel/cpu/proc.c
View File

@ -98,6 +98,8 @@ static int show_cpuinfo(struct seq_file *m, void *v)
seq_printf(m, "physical id\t: %d\n", phys_proc_id[n]);
seq_printf(m, "siblings\t: %d\n",
c->x86_num_cores * smp_num_siblings);
seq_printf(m, "core id\t\t: %d\n", cpu_core_id[n]);
seq_printf(m, "cpu cores\t: %d\n", c->x86_num_cores);
}
#endif
@ -130,13 +132,6 @@ static int show_cpuinfo(struct seq_file *m, void *v)
c->loops_per_jiffy/(500000/HZ),
(c->loops_per_jiffy/(5000/HZ)) % 100);
#ifdef CONFIG_SMP
/* Put new fields at the end to lower the probability of
breaking user space parsers. */
seq_printf(m, "core id\t\t: %d\n", cpu_core_id[n]);
seq_printf(m, "cpu cores\t: %d\n", c->x86_num_cores);
#endif
return 0;
}

11
arch/x86_64/kernel/setup.c
View File

@ -1152,6 +1152,8 @@ static int show_cpuinfo(struct seq_file *m, void *v)
seq_printf(m, "physical id\t: %d\n", phys_proc_id[cpu]);
seq_printf(m, "siblings\t: %d\n",
c->x86_num_cores * smp_num_siblings);
seq_printf(m, "core id\t\t: %d\n", cpu_core_id[cpu]);
seq_printf(m, "cpu cores\t: %d\n", c->x86_num_cores);
}
#endif
@ -1195,15 +1197,8 @@ static int show_cpuinfo(struct seq_file *m, void *v)
}
}
seq_printf(m, "\n");
seq_printf(m, "\n\n");
#ifdef CONFIG_SMP
/* Put new fields at the end to lower the probability of
breaking user space parsers. */
seq_printf(m, "core id\t\t: %d\n", cpu_core_id[c - cpu_data]);
seq_printf(m, "cpu cores\t: %d\n", c->x86_num_cores);
#endif
seq_printf(m, "\n");
return 0;
}

2
arch/x86_64/kernel/smpboot.c
View File

@ -652,7 +652,7 @@ static __cpuinit void detect_siblings(void)
int i;
if (smp_num_siblings > 1) {
for_each_online_cpu (i) {
if (cpu_core_id[cpu] == phys_proc_id[i]) {
if (cpu_core_id[cpu] == cpu_core_id[i]) {
siblings++;
cpu_set(i, cpu_sibling_map[cpu]);
}