]> granicus.if.org Git - apache/commit
"Thanks for checking in the changes to mod_mem_cache.
authorCliff Woolley <jwoolley@apache.org>
Wed, 10 Dec 2003 03:22:32 +0000 (03:22 +0000)
committerCliff Woolley <jwoolley@apache.org>
Wed, 10 Dec 2003 03:22:32 +0000 (03:22 +0000)
commitd6b724ccb8b6dcd342e5c794bb62211dacc229f3
tree7bf8932d0a8507a30be16bf0ddf2408fec429130
parent14de2855679e148d537759665de3b844b7d6be31
"Thanks for checking in the changes to mod_mem_cache.
 I think there is a piece missing to that fixe;
 the adjustment of the queue_clock value in cache_cache.c
 (cache_insert()):
 Sorry about not finding/pointing that out before asking you to check in
 mod_mem_cache changes:

 queue_clock is initialized to 0 when initializing the cache.
 Based on the current conditional test (cache_cache.c, line 164):
 ----------------------------------------

        priority = c->get_pri(ejected);

        if (c->queue_clock < priority)
            c->queue_clock = priority;

 ----------------------------------------

 and the fact that the new get_pri() function return a negative value,
 queue_clock will NEVER be adjusted with the ejected element priority.
 This is a patch that should fix that problem:"

Submitted by: Jean-Jacques Clar
Generally glanced at by: Cliff Woolley (+1 on concept)
     (better to get it in there than to
      wait around for a year before I get
      a chance to really test it)

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@102008 13f79535-47bb-0310-9956-ffa450edef68
modules/experimental/cache_cache.c
modules/experimental/mod_mem_cache.c