[S+Q3 16/23] slub: Get rid of useless function count_free()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Christoph Lameter
Date: Tuesday, August 3, 2010 - 7:45 pm

count_free() == available()

Signed-off-by: Christoph Lameter <cl@linux-foundation.org>

---
 mm/slub.c |   11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

Index: linux-2.6/mm/slub.c
===================================================================
--- linux-2.6.orig/mm/slub.c	2010-07-30 18:44:54.767739966 -0500
+++ linux-2.6/mm/slub.c	2010-07-30 18:45:24.248349179 -0500
@@ -1697,11 +1697,6 @@
 	return 1;
 }
 
-static int count_free(struct page *page)
-{
-	return available(page);
-}
-
 static unsigned long count_partial(struct kmem_cache_node *n,
 					int (*get_count)(struct page *))
 {
@@ -1750,7 +1745,7 @@
 		if (!n)
 			continue;
 
-		nr_free  = count_partial(n, count_free);
+		nr_free  = count_partial(n, available);
 		nr_slabs = node_nr_slabs(n);
 		nr_objs  = node_nr_objs(n);
 
@@ -3906,7 +3901,7 @@
 			x = atomic_long_read(&n->total_objects);
 		else if (flags & SO_OBJECTS)
 			x = atomic_long_read(&n->total_objects) -
-				count_partial(n, count_free);
+				count_partial(n, available);
 
 			else
 				x = atomic_long_read(&n->nr_slabs);
@@ -4792,7 +4787,7 @@
 		nr_partials += n->nr_partial;
 		nr_slabs += atomic_long_read(&n->nr_slabs);
 		nr_objs += atomic_long_read(&n->total_objects);
-		nr_free += count_partial(n, count_free);
+		nr_free += count_partial(n, available);
 	}
 
 	nr_inuse = nr_objs - nr_free;

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 07/23] slub: Use kmem_cache flags to detect if slab ..., Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 08/23] slub: remove dynamic dma slab allocation, Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 11/23] slub: Dynamically size kmalloc cache allocations, Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 12/23] slub: Extract hooks for memory checkers from ..., Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 13/23] slub: Move gfpflag masking out of the hotpath, Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 15/23] slub: Allow resizing of per cpu queues, Christoph Lameter, (Tue Aug 3, 7:45 pm)
[S+Q3 16/23] slub: Get rid of useless function count_free(), Christoph Lameter, (Tue Aug 3, 7:45 pm)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), David Rientjes, (Tue Aug 3, 9:39 pm)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Wed Aug 4, 9:17 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), David Rientjes, (Thu Aug 5, 1:38 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Thu Aug 5, 10:33 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), David Rientjes, (Mon Aug 16, 9:56 pm)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 17, 6:56 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 17, 10:23 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 17, 10:29 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), David Rientjes, (Tue Aug 17, 11:02 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 17, 11:47 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), David Rientjes, (Tue Aug 17, 11:54 am)
Re: [S+Q3 00/23] SLUB: The Unified slab allocator (V3), Christoph Lameter, (Tue Aug 17, 12:34 pm)