diff options
author | Andrew Tridgell <tridge@samba.org> | 1998-08-17 06:47:53 +0000 |
---|---|---|
committer | Andrew Tridgell <tridge@samba.org> | 1998-08-17 06:47:53 +0000 |
commit | 72ed7049d88e5296ebec362189e62a384385ad34 (patch) | |
tree | 4609523acc63ae3e14201b49a4b9c47d1ec8e4d6 /source3/smbd/conn.c | |
parent | 69c6f1624d79e4cf4296856d66216cca90863286 (diff) | |
download | samba-72ed7049d88e5296ebec362189e62a384385ad34.tar.gz samba-72ed7049d88e5296ebec362189e62a384385ad34.tar.bz2 samba-72ed7049d88e5296ebec362189e62a384385ad34.zip |
added some optimisation for the case where the number of open files is
very large. files.c now promotes a files_struct to the top of the list
if it is used when it is more than 10 elements from the top.
also moved common linked list code for the 5 sets of linked lists that
I've created over the past few days into dlinklist.h (I've explained
to Chris why I didn't use the ubiqx code)
(This used to be commit 1eb9ae2996b5a243a147f485e7e353d54f820852)
Diffstat (limited to 'source3/smbd/conn.c')
-rw-r--r-- | source3/smbd/conn.c | 27 |
1 files changed, 10 insertions, 17 deletions
diff --git a/source3/smbd/conn.c b/source3/smbd/conn.c index b110e8d082..2afbfb7d7e 100644 --- a/source3/smbd/conn.c +++ b/source3/smbd/conn.c @@ -72,10 +72,16 @@ find a conn given a cnum ****************************************************************************/ connection_struct *conn_find(int cnum) { + int count=0; connection_struct *conn; - for (conn=Connections;conn;conn=conn->next) { - if (conn->cnum == cnum) return conn; + for (conn=Connections;conn;conn=conn->next,count++) { + if (conn->cnum == cnum) { + if (count > 10) { + DLIST_PROMOTE(Connections, conn); + } + return conn; + } } return NULL; @@ -114,14 +120,7 @@ connection_struct *conn_new(void) string_init(&conn->connectpath,""); string_init(&conn->origpath,""); - /* hook into the front of the list */ - if (!Connections) { - Connections = conn; - } else { - Connections->prev = conn; - conn->next = Connections; - Connections = conn; - } + DLIST_ADD(Connections, conn); return conn; } @@ -165,13 +164,7 @@ free a conn structure ****************************************************************************/ void conn_free(connection_struct *conn) { - if (conn == Connections) { - Connections = conn->next; - if (Connections) Connections->prev = NULL; - } else { - conn->prev->next = conn->next; - if (conn->next) conn->next->prev = conn->prev; - } + DLIST_REMOVE(Connections, conn); if (conn->ngroups && conn->groups) { free(conn->groups); |