From 9a0856915f92f3b231638d2c7bddd9a78da8e3c3 Mon Sep 17 00:00:00 2001 From: phk Date: Fri, 15 Sep 2006 08:48:13 +0000 Subject: [PATCH] Add XXX comment git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@986 d4fa192b-c00b-0410-8231-f00ffab90ce4 --- varnish-cache/bin/varnishd/cache_session.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/varnish-cache/bin/varnishd/cache_session.c b/varnish-cache/bin/varnishd/cache_session.c index 01c55e89..0e9ed0ca 100644 --- a/varnish-cache/bin/varnishd/cache_session.c +++ b/varnish-cache/bin/varnishd/cache_session.c @@ -10,6 +10,15 @@ * * We identify srcaddrs instead of full addr+port because the desired level * of granularity is "whois is abuse@ or tech-c@ in the RIPE database. + * + * XXX: The two-list session management is actually not a good idea + * XXX: come to think of it, because we want the sessions reused in + * XXX: Most Recently Used order. + * XXX: Another and maybe more interesting option would be to cache + * XXX: free sessions in the worker threads and postpone session + * XXX: allocation until then. This does not quite implment MRU order + * XXX: but it does save some locking, although not that much because + * XXX: we still have to do the source-addr lookup. */ #include -- 2.39.5