Bug fix in the in-proxy cleaner component

[bluesky.git] / bluesky / cleaner.c

diff --git a/bluesky/cleaner.c b/bluesky/cleaner.c
index d7e29eb..3ca15b8 100644 (file)
--- a/bluesky/cleaner.c
+++ b/bluesky/cleaner.c
@@ -155,6 +155,8 @@ static BlueSkyCleanerItem *bluesky_cleaner_find_checkpoint(BlueSkyFS *fs)
     g_mutex_unlock(cachefile->lock);
 
     BlueSkyCleanerItem *checkpoint = bluesky_cleaner_deserialize(data);
+    checkpoint->location.directory = BLUESKY_CLOUD_DIR_CLEANER;
+    checkpoint->location.directory = seq;
     bluesky_string_unref(data);
 
     return checkpoint;
@@ -200,6 +202,29 @@ gboolean needs_merging(BlueSkyFS *fs, BlueSkyCloudPointer location)
     return TRUE;
 }
 
+/* For an inode which has been modified by the cleaner and must be flushed out
+ * to cloud storage, mark it as appropriately dirty.  We will bypass writingt
+ * the inode to the journal if possible--but if there have ben other
+ * uncommitted changes besides what the cleaner did then we will force a
+ * journal write as well since the cloud shouldn't contain newer data than the
+ * journal.  Inode must be locked. */
+static void cleaner_flush_inode(BlueSkyInode *inode)
+{
+    // if (inode->change_commit != inode->change_count) {
+    if (TRUE) {
+        /* bluesky_inode_start_sync schedules a flush to the cloud so we're all
+         * done. */
+        bluesky_inode_start_sync(inode);
+        return;
+    }
+
+    g_assert(inode->unlogged_list == NULL);
+
+    bluesky_list_unlink(&inode->fs->dirty_list, inode->dirty_list);
+    inode->dirty_list = bluesky_list_prepend(&inode->fs->dirty_list, inode);
+    inode->change_cloud = inode->change_count;
+}
+
 static void merge_inode(BlueSkyFS *fs, BlueSkyCleanerItem *cleaner_inode)
 {
     /* There are two versions we are concerned with: cleaner_ is for the data
@@ -219,8 +244,8 @@ static void merge_inode(BlueSkyFS *fs, BlueSkyCleanerItem *cleaner_inode)
 
     BlueSkyCloudLog *proxy_item = entry->item;
     g_mutex_lock(proxy_item->lock);
-    BlueSkyCloudPointer proxy_location = entry->item->location;
-    BlueSkyCloudID proxy_id = entry->item->id;
+    //BlueSkyCloudPointer proxy_location = entry->item->location;
+    //BlueSkyCloudID proxy_id = entry->item->id;
     g_mutex_unlock(proxy_item->lock);
     g_mutex_unlock(fs->lock);
 
@@ -254,6 +279,7 @@ static void merge_inode(BlueSkyFS *fs, BlueSkyCleanerItem *cleaner_inode)
             b->ref->location = cb->location;
         }
     }
+    cleaner_flush_inode(proxy_inode);
     g_mutex_unlock(proxy_inode->lock);
 
     /* Mark the inode as modified so it will get written back to the cloud.  We
@@ -305,5 +331,25 @@ void bluesky_cleaner_merge(BlueSkyFS *fs)
         bluesky_cleaner_item_free(imap);
     }
 
+    fs->log_state->latest_cleaner_seq_seen = checkpoint->location.sequence;
     bluesky_cleaner_item_free(checkpoint);
 }
+
+/* Run the cleaner as a background task. */
+static gpointer cleaner_thread(BlueSkyFS *fs)
+{
+    while (TRUE) {
+        struct timespec delay;
+        delay.tv_sec = 30;
+        delay.tv_nsec = 0;
+        nanosleep(&delay, NULL);
+        bluesky_cleaner_merge(fs);
+    }
+
+    return NULL;
+}
+
+void bluesky_cleaner_thread_launch(BlueSkyFS *fs)
+{
+    g_thread_create((GThreadFunc)cleaner_thread, fs, FALSE, NULL);
+}