From: phk Date: Fri, 15 Sep 2006 08:48:13 +0000 (+0000) Subject: Add XXX comment X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9a0856915f92f3b231638d2c7bddd9a78da8e3c3;p=varnish Add XXX comment git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@986 d4fa192b-c00b-0410-8231-f00ffab90ce4 --- 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