diff options
| author | Aaro Koskinen <aaro.koskinen@nokia.com> | 2009-08-28 14:28:54 +0300 |
|---|---|---|
| committer | Pekka Enberg <penberg@cs.helsinki.fi> | 2009-08-30 14:56:48 +0300 |
| commit | acdfcd04d9df7d084ff752f82afad6ed4ad5f363 (patch) | |
| tree | edee979b012067348978d7ea23a58acd4b532a7d /include/linux | |
| parent | cf5d11317e8f2671d3115622aec76274a40f4fc2 (diff) | |
| download | linux-3.10-acdfcd04d9df7d084ff752f82afad6ed4ad5f363.tar.gz linux-3.10-acdfcd04d9df7d084ff752f82afad6ed4ad5f363.tar.bz2 linux-3.10-acdfcd04d9df7d084ff752f82afad6ed4ad5f363.zip | |
SLUB: fix ARCH_KMALLOC_MINALIGN cases 64 and 256
If the minalign is 64 bytes, then the 96 byte cache should not be created
because it would conflict with the 128 byte cache.
If the minalign is 256 bytes, patching the size_index table should not
result in a buffer overrun.
The calculation "(i - 1) / 8" used to access size_index[] is moved to
a separate function as suggested by Christoph Lameter.
Acked-by: Christoph Lameter <cl@linux-foundation.org>
Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/slub_def.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/include/linux/slub_def.h b/include/linux/slub_def.h index 4dcbc2c7149..aa5d4a69d46 100644 --- a/include/linux/slub_def.h +++ b/include/linux/slub_def.h @@ -152,12 +152,10 @@ static __always_inline int kmalloc_index(size_t size) if (size <= KMALLOC_MIN_SIZE) return KMALLOC_SHIFT_LOW; -#if KMALLOC_MIN_SIZE <= 64 - if (size > 64 && size <= 96) + if (KMALLOC_MIN_SIZE <= 32 && size > 64 && size <= 96) return 1; - if (size > 128 && size <= 192) + if (KMALLOC_MIN_SIZE <= 64 && size > 128 && size <= 192) return 2; -#endif if (size <= 8) return 3; if (size <= 16) return 4; if (size <= 32) return 5; |