summaryrefslogtreecommitdiffstats
path: root/include/linux/slab_def.h
diff options
context:
space:
mode:
authorChristoph Lameter <cl@linux.com>2012-11-28 16:23:01 +0000
committerPekka Enberg <penberg@kernel.org>2012-12-11 12:14:27 +0200
commit3c58346525d82625e68e24f071804c2dc057b6f4 (patch)
tree33dc428ff98a05bed75350b9cfc65e0f33f5ce43 /include/linux/slab_def.h
parent59a09917c95e5209135b4f1a87f1263d6ef40fdb (diff)
downloadop-kernel-dev-3c58346525d82625e68e24f071804c2dc057b6f4.zip
op-kernel-dev-3c58346525d82625e68e24f071804c2dc057b6f4.tar.gz
slab: Simplify bootstrap
The nodelists field in kmem_cache is pointing to the first unused object in the array field when bootstrap is complete. A problem with the current approach is that the statically sized kmem_cache structure use on boot can only contain NR_CPUS entries. If the number of nodes plus the number of cpus is greater then we would overwrite memory following the kmem_cache_boot definition. Increase the size of the array field to ensure that also the node pointers fit into the array field. Once we do that we no longer need the kmem_cache_nodelists array and we can then also use that structure elsewhere. Acked-by: Glauber Costa <glommer@parallels.com> Signed-off-by: Christoph Lameter <cl@linux.com> Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'include/linux/slab_def.h')
-rw-r--r--include/linux/slab_def.h6
1 files changed, 5 insertions, 1 deletions
diff --git a/include/linux/slab_def.h b/include/linux/slab_def.h
index cc290f0..45c0356 100644
--- a/include/linux/slab_def.h
+++ b/include/linux/slab_def.h
@@ -89,9 +89,13 @@ struct kmem_cache {
* (see kmem_cache_init())
* We still use [NR_CPUS] and not [1] or [0] because cache_cache
* is statically defined, so we reserve the max number of cpus.
+ *
+ * We also need to guarantee that the list is able to accomodate a
+ * pointer for each node since "nodelists" uses the remainder of
+ * available pointers.
*/
struct kmem_list3 **nodelists;
- struct array_cache *array[NR_CPUS];
+ struct array_cache *array[NR_CPUS + MAX_NUMNODES];
/*
* Do not add fields after array[]
*/
OpenPOWER on IntegriCloud