From eddb388ef90258be406bdf1355f7c65bdd71bbe8 Mon Sep 17 00:00:00 2001
From: antirez <antirez@metal.(none)>
Date: Thu, 2 Sep 2010 10:34:39 +0200
Subject: [PATCH] memory fragmentation ratio in INFO output

---
 src/config.h  |  5 +++++
 src/redis.c   |  2 ++
 src/zmalloc.c | 42 ++++++++++++++++++++++++++++++++++++++++++
 src/zmalloc.h |  1 +
 4 files changed, 50 insertions(+)

diff --git a/src/config.h b/src/config.h
index 6e98fbb2..acc95cf5 100644
--- a/src/config.h
+++ b/src/config.h
@@ -21,6 +21,11 @@
 #define redis_stat stat
 #endif
 
+/* test for proc filesystem */
+#ifdef __linux__
+#define HAVE_PROCFS 1
+#endif
+
 /* test for backtrace() */
 #if defined(__APPLE__) || defined(__linux__)
 #define HAVE_BACKTRACE 1
diff --git a/src/redis.c b/src/redis.c
index 77e67c58..8206b5d3 100644
--- a/src/redis.c
+++ b/src/redis.c
@@ -1166,6 +1166,7 @@ sds genRedisInfoString(void) {
         "blocked_clients:%d\r\n"
         "used_memory:%zu\r\n"
         "used_memory_human:%s\r\n"
+        "mem_fragmentation_ratio:%.2f\r\n"
         "changes_since_last_save:%lld\r\n"
         "bgsave_in_progress:%d\r\n"
         "last_save_time:%ld\r\n"
@@ -1192,6 +1193,7 @@ sds genRedisInfoString(void) {
         server.blpop_blocked_clients,
         zmalloc_used_memory(),
         hmem,
+        zmalloc_get_fragmentation_ratio(),
         server.dirty,
         server.bgsavechildpid != -1,
         server.lastsave,
diff --git a/src/zmalloc.c b/src/zmalloc.c
index 5c1b5e9a..81fc4c04 100644
--- a/src/zmalloc.c
+++ b/src/zmalloc.c
@@ -32,6 +32,10 @@
 #include <stdlib.h>
 #include <string.h>
 #include <pthread.h>
+#include <unistd.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <fcntl.h>
 #include "config.h"
 
 #if defined(__sun)
@@ -170,3 +174,41 @@ size_t zmalloc_used_memory(void) {
 void zmalloc_enable_thread_safeness(void) {
     zmalloc_thread_safe = 1;
 }
+
+/* Fragmentation = RSS / allocated-bytes */
+float zmalloc_get_fragmentation_ratio(void) {
+#ifdef HAVE_PROCFS
+    size_t allocated = zmalloc_used_memory();
+    int page = sysconf(_SC_PAGESIZE);
+    size_t rss;
+    char buf[4096];
+    char filename[256];
+    int fd, count;
+    char *p, *x;
+
+    snprintf(filename,256,"/proc/%d/stat",getpid());
+    if ((fd = open(filename,O_RDONLY)) == -1) return 0;
+    if (read(fd,buf,4096) <= 0) {
+        close(fd);
+        return 0;
+    }
+    close(fd);
+
+    p = buf;
+    count = 23; /* RSS is the 24th field in /proc/<pid>/stat */
+    while(p && count--) {
+        p = strchr(p,' ');
+        if (p) p++;
+    }
+    if (!p) return 0;
+    x = strchr(p,' ');
+    if (!x) return 0;
+    *x = '\0';
+
+    rss = strtoll(p,NULL,10);
+    rss *= page;
+    return (float)rss/allocated;
+#else
+    return 0;
+#endif
+}
diff --git a/src/zmalloc.h b/src/zmalloc.h
index db858bba..281aa3a8 100644
--- a/src/zmalloc.h
+++ b/src/zmalloc.h
@@ -38,5 +38,6 @@ void zfree(void *ptr);
 char *zstrdup(const char *s);
 size_t zmalloc_used_memory(void);
 void zmalloc_enable_thread_safeness(void);
+float zmalloc_get_fragmentation_ratio(void);
 
 #endif /* _ZMALLOC_H */