kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-03 11:25:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Change the traversal to use ao2_callback() instead of an ao2_iterator. Using

Browse Source 
ao2_callback() is a much more efficient way of performing an operation on every
item in the container.  This change makes hashtest2 run in about 25% of the
time it ran before on my system.

In general, I would say that it makes the most sense to use an ao2_iterator if
the operation being performed is going to take a long time and you don't want
to keep the container locked while you work with each object.  Otherwise,
the use of ao2_callback is preferred.


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@82282 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Russell Bryant
2007-09-12 15:53:40 +00:00
parent 7e19df05b9
commit 99fb378cdc
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
utils/hashtest2.c
View File

@@ -148,16 +148,17 @@ static void add_element(void)
els_added++; /* unprotected, sometimes off, but, not really important, either */
}
static int do_nothing_cb(void *obj, void *arg, int flags)
{
return 0;
}
static void traverse_elements(void)
{
struct ht_element *el;
struct ao2_iterator it = ao2_iterator_init(glob_hashtab, 0);
#ifdef DEBUG
printf("Traverse hashtab\n");
#endif
while ((el = ao2_iterator_next(&it))) {
ao2_ref(el,-1);
}
ao2_callback(glob_hashtab, OBJ_NODATA, do_nothing_cb, NULL);
els_traversals++; /* unprotected, sometimes off, but, not really important, either */
}