In mempool_resize(), I plan to replace the calls
to spin_unlock_irqrestore() -> pool_free -> spin_lock_
irqsave with the new sequence. The newer code
has fewer calls to spin_lock_irqsave and spin_unlock_
restore(). I think the new code should execute faster.
I have tested this code on a UP, if somebody could
test the patch on an SMP box and let me know if it
works, I would be thankful. I also have a sample test
case that uses mempool_resize().
I have also added the BUG_ON sanity checks in
mempool_create().
New Code
if (new_min_nr < pool->min_nr) {
pool->min_nr = new_min_nr;
/*
* Free possible excess elements.
*/
INIT_LIST_HEAD(&tmp_pool_to_free);
while (pool->curr_nr > pool->min_nr) {
tmp = pool->elements.next;
if (tmp == &pool->elements)
BUG();
list_del(tmp);
list_add(tmp, &tmp_pool_to_free);
pool->curr_nr--;
}
spin_unlock_irqrestore(&pool->lock, flags);
list_for_each(tmp, &tmp_pool_to_free) {
element = tmp;
pool->free(element, pool->pool_data);
}
return;
}
Original Code
if (new_min_nr < pool->min_nr) {
pool->min_nr = new_min_nr;
/*
* Free possible excess elements.
*/
while (pool->curr_nr > pool->min_nr) {
tmp = pool->elements.next;
if (tmp == &pool->elements)
BUG();
list_del(tmp);
element = tmp;
pool->curr_nr--;
spin_unlock_irqrestore(&pool->lock,
flags);
pool->free(element, pool->pool_data);
spin_lock_irqsave(&pool->lock, flags);
}
spin_unlock_irqrestore(&pool->lock, flags);
return;
}
Diffs are attached.
Balbir
__________________________________________________
Do You Yahoo!?
Yahoo! Sports - live college hoops coverage
http://sports.yahoo.com/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Fri Mar 15 2002 - 22:00:22 EST