Commit 5577bd8a authored by Christoph Lameter's avatar Christoph Lameter Committed by Linus Torvalds

SLUB: Do our own flags based on PG_active and PG_error

The atomicity when handling flags in SLUB is not necessary since both flags
used by SLUB are not updated in a racy way.  Flag updates are either done
during slab creation or destruction or under slab_lock.  Some of these flags
do not have the non atomic variants that we need.  So define our own.
Signed-off-by: default avatarChristoph Lameter <clameter@sgi.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent eefaca9c
...@@ -99,42 +99,42 @@ ...@@ -99,42 +99,42 @@
* the fast path and disables lockless freelists. * the fast path and disables lockless freelists.
*/ */
#define FROZEN (1 << PG_active)
#ifdef CONFIG_SLUB_DEBUG
#define SLABDEBUG (1 << PG_error)
#else
#define SLABDEBUG 0
#endif
static inline int SlabFrozen(struct page *page) static inline int SlabFrozen(struct page *page)
{ {
return PageActive(page); return page->flags & FROZEN;
} }
static inline void SetSlabFrozen(struct page *page) static inline void SetSlabFrozen(struct page *page)
{ {
SetPageActive(page); page->flags |= FROZEN;
} }
static inline void ClearSlabFrozen(struct page *page) static inline void ClearSlabFrozen(struct page *page)
{ {
ClearPageActive(page); page->flags &= ~FROZEN;
} }
static inline int SlabDebug(struct page *page) static inline int SlabDebug(struct page *page)
{ {
#ifdef CONFIG_SLUB_DEBUG return page->flags & SLABDEBUG;
return PageError(page);
#else
return 0;
#endif
} }
static inline void SetSlabDebug(struct page *page) static inline void SetSlabDebug(struct page *page)
{ {
#ifdef CONFIG_SLUB_DEBUG page->flags |= SLABDEBUG;
SetPageError(page);
#endif
} }
static inline void ClearSlabDebug(struct page *page) static inline void ClearSlabDebug(struct page *page)
{ {
#ifdef CONFIG_SLUB_DEBUG page->flags &= ~SLABDEBUG;
ClearPageError(page);
#endif
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment