From 9b24d8adbeb2fb4ff663f3b205b73d55425f0d4f Mon Sep 17 00:00:00 2001
From: antirez <antirez@gmail.com>
Date: Mon, 3 Jan 2011 17:27:47 +0100
Subject: [PATCH] serious performance enhancement of diskstore

---
 src/dscache.c | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/src/dscache.c b/src/dscache.c
index 3a06b03e..06264ef6 100644
--- a/src/dscache.c
+++ b/src/dscache.c
@@ -587,6 +587,8 @@ void dsCreateIOJob(int type, redisDb *db, robj *key, robj *val) {
 #define REDIS_IO_LOADINPROG 4
 #define REDIS_IO_SAVEINPROG 8
 
+void cacheScheduleIOPushJobs(int onlyloads);
+
 void cacheScheduleIOAddFlag(redisDb *db, robj *key, long flag) {
     struct dictEntry *de = dictFind(db->io_queued,key);
 
@@ -647,6 +649,7 @@ void cacheScheduleIO(redisDb *db, robj *key, int type) {
      * in queue for the same key. */
     if (type == REDIS_IO_LOAD && !(flags & REDIS_IO_SAVE)) {
         listAddNodeHead(server.cache_io_queue, op);
+        cacheScheduleIOPushJobs(1);
     } else {
         /* FIXME: probably when this happens we want to at least move
          * the write job about this queue on top, and set the creation time
@@ -655,17 +658,25 @@ void cacheScheduleIO(redisDb *db, robj *key, int type) {
     }
 }
 
-void cacheCron(void) {
+/* Push scheduled IO operations into IO Jobs that the IO thread can process.
+ * If 'onlyloads' is true only IO_LOAD jobs are processed: this is useful
+ * since it's save to push LOAD IO jobs from any place of the code, while
+ * SAVE io jobs should never be pushed while we are processing a command
+ * (not protected by lookupKey() that will block on keys in IO_SAVEINPROG
+ * state. */
+#define MAX_IO_JOBS_QUEUE 100
+void cacheScheduleIOPushJobs(int onlyloads) {
     time_t now = time(NULL);
     listNode *ln;
     int jobs, topush = 0;
 
-    /* Sync stuff on disk, but only if we have less than 100 IO jobs */
+    /* Sync stuff on disk, but only if we have less
+     * than MAX_IO_JOBS_QUEUE IO jobs. */
     lockThreadedIO();
     jobs = listLength(server.io_newjobs);
     unlockThreadedIO();
 
-    topush = 100-jobs;
+    topush = MAX_IO_JOBS_QUEUE-jobs;
     if (topush < 0) topush = 0;
     if (topush > (signed)listLength(server.cache_io_queue))
         topush = listLength(server.cache_io_queue);
@@ -677,7 +688,7 @@ void cacheCron(void) {
         topush--;
 
         if (op->type == REDIS_IO_LOAD ||
-            (now - op->ctime) >= server.cache_flush_delay)
+            (!onlyloads && (now - op->ctime) >= server.cache_flush_delay))
         {
             struct dictEntry *de;
             robj *val;
@@ -732,6 +743,11 @@ void cacheCron(void) {
             break; /* too early */
         }
     }
+}
+
+void cacheCron(void) {
+    /* Push jobs */
+    cacheScheduleIOPushJobs(0);
 
     /* Reclaim memory from the object cache */
     while (server.ds_enabled && zmalloc_used_memory() >
-- 
2.47.2