]> wimlib.net Git - wimlib/blobdiff - src/mount_image.c
Don't call unmount_progress_func() when no progress msgs requested
[wimlib] / src / mount_image.c
index ea44f220a868e48f836f06f0ade853f4747667e1..3f1e9934961cee526b727899adefe86f488726d5 100644 (file)
@@ -33,7 +33,7 @@
 #include "sha1.h"
 #include "lookup_table.h"
 #include "xml.h"
-#include "io.h"
+#include "buffer_io.h"
 #include "timestamp.h"
 #include <limits.h>
 #include <stdlib.h>
 #include <ftw.h>
 #include <mqueue.h>
 #include <utime.h>
+#include <signal.h>
 
 #ifdef ENABLE_XATTR
 #include <attr/xattr.h>
 #endif
 
+#define MSG_VERSION_TOO_HIGH   -1
+#define MSG_BREAK_LOOP         -2
+
 /* File descriptor to a file open on the WIM filesystem. */
 struct wimlib_fd {
        struct inode *f_inode;
@@ -76,12 +80,17 @@ struct wimfs_context {
        /* Flags passed to wimlib_mount(). */
        int mount_flags;
 
+       int default_lookup_flags;
+
        /* Next inode number to be assigned. */
        u64 next_ino;
 
        /* List of lookup table entries in the staging directory */
        struct list_head staging_list;
 
+       /* List of inodes in the mounted image */
+       struct hlist_head *image_inode_list;
+
        /* Name and message queue descriptors for message queues between the filesystem
         * daemon process and the unmount process.  These are used when the filesystem
         * is unmounted and the process running wimlib_mount() (i.e. the `imagex
@@ -119,10 +128,7 @@ static inline bool wimfs_ctx_readonly(const struct wimfs_context *ctx)
 
 static inline int get_lookup_flags(const struct wimfs_context *ctx)
 {
-       if (ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
-               return LOOKUP_FLAG_ADS_OK;
-       else
-               return 0;
+       return ctx->default_lookup_flags;
 }
 
 /* Returns nonzero if write permission is requested on the file open flags */
@@ -210,7 +216,6 @@ out:
 
 static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
 {
-       wimlib_assert(fd != NULL);
        wimlib_assert(inode != NULL);
 
        pthread_mutex_lock(&inode->i_mutex);
@@ -232,7 +237,6 @@ static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
 
 static int lte_put_fd(struct lookup_table_entry *lte, struct wimlib_fd *fd)
 {
-       wimlib_assert(fd != NULL);
        wimlib_assert(fd->f_lte == lte);
 
        if (!lte) /* Empty stream with no lookup table entry */
@@ -256,7 +260,6 @@ static int lte_put_fd(struct lookup_table_entry *lte, struct wimlib_fd *fd)
 static int close_wimlib_fd(struct wimlib_fd *fd)
 {
        int ret;
-       wimlib_assert(fd != NULL);
        DEBUG("Closing fd (inode = %lu, opened = %u, allocated = %u)",
              fd->f_inode->ino, fd->f_inode->num_opened_fds,
              fd->f_inode->num_allocated_fds);
@@ -268,6 +271,36 @@ static int close_wimlib_fd(struct wimlib_fd *fd)
        return 0;
 }
 
+static int create_dentry(struct wimfs_context *ctx, const char *path,
+                        struct dentry **dentry_ret)
+{
+       struct dentry *parent;
+       struct dentry *new;
+       const char *basename;
+
+       parent = get_parent_dentry(ctx->wim, path);
+       if (!parent)
+               return -ENOENT;
+
+       if (!dentry_is_directory(parent))
+               return -ENOTDIR;
+
+       basename = path_basename(path);
+       if (get_dentry_child_with_name(parent, basename))
+               return -EEXIST;
+
+       new = new_dentry_with_inode(basename);
+       if (!new)
+               return -ENOMEM;
+
+       new->d_inode->resolved = 1;
+       new->d_inode->ino = ctx->next_ino++;
+       dentry_add_child(parent, new);
+       hlist_add_head(&new->d_inode->hlist, ctx->image_inode_list);
+       *dentry_ret = new;
+       return 0;
+}
+
 /* Remove a dentry; i.e. remove a reference to the corresponding inode.
  *
  * If there are no remaining references to the inode either through dentries or
@@ -295,6 +328,49 @@ static void remove_dentry(struct dentry *dentry,
        put_dentry(dentry);
 }
 
+/* Transfers file attributes from a struct inode to a `stat' buffer.
+ *
+ * The lookup table entry tells us which stream in the inode we are statting.
+ * For a named data stream, everything returned is the same as the unnamed data
+ * stream except possibly the size and block count. */
+static int inode_to_stbuf(const struct inode *inode,
+                         struct lookup_table_entry *lte, struct stat *stbuf)
+{
+       if (inode_is_symlink(inode))
+               stbuf->st_mode = S_IFLNK | 0777;
+       else if (inode_is_directory(inode))
+               stbuf->st_mode = S_IFDIR | 0755;
+       else
+               stbuf->st_mode = S_IFREG | 0755;
+
+       stbuf->st_ino   = (ino_t)inode->ino;
+       stbuf->st_nlink = inode->link_count;
+       stbuf->st_uid   = getuid();
+       stbuf->st_gid   = getgid();
+
+       if (lte) {
+               if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
+                       struct stat native_stat;
+                       if (stat(lte->staging_file_name, &native_stat) != 0) {
+                               DEBUG("Failed to stat `%s': %m",
+                                     lte->staging_file_name);
+                               return -errno;
+                       }
+                       stbuf->st_size = native_stat.st_size;
+               } else {
+                       stbuf->st_size = wim_resource_size(lte);
+               }
+       } else {
+               stbuf->st_size = 0;
+       }
+
+       stbuf->st_atime   = wim_timestamp_to_unix(inode->last_access_time);
+       stbuf->st_mtime   = wim_timestamp_to_unix(inode->last_write_time);
+       stbuf->st_ctime   = wim_timestamp_to_unix(inode->creation_time);
+       stbuf->st_blocks  = (stbuf->st_size + 511) / 512;
+       return 0;
+}
+
 /* Creates a new staging file and returns its file descriptor opened for
  * writing.
  *
@@ -363,7 +439,12 @@ static int create_staging_file(char **name_ret, int open_flags,
  * extract, or NULL if there was no lookup table entry present for the stream
  *
  * @size:  Number of bytes of the stream we want to extract (this supports the
- * wimfs_truncate() function).
+ * wimfs_truncate() function).  It may be more than the actual stream length, in
+ * which case the extra space is filled with zeroes.
+ *
+ * @ctx:  Context for the WIM filesystem.
+ *
+ * Returns 0 on success or a negative error code on failure.
  */
 static int extract_resource_to_staging_dir(struct inode *inode,
                                           u32 stream_id,
@@ -375,19 +456,38 @@ static int extract_resource_to_staging_dir(struct inode *inode,
        int ret;
        int fd;
        struct lookup_table_entry *old_lte, *new_lte;
+       off_t extract_size;
 
        DEBUG("Extracting resource to staging dir: inode %"PRIu64", "
              "stream id %"PRIu32, inode->ino, stream_id);
 
        old_lte = *lte;
+
+       wimlib_assert(old_lte == NULL ||
+                     old_lte->resource_location != RESOURCE_IN_STAGING_FILE);
+
+       /* Create the staging file */
        fd = create_staging_file(&staging_file_name, O_WRONLY, ctx);
        if (fd == -1)
                return -errno;
 
-       if (old_lte)
-               ret = extract_wim_resource_to_fd(old_lte, fd, size);
-       else
+       /* Extract the stream to the staging file (possibly truncated) */
+       if (old_lte) {
+               extract_size = min(wim_resource_size(old_lte), size);
+               ret = extract_wim_resource_to_fd(old_lte, fd, extract_size);
+       } else {
                ret = 0;
+               extract_size = 0;
+       }
+
+       /* In the case of truncate() to more than the file length, extend the
+        * file with zeroes by calling ftruncate() on the underlying staging
+        * file */
+       if (ret == 0 && size > extract_size)
+               ret = ftruncate(fd, size);
+
+       /* Close the staging file descriptor and check for errors.  If there's
+        * an error, unlink the staging file. */
        if (ret != 0 || close(fd) != 0) {
                if (errno != 0)
                        ret = -errno;
@@ -397,28 +497,29 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                goto out_delete_staging_file;
        }
 
+       /* Now deal with the lookup table entries.  We may be able to re-use the
+        * existing entry, but we may have to create a new one instead. */
+
        if (old_lte && inode->link_count == old_lte->refcnt) {
-               /* The reference count of the existing lookup table
-                * entry is the same as the link count of the inode that
-                * contains the stream we're opening.  Therefore, ALL
-                * the references to the lookup table entry correspond
-                * to the stream we're trying to extract, so the lookup
-                * table entry can be re-used.  */
+               /* The reference count of the existing lookup table entry is the
+                * same as the link count of the inode that contains the stream
+                * we're opening.  Therefore, ALL the references to the lookup
+                * table entry correspond to the stream we're trying to extract,
+                * so the lookup table entry can be re-used.  */
                DEBUG("Re-using lookup table entry");
                lookup_table_unlink(ctx->wim->lookup_table, old_lte);
                new_lte = old_lte;
        } else {
                if (old_lte) {
                        /* There's an existing lookup table entry, but its
-                        * reference count is creater than the link count for
+                        * reference count is greater than the link count for
                         * the inode containing a stream we're opening.
                         * Therefore, we need to split the lookup table entry.
-                        * */
+                        */
                        wimlib_assert(old_lte->refcnt > inode->link_count);
                        DEBUG("Splitting lookup table entry "
                              "(inode->link_count = %u, old_lte->refcnt = %u)",
                              inode->link_count, old_lte->refcnt);
-
                }
 
                new_lte = new_lookup_table_entry();
@@ -463,7 +564,6 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                }
        }
 
-       new_lte->resource_entry.original_size = size;
        new_lte->refcnt                       = inode->link_count;
        new_lte->resource_location            = RESOURCE_IN_STAGING_FILE;
        new_lte->staging_file_name            = staging_file_name;
@@ -504,8 +604,7 @@ out_delete_staging_file:
  * Creates a randomly named staging directory and saves its name in the
  * filesystem context structure.
  */
-static int make_staging_dir(struct wimfs_context *ctx,
-                           const char *user_prefix)
+static int make_staging_dir(struct wimfs_context *ctx, const char *user_prefix)
 {
        static const size_t random_suffix_len = 10;
        static const char *common_suffix = ".staging";
@@ -581,14 +680,12 @@ static int remove_file_or_directory(const char *fpath, const struct stat *sb,
        }
 }
 
-
 /*
  * Deletes the staging directory and all the files contained in it.
  */
 static int delete_staging_dir(struct wimfs_context *ctx)
 {
        int ret;
-       wimlib_assert(ctx->staging_dir_name != NULL);
        ret = nftw(ctx->staging_dir_name, remove_file_or_directory,
                   10, FTW_DEPTH);
        FREE(ctx->staging_dir_name);
@@ -596,6 +693,108 @@ static int delete_staging_dir(struct wimfs_context *ctx)
        return ret;
 }
 
+static void inode_update_lte_ptr(struct inode *inode,
+                                struct lookup_table_entry *old_lte,
+                                struct lookup_table_entry *new_lte)
+{
+       if (inode->lte == old_lte) {
+               inode->lte = new_lte;
+       } else {
+               for (unsigned i = 0; i < inode->num_ads; i++) {
+                       if (inode->ads_entries[i].lte == old_lte) {
+                               inode->ads_entries[i].lte = new_lte;
+                               break;
+                       }
+               }
+       }
+}
+
+static int update_lte_of_staging_file(struct lookup_table_entry *lte,
+                                     struct lookup_table *table)
+{
+       struct lookup_table_entry *duplicate_lte;
+       int ret;
+       u8 hash[SHA1_HASH_SIZE];
+       struct stat stbuf;
+
+       ret = sha1sum(lte->staging_file_name, hash);
+       if (ret != 0)
+               return ret;
+       lookup_table_unlink(table, lte);
+       duplicate_lte = __lookup_resource(table, hash);
+       if (duplicate_lte) {
+               /* Merge duplicate lookup table entries */
+               duplicate_lte->refcnt += lte->refcnt;
+               inode_update_lte_ptr(lte->lte_inode, lte, duplicate_lte);
+               free_lookup_table_entry(lte);
+       } else {
+               if (stat(lte->staging_file_name, &stbuf) != 0) {
+                       ERROR_WITH_ERRNO("Failed to stat `%s'", lte->staging_file_name);
+                       return WIMLIB_ERR_STAT;
+               }
+               if (stbuf.st_size == 0) {
+                       /* Zero-length stream.  No lookup table entry needed. */
+                       inode_update_lte_ptr(lte->lte_inode, lte, NULL);
+                       free_lookup_table_entry(lte);
+               } else {
+                       wimlib_assert(&lte->file_on_disk == &lte->staging_file_name);
+                       lte->resource_entry.original_size = stbuf.st_size;
+                       lte->resource_entry.size = stbuf.st_size;
+                       lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
+                       lte->file_on_disk_fp = NULL;
+                       copy_hash(lte->hash, hash);
+                       lookup_table_insert(table, lte);
+               }
+       }
+       return 0;
+}
+
+static int inode_close_fds(struct inode *inode)
+{
+       u16 num_opened_fds = inode->num_opened_fds;
+       for (u16 i = 0, j = 0; j < num_opened_fds; i++) {
+               struct wimlib_fd *fd = inode->fds[i];
+               if (fd) {
+                       wimlib_assert(fd->f_inode == inode);
+                       int ret = close_wimlib_fd(fd);
+                       if (ret != 0)
+                               return ret;
+                       j++;
+               }
+       }
+       return 0;
+}
+
+/* Overwrites the WIM file, with changes saved. */
+static int rebuild_wim(struct wimfs_context *ctx, int write_flags,
+                      wimlib_progress_func_t progress_func)
+{
+       int ret;
+       struct lookup_table_entry *lte, *tmp;
+       WIMStruct *w = ctx->wim;
+
+       DEBUG("Closing all staging file descriptors.");
+       list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
+               ret = inode_close_fds(lte->lte_inode);
+               if (ret != 0)
+                       return ret;
+       }
+
+       DEBUG("Calculating SHA1 checksums for all new staging files.");
+       list_for_each_entry(lte, &ctx->staging_list, staging_list) {
+               ret = update_lte_of_staging_file(lte, w->lookup_table);
+               if (ret != 0)
+                       return ret;
+       }
+
+       xml_update_image_info(w, w->current_image);
+       ret = wimlib_overwrite(w, write_flags, 0, progress_func);
+       if (ret != 0)
+               ERROR("Failed to commit changes to mounted WIM image");
+       return ret;
+}
+
+
 
 /* Simple function that returns the concatenation of 2 strings. */
 static char *strcat_dup(const char *s1, const char *s2, size_t max_len)
@@ -622,12 +821,12 @@ static int set_message_queue_names(struct wimfs_context *ctx,
        dir_path = realpath(mount_dir, NULL);
        if (!dir_path) {
                ERROR_WITH_ERRNO("Failed to resolve path \"%s\"", mount_dir);
-               return WIMLIB_ERR_NOTDIR;
+               if (errno == ENOMEM)
+                       return WIMLIB_ERR_NOMEM;
+               else
+                       return WIMLIB_ERR_NOTDIR;
        }
 
-       DEBUG("Using absolute dir_path = `%s'", dir_path);
-
-
        p = dir_path;
        while (*p) {
                if (*p == '/')
@@ -670,57 +869,40 @@ static void free_message_queue_names(struct wimfs_context *ctx)
  * Opens two POSIX message queue: one for sending messages from the unmount
  * process to the daemon process, and one to go the other way.  The names of the
  * message queues, which must be system-wide unique, are be based on the mount
- * point.  (There of course is still a possibility of a collision if one were to
- * unmount two identically named directories simultaneously...)
+ * point.
  *
  * @daemon specifies whether the calling process is the filesystem daemon or the
  * unmount process.
  */
 static int open_message_queues(struct wimfs_context *ctx, bool daemon)
 {
-       int flags;
-       int ret;
-
-       wimlib_assert(ctx->unmount_to_daemon_mq_name != NULL &&
-                     ctx->daemon_to_unmount_mq_name != NULL);
+       int unmount_to_daemon_mq_flags = O_WRONLY | O_CREAT;
+       int daemon_to_unmount_mq_flags = O_RDONLY | O_CREAT;
 
        if (daemon)
-               flags = O_RDONLY | O_CREAT;
-       else
-               flags = O_WRONLY | O_CREAT;
+               swap(unmount_to_daemon_mq_flags, daemon_to_unmount_mq_flags);
 
        DEBUG("Opening message queue \"%s\"", ctx->unmount_to_daemon_mq_name);
        ctx->unmount_to_daemon_mq = mq_open(ctx->unmount_to_daemon_mq_name,
-                                           flags, 0700, NULL);
+                                           unmount_to_daemon_mq_flags, 0700, NULL);
 
        if (ctx->unmount_to_daemon_mq == (mqd_t)-1) {
                ERROR_WITH_ERRNO("mq_open()");
-               ret = WIMLIB_ERR_MQUEUE;
-               goto out;
+               return WIMLIB_ERR_MQUEUE;
        }
 
-       if (daemon)
-               flags = O_WRONLY | O_CREAT;
-       else
-               flags = O_RDONLY | O_CREAT;
-
        DEBUG("Opening message queue \"%s\"", ctx->daemon_to_unmount_mq_name);
        ctx->daemon_to_unmount_mq = mq_open(ctx->daemon_to_unmount_mq_name,
-                                           flags, 0700, NULL);
+                                           daemon_to_unmount_mq_flags, 0700, NULL);
 
        if (ctx->daemon_to_unmount_mq == (mqd_t)-1) {
                ERROR_WITH_ERRNO("mq_open()");
-               ret = WIMLIB_ERR_MQUEUE;
-               goto out_close_unmount_to_daemon_mq;
+               mq_close(ctx->unmount_to_daemon_mq);
+               mq_unlink(ctx->unmount_to_daemon_mq_name);
+               ctx->unmount_to_daemon_mq = (mqd_t)-1;
+               return WIMLIB_ERR_MQUEUE;
        }
-       ret = 0;
-       goto out;
-out_close_unmount_to_daemon_mq:
-       mq_close(ctx->unmount_to_daemon_mq);
-       mq_unlink(ctx->unmount_to_daemon_mq_name);
-       ctx->unmount_to_daemon_mq = (mqd_t)-1;
-out:
-       return ret;
+       return 0;
 }
 
 /* Try to determine the maximum message size of a message queue.  The return
@@ -756,7 +938,7 @@ static long mq_get_msgsize(mqd_t mq)
 }
 
 static int get_mailbox(mqd_t mq, long needed_msgsize, long *msgsize_ret,
-                      char **mailbox_ret)
+                      void **mailbox_ret)
 {
        long msgsize;
        char *mailbox;
@@ -788,6 +970,7 @@ static void unlink_message_queues(struct wimfs_context *ctx)
 /* Closes the message queues, which are allocated in static variables */
 static void close_message_queues(struct wimfs_context *ctx)
 {
+       DEBUG("Closing message queues");
        mq_close(ctx->unmount_to_daemon_mq);
        ctx->unmount_to_daemon_mq = (mqd_t)(-1);
        mq_close(ctx->daemon_to_unmount_mq);
@@ -795,217 +978,504 @@ static void close_message_queues(struct wimfs_context *ctx)
        unlink_message_queues(ctx);
 }
 
-static int wimfs_access(const char *path, int mask)
+
+struct unmount_msg_hdr {
+       u32 min_version;
+       u32 cur_version;
+       u32 msg_type;
+       u32 msg_size;
+} PACKED;
+
+struct msg_unmount_request {
+       struct unmount_msg_hdr hdr;
+       u32 unmount_flags;
+       u8 want_progress_messages;
+} PACKED;
+
+struct msg_daemon_info {
+       struct unmount_msg_hdr hdr;
+       pid_t daemon_pid;
+       u32 mount_flags;
+} PACKED;
+
+struct msg_unmount_finished {
+       struct unmount_msg_hdr hdr;
+       int32_t status;
+} PACKED;
+
+struct msg_write_streams_progress {
+       struct unmount_msg_hdr hdr;
+       union wimlib_progress_info info;
+} PACKED;
+
+enum {
+       MSG_TYPE_UNMOUNT_REQUEST,
+       MSG_TYPE_DAEMON_INFO,
+       MSG_TYPE_WRITE_STREAMS_PROGRESS,
+       MSG_TYPE_UNMOUNT_FINISHED,
+       MSG_TYPE_MAX,
+};
+
+struct msg_handler_context {
+       bool is_daemon;
+       int timeout_seconds;
+       union {
+               struct {
+                       pid_t daemon_pid;
+                       int mount_flags;
+                       int status;
+                       wimlib_progress_func_t progress_func;
+               } unmount;
+               struct {
+                       struct wimfs_context *wimfs_ctx;
+               } daemon;
+       };
+};
+
+static int send_unmount_request_msg(mqd_t mq, int unmount_flags,
+                                   u8 want_progress_messages)
 {
-       /* Permissions not implemented */
+       DEBUG("Sending unmount request msg");
+       struct msg_unmount_request msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type    = MSG_TYPE_UNMOUNT_REQUEST,
+                       .msg_size    = sizeof(msg),
+               },
+               .unmount_flags = unmount_flags,
+               .want_progress_messages = want_progress_messages,
+       };
+
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1)) {
+               ERROR_WITH_ERRNO("Failed to communicate with filesystem daemon");
+               return WIMLIB_ERR_MQUEUE;
+       }
        return 0;
 }
 
