bluesky_fs_key_equal_func);
fs->next_inum = BLUESKY_ROOT_INUM + 1;
fs->store = bluesky_store_new("file");
+ fs->flushd_lock = g_mutex_new();
return fs;
}
void bluesky_inode_unref(BlueSkyInode *inode)
{
if (g_atomic_int_dec_and_test(&inode->refcount)) {
- g_print("Reference count for inode %"PRIu64" dropped to zero.\n",
- inode->inum);
+ if (bluesky_verbose) {
+ g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
+ "Reference count for inode %"PRIu64" dropped to zero.",
+ inode->inum);
+ }
/* Sanity check: Is the inode clean? */
if (inode->change_commit < inode->change_count
+ || inode->accessed_list != NULL
|| inode->dirty_list != NULL) {
- g_warning("Dropping inode which is not clean (commit %"PRIi64" < change %"PRIi64"; dirty_list = %p)\n", inode->change_commit, inode->change_count, inode->dirty_list);
+ g_warning("Dropping inode which is not clean (commit %"PRIi64" < change %"PRIi64"; accessed_list = %p; dirty_list = %p)\n", inode->change_commit, inode->change_count, inode->accessed_list, inode->dirty_list);
}
+ /* These shouldn't be needed, but in case the above warning fires and
+ * we delete the inode anyway, we ought to be sure the inode is not on
+ * any LRU list. */
g_mutex_lock(inode->fs->lock);
bluesky_list_unlink(&inode->fs->accessed_list, inode->accessed_list);
bluesky_list_unlink(&inode->fs->dirty_list, inode->dirty_list);
BlueSkyStoreAsync *barrier = bluesky_store_async_new(inode->fs->store);
barrier->op = STORE_OP_BARRIER;
- g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
- "Synchronous writeback for inode %"PRIu64"...", inode->inum);
+ if (bluesky_verbose) {
+ g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
+ "Synchronous writeback for inode %"PRIu64"...", inode->inum);
+ }
bluesky_inode_start_sync(inode, barrier);
bluesky_store_async_submit(barrier);
bluesky_store_async_wait(barrier);
bluesky_store_async_unref(barrier);
- g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
- "Writeback for inode %"PRIu64" complete", inode->inum);
+ if (bluesky_verbose) {
+ g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
+ "Writeback for inode %"PRIu64" complete", inode->inum);
+ }
}
static void complete_inode_fetch(BlueSkyStoreAsync *async, BlueSkyInode *inode)
{
- g_print("Completing fetch of inode %"PRIu64"...\n", inode->inum);
+ if (bluesky_verbose) {
+ g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
+ "Completing fetch of inode %"PRIu64"...", inode->inum);
+ }
if (async->result != 0
|| !bluesky_deserialize_inode(inode, async->data->data))
{
- g_print(" failed to load inode, cleaning up\n");
+ if (bluesky_verbose) {
+ g_log("bluesky/inode", G_LOG_LEVEL_DEBUG,
+ " failed to load inode, cleaning up");
+ }
g_mutex_lock(inode->fs->lock);
g_hash_table_remove(inode->fs->inodes, &inode->inum);
+ bluesky_list_unlink(&inode->fs->accessed_list, inode->accessed_list);
+ inode->accessed_list = NULL;
g_mutex_unlock(inode->fs->lock);
bluesky_inode_unref(inode);
}
inode->access_time = bluesky_get_current_time();
g_mutex_lock(inode->fs->lock);
+ bluesky_list_unlink(&inode->fs->accessed_list, inode->accessed_list);
inode->accessed_list = bluesky_list_prepend(&inode->fs->accessed_list, inode);
g_mutex_unlock(inode->fs->lock);
sprintf(key, "inode-%016"PRIx64, inum);
BlueSkyInode *inode = bluesky_new_inode(inum, fs, BLUESKY_PENDING);
+ inode->change_count = 0;
bluesky_inode_ref(inode); // Extra ref held by fetching process
g_mutex_lock(inode->lock);
bluesky_insert_inode(fs, inode);