diff options
author | Michel Lespinasse <walken@google.com> | 2012-08-14 17:53:20 -0400 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2012-08-16 02:32:19 -0400 |
commit | 48f2474144ac708f1faad97e82a863ca8214b602 (patch) | |
tree | 93ee03853f20d4dafe041d9ddf5641bd7437559a /mm | |
parent | d9b7f22623b5fa9cc189581dcdfb2ac605933bf4 (diff) |
slab: do not call compound_head() in page_get_cache()
page_get_cache() does not need to call compound_head(), as its unique
caller virt_to_slab() already makes sure to return a head page.
Additionally, removing the compound_head() call makes page_get_cache()
consistent with page_get_slab().
Signed-off-by: Michel Lespinasse <walken@google.com>
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Pekka Enberg <penberg@kernel.org>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/slab.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -517,7 +517,6 @@ static bool slab_max_order_set __initdata; | |||
517 | 517 | ||
518 | static inline struct kmem_cache *page_get_cache(struct page *page) | 518 | static inline struct kmem_cache *page_get_cache(struct page *page) |
519 | { | 519 | { |
520 | page = compound_head(page); | ||
521 | BUG_ON(!PageSlab(page)); | 520 | BUG_ON(!PageSlab(page)); |
522 | return page->slab_cache; | 521 | return page->slab_cache; |
523 | } | 522 | } |