]> git.baikalelectronics.ru Git - kernel.git/commit
cpu topology: remove stale arch_provides_topology_pointers and define_siblings_show_m...
authorHanjun Guo <hanjun.guo@linaro.org>
Sat, 27 Jul 2013 03:00:49 +0000 (11:00 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 29 Jul 2013 20:12:45 +0000 (13:12 -0700)
commitf61688e062b07c65afc5614a501a08b2a8a927a2
tree4b27ba3f5347e018e983712efb3a419e2125e9b8
parentd9bfde04b46a5c0fd696fdd45dfddafd23cd4d05
cpu topology: remove stale arch_provides_topology_pointers and define_siblings_show_map/list()

arch_provides_topology_pointers was introduced in commit d1f62b502 (x86:
cleanup early per cpu variables/accesses v4) to indicate pointers to the
topology cpumask_t maps are valid to avoid copying data on to/off of the
stack.

But later in commit 8a8ff573 (cpumask: Use topology_core_cpumask()/
topology_thread_cpumask()), the pointers to the topology struct cpumask maps
are always valid.

After that commit, the only difference is that there is a redundant
"unsigned int cpu = dev->id;" if arch_provides_topology_pointers defined, but
dev->id is type 'u32' which devolves to 'unsigned int' on all supported arches.
So this arch_provides_topology_pointers define is pointless and only cause
obfuscation now, remove it.

Tested on x86 machine, topology information in sys/devices/system/cpu/
cpuX/topology/ is the same after appling this patch set.

Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/base/topology.c