Skip to content

Commit

Permalink
[PATCH] slab: use parameter passed to cache_reap to determine pointer…
Browse files Browse the repository at this point in the history
… to work structure

Use the pointer passed to cache_reap to determine the work pointer and
consolidate exit paths.

Signed-off-by: Christoph Lameter <clameter@sgi.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
  • Loading branch information
Christoph Lameter authored and Linus Torvalds committed Feb 11, 2007
1 parent 8c8cc2c commit 7c5cae3
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions mm/slab.c
Original file line number Diff line number Diff line change
Expand Up @@ -4029,18 +4029,17 @@ void drain_array(struct kmem_cache *cachep, struct kmem_list3 *l3,
* If we cannot acquire the cache chain mutex then just give up - we'll try
* again on the next iteration.
*/
static void cache_reap(struct work_struct *unused)
static void cache_reap(struct work_struct *w)
{
struct kmem_cache *searchp;
struct kmem_list3 *l3;
int node = numa_node_id();
struct delayed_work *work =
container_of(w, struct delayed_work, work);

if (!mutex_trylock(&cache_chain_mutex)) {
if (!mutex_trylock(&cache_chain_mutex))
/* Give up. Setup the next iteration. */
schedule_delayed_work(&__get_cpu_var(reap_work),
round_jiffies_relative(REAPTIMEOUT_CPUC));
return;
}
goto out;

list_for_each_entry(searchp, &cache_chain, next) {
check_irq_on();
Expand Down Expand Up @@ -4083,9 +4082,9 @@ static void cache_reap(struct work_struct *unused)
mutex_unlock(&cache_chain_mutex);
next_reap_node();
refresh_cpu_vm_stats(smp_processor_id());
out:
/* Set up the next iteration */
schedule_delayed_work(&__get_cpu_var(reap_work),
round_jiffies_relative(REAPTIMEOUT_CPUC));
schedule_delayed_work(work, round_jiffies_relative(REAPTIMEOUT_CPUC));
}

#ifdef CONFIG_PROC_FS
Expand Down

0 comments on commit 7c5cae3

Please sign in to comment.