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 01c55e89f54644f3ba83b2870b2850b21e704a11..0e9ed0ca656f25a2231bbe88872c7c9a31d609f4 100644 (file)
--- 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 <stdlib.h>