-static int wimfs_chmod(const char *path, mode_t mask)
+static int send_daemon_info_msg(mqd_t mq, pid_t pid, int mount_flags)
 {
-       struct dentry *dentry;
-       struct wimfs_context *ctx = wimfs_get_context();
-       struct inode *inode;
-       struct stat stbuf;
+       DEBUG("Sending daemon info msg (pid = %d, mount_flags=%x)",
+             pid, mount_flags);
+
+       struct msg_daemon_info msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type = MSG_TYPE_DAEMON_INFO,
+                       .msg_size = sizeof(msg),
+               },
+               .daemon_pid = pid,
+               .mount_flags = mount_flags,
+       };
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1)) {
+               ERROR_WITH_ERRNO("Failed to send daemon info to unmount process");
+               return WIMLIB_ERR_MQUEUE;
+       }
+       return 0;
+}
+
+static void send_unmount_finished_msg(mqd_t mq, int status)
+{
+       DEBUG("Sending unmount finished msg");
+       struct msg_unmount_finished msg = {
+               .hdr = {
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                       .cur_version = WIMLIB_VERSION_CODE,
+                       .msg_type = MSG_TYPE_UNMOUNT_FINISHED,
+                       .msg_size = sizeof(msg),
+               },
+               .status = status,
+       };
+       if (mq_send(mq, (void*)&msg, sizeof(msg), 1))
+               ERROR_WITH_ERRNO("Failed to send status to unmount process");
+}
+
+static int unmount_progress_func(enum wimlib_progress_msg msg,
+                                const union wimlib_progress_info *info)
+{
+       if (msg == WIMLIB_PROGRESS_MSG_WRITE_STREAMS) {
+               struct msg_write_streams_progress msg = {
+                       .hdr = {
+                               .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                               .cur_version = WIMLIB_VERSION_CODE,
+                               .msg_type = MSG_TYPE_WRITE_STREAMS_PROGRESS,
+                               .msg_size = sizeof(msg),
+                       },
+                       .info = *info,
+               };
+               if (mq_send(wimfs_get_context()->daemon_to_unmount_mq,
+                           (void*)&msg, sizeof(msg), 1))
+               {
+                       ERROR_WITH_ERRNO("Failed to send status to unmount process");
+               }
+       }
+       return 0;
+}
+
+static int msg_unmount_request_handler(const void *_msg,
+                                      struct msg_handler_context *handler_ctx)
+{
+       const struct msg_unmount_request *msg;
+       struct wimfs_context *wimfs_ctx;
+       int status = 0;
        int ret;
+       int unmount_flags;
+       wimlib_progress_func_t progress_func;
 
-       ret = lookup_resource(ctx->wim, path,
-                             get_lookup_flags(ctx) | LOOKUP_FLAG_DIRECTORY_OK,
-                             &dentry, NULL, NULL);
-       if (ret != 0)
-               return ret;
-       inode = dentry->d_inode;
-       inode_to_stbuf(inode, NULL, &stbuf);
-       if (mask == stbuf.st_mode)
-               return 0;
+       DEBUG("Handling unmount request msg");
+
+       msg = _msg;
+       wimfs_ctx = handler_ctx->daemon.wimfs_ctx;
+
+       if (msg->hdr.msg_size < sizeof(*msg)) {
+               status = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+               goto out;
+       }
+
+       unmount_flags = msg->unmount_flags;
+       if (msg->want_progress_messages)
+               progress_func = unmount_progress_func;
        else
-               return -EPERM;
+               progress_func = NULL;
 
+       ret = send_daemon_info_msg(wimfs_ctx->daemon_to_unmount_mq, getpid(),
+                                  wimfs_ctx->mount_flags);
+       if (ret != 0) {
+               status = ret;
+               goto out;
+       }
+
+       if (wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
+               if (unmount_flags & WIMLIB_UNMOUNT_FLAG_COMMIT) {
+                       int write_flags = 0;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_CHECK_INTEGRITY)
+                               write_flags |= WIMLIB_WRITE_FLAG_CHECK_INTEGRITY;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_REBUILD)
+                               write_flags |= WIMLIB_WRITE_FLAG_REBUILD;
+                       if (unmount_flags & WIMLIB_UNMOUNT_FLAG_RECOMPRESS)
+                               write_flags |= WIMLIB_WRITE_FLAG_RECOMPRESS;
+                       status = rebuild_wim(wimfs_ctx, write_flags,
+                                            progress_func);
+               }
+       } else {
+               DEBUG("Read-only mount");
+               status = 0;
+       }
+
+out:
+       if (wimfs_ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
+               ret = delete_staging_dir(wimfs_ctx);
+               if (ret != 0) {
+                       ERROR("Failed to delete the staging directory");
+                       if (status == 0)
+                               status = ret;
+               }
+       }
+       send_unmount_finished_msg(wimfs_ctx->daemon_to_unmount_mq, status);
+       return MSG_BREAK_LOOP;
 }
 
