sunrpc/cache: factor out cache_is_expired
authorNeilBrown <neilb@suse.de>
Wed, 3 Feb 2010 06:31:31 +0000 (17:31 +1100)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Sun, 14 Mar 2010 22:08:37 +0000 (18:08 -0400)
This removes a tiny bit of code duplication, but more important
prepares for following patch which will perform the expiry check in
cache_lookup and the rest of the validity check in cache_check.

Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
net/sunrpc/cache.c

index 83592e0125858aa19c0c6cda9b59eb29dc952711..9826c5ceb9950158e961fd608f129b6adf554d0e 100644 (file)
@@ -49,6 +49,12 @@ static void cache_init(struct cache_head *h)
        h->last_refresh = now;
 }
 
+static inline int cache_is_expired(struct cache_detail *detail, struct cache_head *h)
+{
+       return  (h->expiry_time < get_seconds()) ||
+               (detail->flush_time > h->last_refresh);
+}
+
 struct cache_head *sunrpc_cache_lookup(struct cache_detail *detail,
                                       struct cache_head *key, int hash)
 {
@@ -184,9 +190,7 @@ static int cache_make_upcall(struct cache_detail *cd, struct cache_head *h)
 static inline int cache_is_valid(struct cache_detail *detail, struct cache_head *h)
 {
        if (!test_bit(CACHE_VALID, &h->flags) ||
-           h->expiry_time < get_seconds())
-               return -EAGAIN;
-       else if (detail->flush_time > h->last_refresh)
+           cache_is_expired(detail, h))
                return -EAGAIN;
        else {
                /* entry is valid */
@@ -400,8 +404,7 @@ static int cache_clean(void)
                for (ch = *cp ; ch ; cp = & ch->next, ch = *cp) {
                        if (current_detail->nextcheck > ch->expiry_time)
                                current_detail->nextcheck = ch->expiry_time+1;
-                       if (ch->expiry_time >= get_seconds() &&
-                           ch->last_refresh >= current_detail->flush_time)
+                       if (!cache_is_expired(current_detail, ch))
                                continue;
 
                        *cp = ch->next;