diff options
| author |   Joonsoo Kim <iamjoonsoo.kim@lge.com> | 2014-03-12 17:06:19 +0900 |
|---|---|---|
| committer |   Pekka Enberg <penberg@kernel.org> | 2014-04-01 13:38:04 +0300 |
| commit | 24f870d8f0adcd38639f2f66e37aa7591a3fc408 (patch) | |
| tree | be36bd3e209e78faff043fda65337749b3bee7dc /mm/slab.c | |
| parent | slub: fix high order page allocation problem with __GFP_NOFAIL (diff) | |
| download | linux-dev-24f870d8f0adcd38639f2f66e37aa7591a3fc408.tar.xz linux-dev-24f870d8f0adcd38639f2f66e37aa7591a3fc408.zip | |
slab: fix wrongly used macro
commit 'slab: restrict the number of objects in a slab' uses
__builtin_constant_p() on #if macro. It is wrong usage of builtin
function, but it is compiled on x86 without any problem, so I can't
find it before 0 day build system find it.
This commit fixes the situation by using KMALLOC_MIN_SIZE, instead of
KMALLOC_SHIFT_LOW. KMALLOC_SHIFT_LOW is parsed to ilog2() on some
architecture and this ilog2() uses __builtin_constant_p() and results in
the problem. This problem would disappear by using KMALLOC_MIN_SIZE,
since it is just constant.
Tested-by: David Rientjes <rientjes@google.com>
Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm/slab.c')
0 files changed, 0 insertions, 0 deletions