-static void inode_update_lte_ptr(struct inode *inode,
-                                struct lookup_table_entry *old_lte,
-                                struct lookup_table_entry *new_lte)
+static int msg_daemon_info_handler(const void *_msg,
+                                  struct msg_handler_context *handler_ctx)
 {
-       if (inode->lte == old_lte) {
-               inode->lte = new_lte;
+       const struct msg_daemon_info *msg = _msg;
+       DEBUG("Handling daemon info msg");
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       handler_ctx->unmount.daemon_pid = msg->daemon_pid;
+       handler_ctx->unmount.mount_flags = msg->mount_flags;
+       handler_ctx->timeout_seconds = 1;
+       DEBUG("pid of daemon is %d; mount flags were %#x",
+             handler_ctx->unmount.daemon_pid,
+             handler_ctx->unmount.mount_flags);
+       return 0;
+}
+
+static int msg_write_streams_progress_handler(const void *_msg,
+                                             struct msg_handler_context *handler_ctx)
+{
+       const struct msg_write_streams_progress *msg = _msg;
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       if (handler_ctx->unmount.progress_func) {
+               handler_ctx->unmount.progress_func(WIMLIB_PROGRESS_MSG_WRITE_STREAMS,
+                                                  &msg->info);
+       }
+       return 0;
+}
+
+static int msg_unmount_finished_handler(const void *_msg,
+                                       struct msg_handler_context *handler_ctx)
+{
+       const struct msg_unmount_finished *msg = _msg;
+       DEBUG("Handling unmount finished message");
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       handler_ctx->unmount.status = msg->status;
+       DEBUG("status is %d", handler_ctx->unmount.status);
+       return MSG_BREAK_LOOP;
+}
+
+static int unmount_timed_out_cb(struct msg_handler_context *handler_ctx)
+{
+       if (handler_ctx->unmount.daemon_pid == 0) {
+               goto out_crashed;
        } else {
-               for (unsigned i = 0; i < inode->num_ads; i++) {
-                       if (inode->ads_entries[i].lte == old_lte) {
-                               inode->ads_entries[i].lte = new_lte;
-                               break;
-                       }
+               kill(handler_ctx->unmount.daemon_pid, 0);
+               if (errno == ESRCH) {
+                       goto out_crashed;
+               } else {
+                       DEBUG("Filesystem daemon is still alive... "
+                             "Waiting another %d seconds\n",
+                             handler_ctx->timeout_seconds);
+                       return 0;
                }
        }
+out_crashed:
+       ERROR("The filesystem daemon has crashed!  Changes to the "
+             "WIM may not have been commited.");
+       return WIMLIB_ERR_FILESYSTEM_DAEMON_CRASHED;
 }
 
-static int update_lte_of_staging_file(struct lookup_table_entry *lte,
-                                     struct lookup_table *table)
+static int daemon_timed_out_cb(struct msg_handler_context *handler_ctx)
 {
-       struct lookup_table_entry *duplicate_lte;
+       DEBUG("Timed out");
+       return WIMLIB_ERR_TIMEOUT;
+}
+
+typedef int (*msg_handler_t)(const void *_msg,
+                            struct msg_handler_context *handler_ctx);
+
+struct msg_handler_callbacks {
+       int (*timed_out)(struct msg_handler_context *);
+       msg_handler_t msg_handlers[MSG_TYPE_MAX];
+};
+
+static const struct msg_handler_callbacks unmount_msg_handler_callbacks = {
+       .timed_out = unmount_timed_out_cb,
+       .msg_handlers = {
+               [MSG_TYPE_DAEMON_INFO] = msg_daemon_info_handler,
+               [MSG_TYPE_WRITE_STREAMS_PROGRESS] = msg_write_streams_progress_handler,
+               [MSG_TYPE_UNMOUNT_FINISHED] = msg_unmount_finished_handler,
+       },
+};
+
+static const struct msg_handler_callbacks daemon_msg_handler_callbacks = {
+       .timed_out = daemon_timed_out_cb,
+       .msg_handlers = {
+               [MSG_TYPE_UNMOUNT_REQUEST] = msg_unmount_request_handler,
+       },
+};
+
+static int receive_message(mqd_t mq, struct msg_handler_context *handler_ctx,
+                          const msg_handler_t msg_handlers[],
+                          long mailbox_size, void *mailbox)
+{
+       struct timeval now;
+       struct timespec timeout;
+       ssize_t bytes_received;
+       struct unmount_msg_hdr *hdr;
        int ret;
-       u8 hash[SHA1_HASH_SIZE];
-       struct stat stbuf;
 
-       wimlib_assert(lte->resource_location == RESOURCE_IN_STAGING_FILE);
-       wimlib_assert(lte->staging_file_name);
+       gettimeofday(&now, NULL);
+       timeout.tv_sec = now.tv_sec + handler_ctx->timeout_seconds;
+       timeout.tv_nsec = now.tv_usec * 1000;
 
-       ret = sha1sum(lte->staging_file_name, hash);
+       bytes_received = mq_timedreceive(mq, mailbox,
+                                        mailbox_size, NULL, &timeout);
+       hdr = mailbox;
+       if (bytes_received == -1) {
+               if (errno == ETIMEDOUT) {
+                       ret = WIMLIB_ERR_TIMEOUT;
+               } else {
+                       ERROR_WITH_ERRNO("mq_timedreceive()");
+                       ret = WIMLIB_ERR_MQUEUE;
+               }
+       } else if (bytes_received < sizeof(*hdr) ||
+                  bytes_received != hdr->msg_size) {
+               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       } else if (WIMLIB_VERSION_CODE < hdr->min_version) {
+               /*ERROR("Cannot understand the received message. "*/
+                     /*"Please upgrade wimlib to at least v%d.%d.%d",*/
+                     /*WIMLIB_GET_MAJOR_VERSION(hdr->min_version),*/
+                     /*WIMLIB_GET_MINOR_VERSION(hdr->min_version),*/
+                     /*WIMLIB_GET_PATCH_VERSION(hdr->min_version));*/
+               ret = MSG_VERSION_TOO_HIGH;
+       } else if (hdr->msg_type >= MSG_TYPE_MAX) {
+               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       } else if (msg_handlers[hdr->msg_type] == NULL) {
+               ret = WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       } else {
+               ret = msg_handlers[hdr->msg_type](mailbox, handler_ctx);
+       }
+       return ret;
+}
+
+static int message_loop(mqd_t mq,
+                       const struct msg_handler_callbacks *callbacks,
+                       struct msg_handler_context *handler_ctx)
+{
+       static const size_t MAX_MSG_SIZE = 512;
+       long msgsize;
+       void *mailbox;
+       int ret;
+
+       DEBUG("Entering message loop");
+
+       ret = get_mailbox(mq, MAX_MSG_SIZE, &msgsize, &mailbox);
        if (ret != 0)
                return ret;
+       while (1) {
+               ret = receive_message(mq, handler_ctx,
+                                     callbacks->msg_handlers,
+                                     msgsize, mailbox);
+               if (ret == 0 || ret == MSG_VERSION_TOO_HIGH) {
+                       continue;
+               } else if (ret == MSG_BREAK_LOOP) {
+                       ret = 0;
+                       break;
+               } else if (ret == WIMLIB_ERR_TIMEOUT) {
+                       if (callbacks->timed_out)
+                               ret = callbacks->timed_out(handler_ctx);
+                       if (ret == 0)
+                               continue;
+                       else
+                               break;
+               } else {
+                       ERROR_WITH_ERRNO("Error communicating with "
+                                        "filesystem daemon");
+                       break;
+               }
+       }
+       FREE(mailbox);
+       DEBUG("Exiting message loop");
+       return ret;
+}
 
-       lookup_table_unlink(table, lte);
+/* Execute `fusermount -u', which is installed setuid root, to unmount the WIM.
+ *
+ * FUSE does not yet implement synchronous unmounts.  This means that fusermount
+ * -u will return before the filesystem daemon returns from wimfs_destroy().
+ *  This is partly what we want, because we need to send a message from this
+ *  process to the filesystem daemon telling whether --commit was specified or
+ *  not.  However, after that, the unmount process must wait for the filesystem
+ *  daemon to finish writing the WIM file.
+ */
+static int execute_fusermount(const char *dir)
+{
+       pid_t pid;
+       int ret;
+       int status;
 
-       duplicate_lte = __lookup_resource(table, hash);
+       pid = fork();
+       if (pid == -1) {
+               ERROR_WITH_ERRNO("Failed to fork()");
+               return WIMLIB_ERR_FORK;
+       }
+       if (pid == 0) {
+               /* Child */
+               execlp("fusermount", "fusermount", "-u", dir, NULL);
+               ERROR_WITH_ERRNO("Failed to execute `fusermount'");
+               exit(WIMLIB_ERR_FUSERMOUNT);
+       }
 
-       if (duplicate_lte) {
-               /* Merge duplicate lookup table entries */
-               duplicate_lte->refcnt += lte->refcnt;
-               list_del(&lte->staging_list);
-               inode_update_lte_ptr(lte->lte_inode, lte, duplicate_lte);
-               free_lookup_table_entry(lte);
-       } else {
-               if (stat(lte->staging_file_name, &stbuf) != 0) {
-                       ERROR_WITH_ERRNO("Failed to stat `%s'", lte->staging_file_name);
-                       return WIMLIB_ERR_STAT;
+       /* Parent */
+       ret = wait(&status);
+       if (ret == -1) {
+               ERROR_WITH_ERRNO("Failed to wait for fusermount process to "
+                                "terminate");
+               return WIMLIB_ERR_FUSERMOUNT;
+       }
+
+       if (status != 0) {
+               if (status == WIMLIB_ERR_FUSERMOUNT)
+                       ERROR("Could not find the `fusermount' program");
+               else
+                       ERROR("fusermount exited with status %d", status);
+
+               /* Try again, but with the `umount' program.  This is required
+                * on other FUSE implementations such as FreeBSD's that do not
+                * have a `fusermount' program. */
+
+               pid = fork();
+               if (pid == -1) {
+                       ERROR_WITH_ERRNO("Failed to fork()");
+                       return WIMLIB_ERR_FORK;
+               }
+               if (pid == 0) {
+                       /* Child */
+                       execlp("umount", "umount", dir, NULL);
+                       ERROR_WITH_ERRNO("Failed to execute `umount'");
+                       exit(WIMLIB_ERR_FUSERMOUNT);
+               }
+
+               /* Parent */
+               ret = wait(&status);
+               if (ret == -1) {
+                       ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
+                                        "terminate");
+                       return WIMLIB_ERR_FUSERMOUNT;
                }
-               if (stbuf.st_size == 0) {
-                       /* Zero-length stream.  No lookup table entry needed. */
-                       inode_update_lte_ptr(lte->lte_inode, lte, NULL);
-                       free_lookup_table_entry(lte);
-               } else {
-                       wimlib_assert(&lte->file_on_disk == &lte->staging_file_name);
-                       lte->resource_entry.original_size = stbuf.st_size;
-                       lte->resource_entry.size = stbuf.st_size;
-                       lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
-                       lte->lte_inode = NULL;
-                       copy_hash(lte->hash, hash);
-                       lookup_table_insert(table, lte);
+               if (status != 0) {
+                       ERROR("`umount' exited with failure status");
+                       return WIMLIB_ERR_FUSERMOUNT;
                }
        }
        return 0;
 }
 
-static int inode_close_fds(struct inode *inode)
+static int wimfs_access(const char *path, int mask)
 {
-       u16 num_opened_fds = inode->num_opened_fds;
-       for (u16 i = 0, j = 0; j < num_opened_fds; i++) {
-               struct wimlib_fd *fd = inode->fds[i];
-               if (fd) {
-                       wimlib_assert(fd->f_inode == inode);
-                       int ret = close_wimlib_fd(fd);
-                       if (ret != 0)
-                               return ret;
-                       j++;
-               }
-       }
+       /* Permissions not implemented */
        return 0;
 }
 
-/* Overwrites the WIM file, with changes saved. */
-static int rebuild_wim(struct wimfs_context *ctx, int write_flags)
+static int wimfs_chmod(const char *path, mode_t mask)
 {
+       struct dentry *dentry;
+       struct wimfs_context *ctx = wimfs_get_context();
+       struct inode *inode;
+       struct stat stbuf;
        int ret;
-       struct lookup_table_entry *lte, *tmp;
-       WIMStruct *w = ctx->wim;
-
-
-       DEBUG("Closing all staging file descriptors.");
-       list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
-               ret = inode_close_fds(lte->lte_inode);
-               if (ret != 0)
-                       return ret;
-       }
-
-       DEBUG("Calculating SHA1 checksums for all new staging files.");
-       list_for_each_entry_safe(lte, tmp, &ctx->staging_list, staging_list) {
-               ret = update_lte_of_staging_file(lte, ctx->wim->lookup_table);
-               if (ret != 0)
-                       return ret;
-       }
-
-       xml_update_image_info(w, w->current_image);
 
-       ret = wimlib_overwrite(w, write_flags, 0, NULL);
-       if (ret != 0) {
-               ERROR("Failed to commit changes");
+       ret = lookup_resource(ctx->wim, path,
+                             get_lookup_flags(ctx) | LOOKUP_FLAG_DIRECTORY_OK,
+                             &dentry, NULL, NULL);
+       if (ret != 0)
                return ret;
-       }
-       return ret;
+       inode = dentry->d_inode;
+       inode_to_stbuf(inode, NULL, &stbuf);
+       if (mask == stbuf.st_mode)
+               return 0;
+       else
+               return -EPERM;
 }
 
 /* Called when the filesystem is unmounted. */
 static void wimfs_destroy(void *p)
 {
-       /* For read-write mounts, the `imagex unmount' command, which is
-        * running in a separate process and is executing the
-        * wimlib_unmount() function, will send this process a byte
-        * through a message queue that indicates whether the --commit
-        * option was specified or not. */
+       struct wimfs_context *wimfs_ctx;
 
-       long msgsize;
-       struct timespec timeout;
-       struct timeval now;
-       ssize_t bytes_received;
-       int ret;
-       char commit;
-       char check_integrity;
-       char status;
-       char *mailbox;
-       struct wimfs_context *ctx = wimfs_get_context();
-       int write_flags;
+       wimfs_ctx = wimfs_get_context();
 
-       if (open_message_queues(ctx, true))
+       if (open_message_queues(wimfs_ctx, true))
                return;
 
-       if (get_mailbox(ctx->unmount_to_daemon_mq, 2, &msgsize, &mailbox))
-               goto out_close_message_queues;
-
-       /* Wait at most 3 seconds before giving up and discarding changes. */
-       gettimeofday(&now, NULL);
-       timeout.tv_sec = now.tv_sec + 3;
-       timeout.tv_nsec = now.tv_usec * 1000;
-       DEBUG("Waiting for message telling us whether to commit or not, and "
-             "whether to include integrity checks.");
+       struct msg_handler_context handler_ctx = {
+               .is_daemon = true,
+               .timeout_seconds = 5,
+               .daemon = {
+                       .wimfs_ctx = wimfs_ctx,
+               },
+       };
 
-       bytes_received = mq_timedreceive(ctx->unmount_to_daemon_mq, mailbox,
-                                        msgsize, NULL, &timeout);
-       if (bytes_received == -1) {
-               ERROR_WITH_ERRNO("mq_timedreceive()");
-               ERROR("Not committing.");
-               commit = 0;
-               check_integrity = 0;
-       } else {
-               DEBUG("Received message: [%d %d]", mailbox[0], mailbox[1]);
-               commit = mailbox[0];
-               check_integrity = mailbox[1];
-       }
+       message_loop(wimfs_ctx->unmount_to_daemon_mq,
+                    &daemon_msg_handler_callbacks,
+                    &handler_ctx);
 
-       status = 0;
-       if (ctx->mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
-               if (commit) {
-                       if (check_integrity)
-                               write_flags = WIMLIB_WRITE_FLAG_CHECK_INTEGRITY;
-                       else
-                               write_flags = 0;
-                       status = rebuild_wim(ctx, write_flags);
-               }
-               ret = delete_staging_dir(ctx);
-               if (ret != 0) {
-                       ERROR("Failed to delete the staging directory");
-                       if (status == 0)
-                               status = ret;
-               }
-       } else {
-               DEBUG("Read-only mount");
-       }
-       DEBUG("Sending status %hhd", status);
-       ret = mq_send(ctx->daemon_to_unmount_mq, &status, 1, 1);
-       if (ret == -1)
-               ERROR_WITH_ERRNO("Failed to send status to unmount process");
-       FREE(mailbox);
-out_close_message_queues:
-       close_message_queues(ctx);
+       close_message_queues(wimfs_ctx);
 }
 
 #if 0
@@ -1109,12 +1579,16 @@ static int wimfs_link(const char *to, const char *from)
        struct inode *inode;
        struct lookup_table_entry *lte;
        WIMStruct *w = wimfs_get_WIMStruct();
+       u16 i;
 
        inode = wim_pathname_to_inode(w, to);
        if (!inode)
                return -ENOENT;
 
-       if (!inode_is_regular_file(inode))
+       if (inode->attributes & FILE_ATTRIBUTE_REPARSE_POINT)
+               return -EEXIST;
+
+       if (inode->attributes & FILE_ATTRIBUTE_DIRECTORY)
                return -EPERM;
 
        from_dentry_parent = get_parent_dentry(w, from);
@@ -1130,17 +1604,15 @@ static int wimfs_link(const char *to, const char *from)
        if (!from_dentry)
                return -ENOMEM;
 
-
        inode_add_dentry(from_dentry, inode);
        from_dentry->d_inode = inode;
        inode->link_count++;
 
-       for (unsigned i = 0; i <= inode->num_ads; i++) {
+       for (i = 0; i <= inode->num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte->refcnt++;
        }
-
        dentry_add_child(from_dentry_parent, from_dentry);
        return 0;
 }
@@ -1151,6 +1623,8 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
        size_t needed_size;
        struct inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
+       u16 i;
+       char *p;
 
        if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR))
                return -ENOTSUP;
@@ -1163,12 +1637,12 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
 
        if (size == 0) {
                needed_size = 0;
-               for (u16 i = 0; i < inode->num_ads; i++)
+               for (i = 0; i < inode->num_ads; i++)
                        needed_size += inode->ads_entries[i].stream_name_utf8_len + 6;
                return needed_size;
        } else {
-               char *p = list;
-               for (u16 i = 0; i < inode->num_ads; i++) {
+               p = list;
+               for (i = 0; i < inode->num_ads; i++) {
                        needed_size = inode->ads_entries[i].stream_name_utf8_len + 6;
                        if (needed_size > size)
                                return -ERANGE;
@@ -1181,37 +1655,22 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
 }
 #endif
 
-/*
- * Create a directory in the WIM.
- * @mode is currently ignored.
- */
+
+/* Create a directory in the WIM.
+ * @mode is currently ignored.  */
 static int wimfs_mkdir(const char *path, mode_t mode)
 {
-       struct dentry *parent;
-       struct dentry *newdir;
-       const char *basename;
-       struct wimfs_context *ctx = wimfs_get_context();
-
-       parent = get_parent_dentry(ctx->wim, path);
-       if (!parent)
-               return -ENOENT;
-
-       if (!dentry_is_directory(parent))
-               return -ENOTDIR;
-
-       basename = path_basename(path);
-       if (get_dentry_child_with_name(parent, basename))
-               return -EEXIST;
+       struct dentry *dentry;
+       int ret;
 
-       newdir = new_dentry_with_inode(basename);
-       newdir->d_inode->attributes |= FILE_ATTRIBUTE_DIRECTORY;
-       newdir->d_inode->resolved = true;
-       newdir->d_inode->ino = ctx->next_ino++;
-       dentry_add_child(parent, newdir);
-       return 0;
+       ret = create_dentry(wimfs_get_context(), path, &dentry);
+       if (ret == 0)
+               dentry->d_inode->attributes = FILE_ATTRIBUTE_DIRECTORY;
+       return ret;
 }
 
-/* Creates a regular file. */
+/* Create a regular file in the WIM.
+ * @mode is currently ignored.  */
 static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 {
        const char *stream_name;
@@ -1229,39 +1688,25 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
                inode = wim_pathname_to_inode(ctx->wim, path);
                if (!inode)
                        return -ENOENT;
-               if (!inode_is_regular_file(inode))
+               if (inode->attributes &
+                   (FILE_ATTRIBUTE_REPARSE_POINT | FILE_ATTRIBUTE_DIRECTORY))
                        return -ENOENT;
                if (inode_get_ads_entry(inode, stream_name, NULL))
                        return -EEXIST;
                new_entry = inode_add_ads(inode, stream_name);
                if (!new_entry)
                        return -ENOMEM;
+               return 0;
        } else {
-               struct dentry *dentry, *parent;
-               const char *basename;
+               struct dentry *dentry;
+               int ret;
 
                /* Make a normal file (not an alternate data stream) */
-
-               /* Make sure that the parent of @path exists and is a directory, and
-                * that the dentry named by @path does not already exist.  */
-               parent = get_parent_dentry(ctx->wim, path);
-               if (!parent)
-                       return -ENOENT;
-               if (!dentry_is_directory(parent))
-                       return -ENOTDIR;
-
-               basename = path_basename(path);
-               if (get_dentry_child_with_name(parent, path))
-                       return -EEXIST;
-
-               dentry = new_dentry_with_inode(basename);
-               if (!dentry)
-                       return -ENOMEM;
-               dentry->d_inode->resolved = true;
-               dentry->d_inode->ino = ctx->next_ino++;
-               dentry_add_child(parent, dentry);
+               ret = create_dentry(ctx, path, &dentry);
+               if (ret == 0)
+                       dentry->d_inode->attributes = FILE_ATTRIBUTE_NORMAL;
+               return ret;
        }
-       return 0;
 }
 
 
@@ -1313,8 +1758,9 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
        if (lte && lte->resource_location == RESOURCE_IN_STAGING_FILE) {
                fd->staging_fd = open(lte->staging_file_name, fi->flags);
                if (fd->staging_fd == -1) {
+                       int errno_save = errno;
                        close_wimlib_fd(fd);
-                       return -errno;
+                       return -errno_save;
                }
        }
        fi->fh = (uintptr_t)fd;
@@ -1348,6 +1794,7 @@ static int wimfs_read(const char *path, char *buf, size_t size,
                      off_t offset, struct fuse_file_info *fi)
 {
        struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       ssize_t ret;
 
        if (!fd)
                return -EBADF;
@@ -1361,7 +1808,6 @@ static int wimfs_read(const char *path, char *buf, size_t size,
                wimlib_assert(fd->f_lte->staging_file_name);
                wimlib_assert(fd->staging_fd != -1);
 
-               ssize_t ret;
                DEBUG("Seek to offset %zu", offset);
 
                if (lseek(fd->staging_fd, offset, SEEK_SET) == -1)
@@ -1372,16 +1818,10 @@ static int wimfs_read(const char *path, char *buf, size_t size,
                return ret;
        } else {
                /* Read from WIM */
-
-               wimlib_assert(fd->f_lte->resource_location == RESOURCE_IN_WIM);
-
                u64 res_size = wim_resource_size(fd->f_lte);
-
                if (offset > res_size)
                        return -EOVERFLOW;
-
                size = min(size, res_size - offset);
-
                if (read_wim_resource(fd->f_lte, (u8*)buf,
                                      size, offset,
                                      WIMLIB_RESOURCE_FLAG_MULTITHREADED) != 0)
@@ -1507,7 +1947,6 @@ static int wimfs_rename(const char *from, const char *to)
 
        dst = get_dentry(w, to);
 
-
        ret = get_names(&file_name_utf16, &file_name_utf8,
                        &file_name_utf16_len, &file_name_utf8_len,
                        path_basename(to));
@@ -1644,39 +2083,23 @@ static int wimfs_setxattr(const char *path, const char *name,
 
 static int wimfs_symlink(const char *to, const char *from)
 {
-       struct dentry *dentry_parent, *dentry;
-       const char *link_name;
-       struct inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
+       struct dentry *dentry;
+       int ret;
 
-       dentry_parent = get_parent_dentry(ctx->wim, from);
-       if (!dentry_parent)
-               return -ENOENT;
-       if (!dentry_is_directory(dentry_parent))
-               return -ENOTDIR;
-
-       link_name = path_basename(from);
-
-       if (get_dentry_child_with_name(dentry_parent, link_name))
-               return -EEXIST;
-       dentry = new_dentry_with_inode(link_name);
-       if (!dentry)
-               return -ENOMEM;
-       inode = dentry->d_inode;
-
-       inode->attributes  = FILE_ATTRIBUTE_REPARSE_POINT;
-       inode->reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
-       inode->ino         = ctx->next_ino++;
-       inode->resolved    = true;
-
-       if (inode_set_symlink(inode, to, ctx->wim->lookup_table, NULL) != 0)
-               goto out_free_dentry;
-
-       dentry_add_child(dentry_parent, dentry);
-       return 0;
-out_free_dentry:
-       free_dentry(dentry);
-       return -ENOMEM;
+       ret = create_dentry(ctx, from, &dentry);
+       if (ret == 0) {
+               dentry->d_inode->attributes = FILE_ATTRIBUTE_REPARSE_POINT;
+               dentry->d_inode->reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
+               if (inode_set_symlink(dentry->d_inode, to,
+                                     ctx->wim->lookup_table, NULL))
+               {
+                       unlink_dentry(dentry);
+                       free_dentry(dentry);
+                       ret = -ENOMEM;
+               }
+       }
+       return ret;
 }
 
 
@@ -1697,24 +2120,20 @@ static int wimfs_truncate(const char *path, off_t size)
        if (ret != 0)
                return ret;
 
-       if (!lte) /* Already a zero-length file */
+       if (lte == NULL && size == 0)
                return 0;
 
        inode = dentry->d_inode;
-
        if (stream_idx == 0)
                stream_id = 0;
        else
                stream_id = inode->ads_entries[stream_idx - 1].stream_id;
 
        if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
-               wimlib_assert(lte->staging_file_name);
                ret = truncate(lte->staging_file_name, size);
                if (ret != 0)
-                       return -errno;
-               lte->resource_entry.original_size = size;
+                       ret = -errno;
        } else {
-               wimlib_assert(lte->resource_location == RESOURCE_IN_WIM);
                /* File in WIM.  Extract it to the staging directory, but only
                 * the first @size bytes of it. */
                ret = extract_resource_to_staging_dir(inode, stream_id,
@@ -1723,7 +2142,7 @@ static int wimfs_truncate(const char *path, off_t size)
        return ret;
 }
 
-/* Remove a regular file */
+/* Unlink a non-directory or alternate data stream */
 static int wimfs_unlink(const char *path)
 {
        struct dentry *dentry;
@@ -1895,6 +2314,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        struct lookup_table *joined_tab, *wim_tab_save;
        struct image_metadata *imd;
        struct wimfs_context ctx;
+       struct hlist_node *cur_node;
+       struct inode *inode;
 
        DEBUG("Mount: wim = %p, image = %d, dir = %s, flags = %d, ",
              wim, image, dir, mount_flags);
@@ -1928,14 +2349,13 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        }
 
        ret = select_wim_image(wim, image);
-
        if (ret != 0)
                goto out;
 
-       imd = &wim->image_metadata[image - 1];
-
        DEBUG("Selected image %d", image);
 
+       imd = wim_get_current_image_metadata(wim);
+
        if (imd->root_dentry->refcnt != 1) {
                ERROR("Cannot mount image that was just exported with "
                      "wimlib_export_image()");
@@ -1951,7 +2371,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        }
 
        if (mount_flags & WIMLIB_MOUNT_FLAG_READWRITE) {
-               ret = lock_wim(wim->fp, wim->filename);
+               ret = lock_wim(wim, wim->fp);
                if (ret != 0)
                        goto out;
        }
@@ -1966,11 +2386,15 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        init_wimfs_context(&ctx);
        ctx.wim = wim;
        ctx.mount_flags = mount_flags;
+       ctx.image_inode_list = &imd->inode_list;
+
+       if (mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
+               ctx.default_lookup_flags = LOOKUP_FLAG_ADS_OK;
 
        DEBUG("Unlinking message queues in case they already exist");
        ret = set_message_queue_names(&ctx, dir);
        if (ret != 0)
-               goto out;
+               goto out_unlock;
        unlink_message_queues(&ctx);
 
        DEBUG("Preparing arguments to fuse_main()");
@@ -1995,7 +2419,14 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
         * assign_inode_numbers() function, and the static variable @next_ino is
         * set to the next available inode number.
         */
-       char optstring[256] = "use_ino,subtype=wimfs,attr_timeout=0";
+       char optstring[256] =
+               "use_ino"
+               ",subtype=wimfs"
+               ",attr_timeout=0"
+#if FUSE_MAJOR_VERSION > 2 || (FUSE_MAJOR_VERSION == 2 && FUSE_MINOR_VERSION >= 8)
+               ",hard_remove"
+#endif
+               ;
        argv[argc++] = "-o";
        argv[argc++] = optstring;
        if ((mount_flags & WIMLIB_MOUNT_FLAG_READWRITE)) {
@@ -2028,15 +2459,18 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                imd->has_been_mounted_rw = 1;
        }
 
-       /* Resolve all the lookup table entries of the dentry tree */
-       DEBUG("Resolving lookup table entries");
-       for_dentry_in_tree(imd->root_dentry, dentry_resolve_ltes,
-                          wim->lookup_table);
+       /* Resolve the lookup table entries for every inode in the image, and
+        * assign inode numbers */
+       DEBUG("Resolving lookup table entries and assigning inode numbers");
 
-       ctx.next_ino = assign_inode_numbers(&imd->inode_list);
+       ctx.next_ino = 1;
+       hlist_for_each_entry(inode, cur_node, &imd->inode_list, hlist) {
+               inode_resolve_ltes(inode, wim->lookup_table);
+               inode->ino = ctx.next_ino++;
+       }
+       /*ctx.next_ino = assign_inode_numbers(&imd->inode_list);*/
        DEBUG("(next_ino = %"PRIu64")", ctx.next_ino);
 
-
        DEBUG("Calling fuse_main()");
 
        ret = fuse_main(argc, argv, &wimfs_operations, &ctx);
@@ -2046,6 +2480,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                ret = WIMLIB_ERR_FUSE;
 out_free_dir_copy:
        FREE(dir_copy);
+out_unlock:
+       wim->wim_locked = 0;
 out_free_message_queue_names:
        free_message_queue_names(&ctx);
 out:
@@ -2056,7 +2492,6 @@ out:
        return ret;
 }
 
-
 /*
  * Unmounts the WIM file that was previously mounted on @dir by using
  * wimlib_mount_image().
@@ -2064,167 +2499,47 @@ out:
 WIMLIBAPI int wimlib_unmount_image(const char *dir, int unmount_flags,
                                   wimlib_progress_func_t progress_func)
 {
-       pid_t pid;
-       int status;
        int ret;
-       char msg[2];
-       struct timeval now;
-       struct timespec timeout;
-       long msgsize;
-       struct wimfs_context ctx;
-       char *mailbox;
+       struct wimfs_context wimfs_ctx;
 
-       init_wimfs_context(&ctx);
+       init_wimfs_context(&wimfs_ctx);
 
-       /* Open message queues between the unmount process and the
-        * filesystem daemon. */
-       ret = set_message_queue_names(&ctx, dir);
+       ret = set_message_queue_names(&wimfs_ctx, dir);
        if (ret != 0)
                goto out;
 
-       ret = open_message_queues(&ctx, false);
+       ret = open_message_queues(&wimfs_ctx, false);
        if (ret != 0)
                goto out_free_message_queue_names;
 
-       /* Send a message to the filesystem daemon saying whether to commit or
-        * not. */
-       msg[0] = (unmount_flags & WIMLIB_UNMOUNT_FLAG_COMMIT) ? 1 : 0;
-       msg[1] = (unmount_flags & WIMLIB_UNMOUNT_FLAG_CHECK_INTEGRITY) ? 1 : 0;
-
-       DEBUG("Sending message: %scommit, %scheck",
-                       (msg[0] ? "" : "don't "),
-                       (msg[1] ? "" : "don't "));
-       ret = mq_send(ctx.unmount_to_daemon_mq, msg, 2, 1);
-       if (ret == -1) {
-               ERROR_WITH_ERRNO("Failed to notify filesystem daemon whether "
-                                "we want to commit changes or not");
-               ret = WIMLIB_ERR_MQUEUE;
-               goto out_close_message_queues;
-       }
-
-       /* Execute `fusermount -u', which is installed setuid root, to unmount
-        * the WIM.
-        *
-        * FUSE does not yet implement synchronous unmounts.  This means that
-        * fusermount -u will return before the filesystem daemon returns from
-        * wimfs_destroy().  This is partly what we want, because we need to
-        * send a message from this process to the filesystem daemon telling
-        * whether --commit was specified or not.  However, after that, the
-        * unmount process must wait for the filesystem daemon to finish writing
-        * the WIM file.
-        */
-
-       pid = fork();
-       if (pid == -1) {
-               ERROR_WITH_ERRNO("Failed to fork()");
-               ret = WIMLIB_ERR_FORK;
-               goto out_close_message_queues;
-       }
-       if (pid == 0) {
-               /* Child */
-               execlp("fusermount", "fusermount", "-u", dir, NULL);
-               ERROR_WITH_ERRNO("Failed to execute `fusermount'");
-               exit(WIMLIB_ERR_FUSERMOUNT);
-       }
-
-       /* Parent */
-       ret = wait(&status);
-       if (ret == -1) {
-               ERROR_WITH_ERRNO("Failed to wait for fusermount process to "
-                                "terminate");
-               ret = WIMLIB_ERR_FUSERMOUNT;
+       ret = send_unmount_request_msg(wimfs_ctx.unmount_to_daemon_mq,
+                                      unmount_flags,
+                                      progress_func != NULL);
+       if (ret != 0)
                goto out_close_message_queues;
-       }
-
-       if (status != 0) {
-               ERROR("fusermount exited with status %d", status);
-
-               /* Try again, but with the `umount' program.  This is required
-                * on other FUSE implementations such as FreeBSD's that do not
-                * have a `fusermount' program. */
-
-               pid = fork();
-               if (pid == -1) {
-                       ERROR_WITH_ERRNO("Failed to fork()");
-                       ret = WIMLIB_ERR_FORK;
-                       goto out_close_message_queues;
-               }
-               if (pid == 0) {
-                       /* Child */
-                       execlp("umount", "umount", dir, NULL);
-                       ERROR_WITH_ERRNO("Failed to execute `umount'");
-                       exit(WIMLIB_ERR_FUSERMOUNT);
-               }
-
-               /* Parent */
-               ret = wait(&status);
-               if (ret == -1) {
-                       ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
-                                        "terminate");
-                       ret = WIMLIB_ERR_FUSERMOUNT;
-                       goto out_close_message_queues;
-               }
-               if (status != 0) {
-                       ERROR("`umount' exited with failure status");
-                       ret = WIMLIB_ERR_FUSERMOUNT;
-                       goto out_close_message_queues;
-               }
-       }
-
-       wimlib_assert(status == 0);
 
-       /* Wait for a message from the filesytem daemon indicating whether  the
-        * filesystem was unmounted successfully (0) or an error occurred (1).
-        * This may take a long time if a big WIM file needs to be rewritten. */
-
-       /* Wait at most 600??? seconds before giving up and returning false.
-        * Either it's a really big WIM file, or (more likely) the
-        * filesystem daemon has crashed or failed for some reason.
-        *
-        * XXX come up with some method to determine if the filesystem
-        * daemon has really crashed or not.
-        *
-        * XXX Idea: have mount daemon write its PID into the WIM file header?
-        * No, this wouldn't work because we know the directory but not the WIM
-        * file...
-        * */
-
-       gettimeofday(&now, NULL);
-       timeout.tv_sec = now.tv_sec + 600;
-       timeout.tv_nsec = now.tv_usec * 1000;
-
-       ret = get_mailbox(ctx.daemon_to_unmount_mq, 2, &msgsize, &mailbox);
+       ret = execute_fusermount(dir);
        if (ret != 0)
                goto out_close_message_queues;
 
-       mailbox[0] = 0;
-       DEBUG("Waiting for message telling us whether the unmount was "
-                       "successful or not.");
-       ret = mq_timedreceive(ctx.daemon_to_unmount_mq, mailbox,
-                             msgsize, NULL, &timeout);
-       if (ret == -1) {
-               if (errno == ETIMEDOUT) {
-                       ERROR("Timed out- probably the filesystem daemon "
-                             "crashed and the WIM was not written "
-                             "successfully.");
-                       ret = WIMLIB_ERR_TIMEOUT;
-               } else {
-                       ERROR_WITH_ERRNO("mq_receive()");
-                       ret = WIMLIB_ERR_MQUEUE;
-               }
-               goto out_free_mailbox;
+       struct msg_handler_context handler_ctx = {
+               .is_daemon = false,
+               .timeout_seconds = 5,
+               .unmount = {
+                       .daemon_pid = 0,
+                       .progress_func = progress_func,
+               },
+       };
 
-       }
-       DEBUG("Received message: Unmount %s", (mailbox[0] ? "Failed" : "Ok"));
-       ret = mailbox[0];
-       if (ret)
-               ERROR("Unmount failed");
-out_free_mailbox:
-       FREE(mailbox);
+       ret = message_loop(wimfs_ctx.daemon_to_unmount_mq,
+                          &unmount_msg_handler_callbacks,
+                          &handler_ctx);
+       if (ret == 0)
+               ret = handler_ctx.unmount.status;
 out_close_message_queues:
-       close_message_queues(&ctx);
+       close_message_queues(&wimfs_ctx);
 out_free_message_queue_names:
-       free_message_queue_names(&ctx);
+       free_message_queue_names(&wimfs_ctx);
 out:
        return ret;
 }