From 78da899c5ceac63c424ad3d9e712b6a26abae7ba Mon Sep 17 00:00:00 2001
From: Raghavendra G <raghavendra@gluster.com>
Date: Thu, 8 Oct 2009 06:21:02 +0000
Subject: performance/stat-prefetch: Add accounting of cache hits and misses.

Signed-off-by: Anand V. Avati <avati@dev.gluster.com>

BUG: 221 (stat prefetch implementation)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=221
---
 xlators/performance/stat-prefetch/src/stat-prefetch.c | 15 +++++++++++++++
 xlators/performance/stat-prefetch/src/stat-prefetch.h |  2 ++
 2 files changed, 17 insertions(+)

(limited to 'xlators')

diff --git a/xlators/performance/stat-prefetch/src/stat-prefetch.c b/xlators/performance/stat-prefetch/src/stat-prefetch.c
index 1056d29e2..0adfded8d 100644
--- a/xlators/performance/stat-prefetch/src/stat-prefetch.c
+++ b/xlators/performance/stat-prefetch/src/stat-prefetch.c
@@ -597,6 +597,10 @@ sp_lookup (call_frame_t *frame, xlator_t *this, loc_t *loc, dict_t *xattr_req)
 
 wind:        
         if (lookup_behind) {
+                if (cache) {
+                        cache->hits++;
+                }
+
                 wind_frame = copy_frame (frame);
                 if (wind_frame == NULL) {
                         op_ret = -1;
@@ -618,6 +622,10 @@ wind:
                 STACK_WIND (wind_frame, sp_lookup_behind_cbk, FIRST_CHILD(this),
                             FIRST_CHILD(this)->fops->lookup, loc, xattr_req);
         } else {
+                if (cache) {
+                        cache->miss++;
+                }
+
                 STACK_WIND (frame, sp_lookup_cbk, FIRST_CHILD(this),
                             FIRST_CHILD(this)->fops->lookup, loc, xattr_req);
      
@@ -1728,10 +1736,17 @@ sp_release (xlator_t *this, fd_t *fd)
         sp_fd_ctx_t *fd_ctx = NULL;
         uint64_t     value  = 0;
         int32_t      ret    = 0;
+        sp_cache_t  *cache  = NULL;
 
         ret = fd_ctx_del (fd, this, &value);
         if (!ret) {
                 fd_ctx = (void *)(long) value;
+                cache = fd_ctx->cache;
+                if (cache) {
+                        gf_log (this->name, GF_LOG_DEBUG, "cache hits: %lu, "
+                                "cache miss: %lu", cache->hits, cache->miss);
+                }
+
                 sp_fd_ctx_free (fd_ctx);      
         }
 
diff --git a/xlators/performance/stat-prefetch/src/stat-prefetch.h b/xlators/performance/stat-prefetch/src/stat-prefetch.h
index 20b5e4a6f..dd6b067c5 100644
--- a/xlators/performance/stat-prefetch/src/stat-prefetch.h
+++ b/xlators/performance/stat-prefetch/src/stat-prefetch.h
@@ -37,6 +37,8 @@ struct sp_cache {
                                          * happen.
                                          */
         gf_lock_t   lock;
+        unsigned long miss;
+        unsigned long hits;
 };
 typedef struct sp_cache sp_cache_t;
 
-- 
cgit