]> wimlib.net Git - wimlib/blobdiff - src/mount_image.c
wimfs_rename(): Use path_basename(to)
[wimlib] / src / mount_image.c
index 7bede83da5026abd01e202c0e0f62e970d441511..e5d7fb98462b4fc5a853e918a82348d137f0f5ba 100644 (file)
@@ -47,6 +47,7 @@
 #include <ftw.h>
 #include <mqueue.h>
 #include <utime.h>
+#include <signal.h>
 
 #ifdef ENABLE_XATTR
 #include <attr/xattr.h>
@@ -56,9 +57,9 @@
 #define MSG_BREAK_LOOP         -2
 
 /* File descriptor to a file open on the WIM filesystem. */
-struct wimlib_fd {
-       struct inode *f_inode;
-       struct lookup_table_entry *f_lte;
+struct wimfs_fd {
+       struct wim_inode *f_inode;
+       struct wim_lookup_table_entry *f_lte;
        int staging_fd;
        u16 idx;
        u32 stream_id;
@@ -79,23 +80,28 @@ struct wimfs_context {
        /* Flags passed to wimlib_mount(). */
        int mount_flags;
 
+       /* Default flags to use when looking up a WIM dentry (depends on whether
+        * the Windows interface to alternate data streams is being used or
+        * not). */
        int default_lookup_flags;
 
-       /* Next inode number to be assigned. */
+       /* Next inode number to be assigned.  Note: I didn't bother with a
+        * bitmap of free inode numbers since this isn't even a "real"
+        * filesystem anyway. */
        u64 next_ino;
 
-       /* List of lookup table entries in the staging directory */
+       /* List of lookup table entries for files 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
-        * unmount' command) needs to communicate with the filesystem daemon running
-        * fuse_main() (i.e. that spawned by the `imagex mount' or `imagex mountrw'
-        * commands */
+       /* 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_unmount_image() (i.e. the `imagex unmount' command) needs to
+        * communicate with the filesystem daemon running fuse_main() (i.e. the
+        * daemon created by the `imagex mount' or `imagex mountrw' commands */
        char *unmount_to_daemon_mq_name;
        char *daemon_to_unmount_mq_name;
        mqd_t unmount_to_daemon_mq;
@@ -147,11 +153,11 @@ static inline int flags_writable(int open_flags)
  *
  * Return 0 iff successful or error code if unsuccessful.
  */
-static int alloc_wimlib_fd(struct inode *inode,
-                          u32 stream_id,
-                          struct lookup_table_entry *lte,
-                          struct wimlib_fd **fd_ret,
-                          bool readonly)
+static int alloc_wimfs_fd(struct wim_inode *inode,
+                         u32 stream_id,
+                         struct wim_lookup_table_entry *lte,
+                         struct wimfs_fd **fd_ret,
+                         bool readonly)
 {
        static const u16 fds_per_alloc = 8;
        static const u16 max_fds = 0xffff;
@@ -159,36 +165,37 @@ static int alloc_wimlib_fd(struct inode *inode,
 
        pthread_mutex_lock(&inode->i_mutex);
 
-       DEBUG("Allocating fd for stream ID %u from inode %lx (open = %u, allocated = %u)",
-             stream_id, inode->ino, inode->num_opened_fds,
-             inode->num_allocated_fds);
+       DEBUG("Allocating fd for stream ID %u from inode %#"PRIx64" "
+             "(open = %u, allocated = %u)",
+             stream_id, inode->i_ino, inode->i_num_opened_fds,
+             inode->i_num_allocated_fds);
 
-       if (inode->num_opened_fds == inode->num_allocated_fds) {
-               struct wimlib_fd **fds;
+       if (inode->i_num_opened_fds == inode->i_num_allocated_fds) {
+               struct wimfs_fd **fds;
                u16 num_new_fds;
 
-               if (inode->num_allocated_fds == max_fds) {
+               if (inode->i_num_allocated_fds == max_fds) {
                        ret = -EMFILE;
                        goto out;
                }
                num_new_fds = min(fds_per_alloc,
-                                 max_fds - inode->num_allocated_fds);
+                                 max_fds - inode->i_num_allocated_fds);
 
-               fds = REALLOC(inode->fds,
-                             (inode->num_allocated_fds + num_new_fds) *
-                               sizeof(inode->fds[0]));
+               fds = REALLOC(inode->i_fds,
+                             (inode->i_num_allocated_fds + num_new_fds) *
+                               sizeof(inode->i_fds[0]));
                if (!fds) {
                        ret = -ENOMEM;
                        goto out;
                }
-               memset(&fds[inode->num_allocated_fds], 0,
+               memset(&fds[inode->i_num_allocated_fds], 0,
                       num_new_fds * sizeof(fds[0]));
-               inode->fds = fds;
-               inode->num_allocated_fds += num_new_fds;
+               inode->i_fds = fds;
+               inode->i_num_allocated_fds += num_new_fds;
        }
        for (u16 i = 0; ; i++) {
-               if (!inode->fds[i]) {
-                       struct wimlib_fd *fd = CALLOC(1, sizeof(*fd));
+               if (!inode->i_fds[i]) {
+                       struct wimfs_fd *fd = CALLOC(1, sizeof(*fd));
                        if (!fd) {
                                ret = -ENOMEM;
                                break;
@@ -199,8 +206,8 @@ static int alloc_wimlib_fd(struct inode *inode,
                        fd->idx        = i;
                        fd->stream_id  = stream_id;
                        *fd_ret        = fd;
-                       inode->fds[i]  = fd;
-                       inode->num_opened_fds++;
+                       inode->i_fds[i]  = fd;
+                       inode->i_num_opened_fds++;
                        if (lte && !readonly)
                                lte->num_opened_fds++;
                        DEBUG("Allocated fd (idx = %u)", fd->idx);
@@ -213,20 +220,20 @@ out:
        return ret;
 }
 
-static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
+static void inode_put_fd(struct wim_inode *inode, struct wimfs_fd *fd)
 {
        wimlib_assert(inode != NULL);
 
        pthread_mutex_lock(&inode->i_mutex);
 
        wimlib_assert(fd->f_inode == inode);
-       wimlib_assert(inode->num_opened_fds != 0);
-       wimlib_assert(fd->idx < inode->num_allocated_fds);
-       wimlib_assert(inode->fds[fd->idx] == fd);
+       wimlib_assert(inode->i_num_opened_fds != 0);
+       wimlib_assert(fd->idx < inode->i_num_allocated_fds);
+       wimlib_assert(inode->i_fds[fd->idx] == fd);
 
-       inode->fds[fd->idx] = NULL;
+       inode->i_fds[fd->idx] = NULL;
        FREE(fd);
-       if (--inode->num_opened_fds == 0 && inode->link_count == 0) {
+       if (--inode->i_num_opened_fds == 0 && inode->i_nlink == 0) {
                pthread_mutex_unlock(&inode->i_mutex);
                free_inode(inode);
        } else {
@@ -234,7 +241,7 @@ 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)
+static int lte_put_fd(struct wim_lookup_table_entry *lte, struct wimfs_fd *fd)
 {
        wimlib_assert(fd->f_lte == lte);
 
@@ -256,12 +263,12 @@ static int lte_put_fd(struct lookup_table_entry *lte, struct wimlib_fd *fd)
 }
 
 /* Close a file descriptor. */
-static int close_wimlib_fd(struct wimlib_fd *fd)
+static int close_wimfs_fd(struct wimfs_fd *fd)
 {
        int ret;
-       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);
+       DEBUG("Closing fd (ino = %#"PRIx64", opened = %u, allocated = %u)",
+             fd->f_inode->i_ino, fd->f_inode->i_num_opened_fds,
+             fd->f_inode->i_num_allocated_fds);
        ret = lte_put_fd(fd->f_lte, fd);
        if (ret != 0)
                return ret;
@@ -270,16 +277,25 @@ static int close_wimlib_fd(struct wimlib_fd *fd)
        return 0;
 }
 
+/*
+ * Add a new dentry with a new inode to a WIM image.
+ *
+ * @ctx:        Context for the mounted WIM image.
+ * @path:       Path to create the dentry at.
+ * @dentry_ret:  Return the pointer to the dentry if successful.
+ *
+ * Returns 0 on success, or negative error number on failure.
+ */
 static int create_dentry(struct wimfs_context *ctx, const char *path,
-                        struct dentry **dentry_ret)
+                        struct wim_dentry **dentry_ret)
 {
-       struct dentry *parent;
-       struct dentry *new;
+       struct wim_dentry *parent;
+       struct wim_dentry *new;
        const char *basename;
 
        parent = get_parent_dentry(ctx->wim, path);
        if (!parent)
-               return -ENOENT;
+               return -errno;
 
        if (!dentry_is_directory(parent))
                return -ENOTDIR;
@@ -290,17 +306,18 @@ static int create_dentry(struct wimfs_context *ctx, const char *path,
 
        new = new_dentry_with_inode(basename);
        if (!new)
-               return -ENOMEM;
+               return -errno;
 
-       new->d_inode->resolved = 1;
-       new->d_inode->ino = ctx->next_ino++;
+       new->d_inode->i_resolved = 1;
+       new->d_inode->i_ino = ctx->next_ino++;
        dentry_add_child(parent, new);
-       hlist_add_head(&new->d_inode->hlist, ctx->image_inode_list);
+       hlist_add_head(&new->d_inode->i_hlist, ctx->image_inode_list);
        *dentry_ret = new;
        return 0;
 }
 
-/* Remove a dentry; i.e. remove a reference to the corresponding inode.
+/* Remove a dentry from a mounted WIM image; i.e. remove an alias for the
+ * corresponding inode.
  *
  * If there are no remaining references to the inode either through dentries or
  * open file descriptors, the inode is freed.  Otherwise, the inode is not
@@ -308,17 +325,17 @@ static int create_dentry(struct wimfs_context *ctx, const char *path,
  *
  * Either way, all lookup table entries referenced by the inode have their
  * reference count decremented.  If a lookup table entry has no open file
- * descriptors and no references remaining, it is freed, and the staging file is
- * unlinked.
+ * descriptors and no references remaining, it is freed, and the corresponding
+ * staging file is unlinked.
  */
-static void remove_dentry(struct dentry *dentry,
-                         struct lookup_table *lookup_table)
+static void remove_dentry(struct wim_dentry *dentry,
+                         struct wim_lookup_table *lookup_table)
 {
-       struct inode *inode = dentry->d_inode;
-       struct lookup_table_entry *lte;
+       struct wim_inode *inode = dentry->d_inode;
+       struct wim_lookup_table_entry *lte;
        unsigned i;
 
-       for (i = 0; i <= inode->num_ads; i++) {
+       for (i = 0; i <= inode->i_num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte_decrement_refcnt(lte, lookup_table);
@@ -327,13 +344,14 @@ static void remove_dentry(struct dentry *dentry,
        put_dentry(dentry);
 }
 
-/* Transfers file attributes from a struct inode to a `stat' buffer.
+/* Transfers file attributes from a struct wim_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)
+static int inode_to_stbuf(const struct wim_inode *inode,
+                         const struct wim_lookup_table_entry *lte,
+                         struct stat *stbuf)
 {
        if (inode_is_symlink(inode))
                stbuf->st_mode = S_IFLNK | 0777;
@@ -342,8 +360,8 @@ static int inode_to_stbuf(const struct inode *inode,
        else
                stbuf->st_mode = S_IFREG | 0755;
 
-       stbuf->st_ino   = (ino_t)inode->ino;
-       stbuf->st_nlink = inode->link_count;
+       stbuf->st_ino   = (ino_t)inode->i_ino;
+       stbuf->st_nlink = inode->i_nlink;
        stbuf->st_uid   = getuid();
        stbuf->st_gid   = getgid();
 
@@ -363,9 +381,9 @@ static int inode_to_stbuf(const struct inode *inode,
                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_atime   = wim_timestamp_to_unix(inode->i_last_access_time);
+       stbuf->st_mtime   = wim_timestamp_to_unix(inode->i_last_write_time);
+       stbuf->st_ctime   = wim_timestamp_to_unix(inode->i_creation_time);
        stbuf->st_blocks  = (stbuf->st_size + 511) / 512;
        return 0;
 }
@@ -374,12 +392,15 @@ static int inode_to_stbuf(const struct inode *inode,
  * writing.
  *
  * @name_ret: A location into which the a pointer to the newly allocated name of
- *                     the staging file is stored.
- * @return:  The file descriptor for the new file.  Returns -1 and sets errno on
- *             error, for any reason possible from the creat() function.
+ *           the staging file is stored.
+ *
+ * @ctx:      Context for the WIM filesystem; this provides the name of the
+ *           staging directory.
+ *
+ * On success, returns the file descriptor for the staging file, opened for
+ * writing.  On failure, returns -1 and sets errno.
  */
-static int create_staging_file(char **name_ret, int open_flags,
-                              struct wimfs_context *ctx)
+static int create_staging_file(char **name_ret, struct wimfs_context *ctx)
 {
        size_t name_len;
        char *name;
@@ -387,7 +408,9 @@ static int create_staging_file(char **name_ret, int open_flags,
        int fd;
        int errno_save;
 
-       name_len = ctx->staging_dir_name_len + 1 + SHA1_HASH_SIZE;
+       static const size_t STAGING_FILE_NAME_LEN = 20;
+
+       name_len = ctx->staging_dir_name_len + 1 + STAGING_FILE_NAME_LEN;
        name = MALLOC(name_len + 1);
        if (!name) {
                errno = ENOMEM;
@@ -399,7 +422,7 @@ static int create_staging_file(char **name_ret, int open_flags,
                memcpy(name, ctx->staging_dir_name, ctx->staging_dir_name_len);
                name[ctx->staging_dir_name_len] = '/';
                randomize_char_array_with_alnum(name + ctx->staging_dir_name_len + 1,
-                                               SHA1_HASH_SIZE);
+                                               STAGING_FILE_NAME_LEN);
                name[name_len] = '\0';
 
 
@@ -407,15 +430,14 @@ static int create_staging_file(char **name_ret, int open_flags,
         * existing file, and try again if so  */
        } while (stat(name, &stbuf) == 0);
 
-       if (errno != ENOENT)
-               /* other error! */
+       if (errno != ENOENT) /* other error?! */
                return -1;
 
        /* doesn't exist--- ok */
 
        DEBUG("Creating staging file `%s'", name);
 
-       fd = open(name, open_flags | O_CREAT | O_TRUNC, 0600);
+       fd = open(name, O_WRONLY | O_CREAT | O_EXCL, 0600);
        if (fd == -1) {
                errno_save = errno;
                FREE(name);
@@ -445,20 +467,20 @@ static int create_staging_file(char **name_ret, int open_flags,
  *
  * Returns 0 on success or a negative error code on failure.
  */
-static int extract_resource_to_staging_dir(struct inode *inode,
+static int extract_resource_to_staging_dir(struct wim_inode *inode,
                                           u32 stream_id,
-                                          struct lookup_table_entry **lte,
+                                          struct wim_lookup_table_entry **lte,
                                           off_t size,
                                           struct wimfs_context *ctx)
 {
        char *staging_file_name;
        int ret;
        int fd;
-       struct lookup_table_entry *old_lte, *new_lte;
+       struct wim_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);
+             "stream id %"PRIu32, inode->i_ino, stream_id);
 
        old_lte = *lte;
 
@@ -466,7 +488,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                      old_lte->resource_location != RESOURCE_IN_STAGING_FILE);
 
        /* Create the staging file */
-       fd = create_staging_file(&staging_file_name, O_WRONLY, ctx);
+       fd = create_staging_file(&staging_file_name, ctx);
        if (fd == -1)
                return -errno;
 
@@ -499,7 +521,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
        /* 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) {
+       if (old_lte && inode->i_nlink == 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
@@ -515,10 +537,10 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                         * 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);
+                       wimlib_assert(old_lte->refcnt > inode->i_nlink);
                        DEBUG("Splitting lookup table entry "
-                             "(inode->link_count = %u, old_lte->refcnt = %u)",
-                             inode->link_count, old_lte->refcnt);
+                             "(inode->i_nlink = %u, old_lte->refcnt = %u)",
+                             inode->i_nlink, old_lte->refcnt);
                }
 
                new_lte = new_lookup_table_entry();
@@ -538,8 +560,8 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                 * file descriptors to the new lookup table entry.  If there's
                 * an old lookup table entry, this number needs to be subtracted
                 * from the fd's opened to the old entry. */
-               for (u16 i = 0, j = 0; j < inode->num_opened_fds; i++) {
-                       struct wimlib_fd *fd = inode->fds[i];
+               for (u16 i = 0, j = 0; j < inode->i_num_opened_fds; i++) {
+                       struct wimfs_fd *fd = inode->i_fds[i];
                        if (fd) {
                                if (fd->stream_id == stream_id) {
                                        wimlib_assert(fd->f_lte == old_lte);
@@ -559,22 +581,22 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                      new_lte->num_opened_fds);
                if (old_lte) {
                        old_lte->num_opened_fds -= new_lte->num_opened_fds;
-                       old_lte->refcnt -= inode->link_count;
+                       old_lte->refcnt -= inode->i_nlink;
                }
        }
 
-       new_lte->refcnt                       = inode->link_count;
+       new_lte->refcnt                       = inode->i_nlink;
        new_lte->resource_location            = RESOURCE_IN_STAGING_FILE;
        new_lte->staging_file_name            = staging_file_name;
        new_lte->lte_inode                    = inode;
        random_hash(new_lte->hash);
 
        if (stream_id == 0)
-               inode->lte = new_lte;
+               inode->i_lte = new_lte;
        else
-               for (u16 i = 0; i < inode->num_ads; i++)
-                       if (inode->ads_entries[i].stream_id == stream_id)
-                               inode->ads_entries[i].lte = new_lte;
+               for (u16 i = 0; i < inode->i_num_ads; i++)
+                       if (inode->i_ads_entries[i].stream_id == stream_id)
+                               inode->i_ads_entries[i].lte = new_lte;
 
        lookup_table_insert(ctx->wim->lookup_table, new_lte);
        list_add(&new_lte->staging_list, &ctx->staging_list);
@@ -582,7 +604,7 @@ static int extract_resource_to_staging_dir(struct inode *inode,
        return 0;
 out_revert_fd_changes:
        for (u16 i = 0, j = 0; j < new_lte->num_opened_fds; i++) {
-               struct wimlib_fd *fd = inode->fds[i];
+               struct wimfs_fd *fd = inode->i_fds[i];
                if (fd && fd->stream_id == stream_id && fd->f_lte == new_lte) {
                        fd->f_lte = old_lte;
                        if (fd->staging_fd != -1) {
@@ -692,26 +714,26 @@ 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)
+static void inode_update_lte_ptr(struct wim_inode *inode,
+                                struct wim_lookup_table_entry *old_lte,
+                                struct wim_lookup_table_entry *new_lte)
 {
-       if (inode->lte == old_lte) {
-               inode->lte = new_lte;
+       if (inode->i_lte == old_lte) {
+               inode->i_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;
+               for (unsigned i = 0; i < inode->i_num_ads; i++) {
+                       if (inode->i_ads_entries[i].lte == old_lte) {
+                               inode->i_ads_entries[i].lte = new_lte;
                                break;
                        }
                }
        }
 }
 
-static int update_lte_of_staging_file(struct lookup_table_entry *lte,
-                                     struct lookup_table *table)
+static int update_lte_of_staging_file(struct wim_lookup_table_entry *lte,
+                                     struct wim_lookup_table *table)
 {
-       struct lookup_table_entry *duplicate_lte;
+       struct wim_lookup_table_entry *duplicate_lte;
        int ret;
        u8 hash[SHA1_HASH_SIZE];
        struct stat stbuf;
@@ -748,14 +770,14 @@ static int update_lte_of_staging_file(struct lookup_table_entry *lte,
        return 0;
 }
 
-static int inode_close_fds(struct inode *inode)
+static int inode_close_fds(struct wim_inode *inode)
 {
-       u16 num_opened_fds = inode->num_opened_fds;
+       u16 num_opened_fds = inode->i_num_opened_fds;
        for (u16 i = 0, j = 0; j < num_opened_fds; i++) {
-               struct wimlib_fd *fd = inode->fds[i];
+               struct wimfs_fd *fd = inode->i_fds[i];
                if (fd) {
                        wimlib_assert(fd->f_inode == inode);
-                       int ret = close_wimlib_fd(fd);
+                       int ret = close_wimfs_fd(fd);
                        if (ret != 0)
                                return ret;
                        j++;
@@ -765,10 +787,11 @@ static int inode_close_fds(struct inode *inode)
 }
 
 /* Overwrites the WIM file, with changes saved. */
-static int rebuild_wim(struct wimfs_context *ctx, int write_flags)
+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;
+       struct wim_lookup_table_entry *lte, *tmp;
        WIMStruct *w = ctx->wim;
 
        DEBUG("Closing all staging file descriptors.");
@@ -786,7 +809,7 @@ static int rebuild_wim(struct wimfs_context *ctx, int write_flags)
        }
 
        xml_update_image_info(w, w->current_image);
-       ret = wimlib_overwrite(w, write_flags, 0, NULL);
+       ret = wimlib_overwrite(w, write_flags, 0, progress_func);
        if (ret != 0)
                ERROR("Failed to commit changes to mounted WIM image");
        return ret;
@@ -987,6 +1010,7 @@ struct unmount_msg_hdr {
 struct msg_unmount_request {
        struct unmount_msg_hdr hdr;
        u32 unmount_flags;
+       u8 want_progress_messages;
 } PACKED;
 
 struct msg_daemon_info {
@@ -1000,41 +1024,49 @@ struct msg_unmount_finished {
        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;
+struct msg_handler_context_hdr {
        int timeout_seconds;
-       union {
-               struct {
-                       /*bool unmount_complete;*/
-                       pid_t daemon_pid;
-                       int mount_flags;
-                       int status;
-               } unmount;
-               struct {
-                       /*int unmount_flags;*/
-                       struct wimfs_context *wimfs_ctx;
-               } daemon;
-       };
 };
 
-static int send_unmount_request_msg(mqd_t mq, int unmount_flags)
+struct unmount_msg_handler_context {
+       struct msg_handler_context_hdr hdr;
+       pid_t daemon_pid;
+       int mount_flags;
+       int status;
+       wimlib_progress_func_t progress_func;
+};
+
+struct daemon_msg_handler_context {
+       struct msg_handler_context_hdr hdr;
+       struct wimfs_context *wimfs_ctx;
+};
+
+static int send_unmount_request_msg(mqd_t mq, int unmount_flags,
+                                   u8 want_progress_messages)
 {
        DEBUG("Sending unmount request msg");
        struct msg_unmount_request msg = {
                .hdr = {
-                       .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
                        .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)) {
@@ -1051,7 +1083,7 @@ static int send_daemon_info_msg(mqd_t mq, pid_t pid, int mount_flags)
 
        struct msg_daemon_info msg = {
                .hdr = {
-                       .min_version = WIMLIB_MAKEVERSION(1, 2, 0),
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
                        .cur_version = WIMLIB_VERSION_CODE,
                        .msg_type = MSG_TYPE_DAEMON_INFO,
                        .msg_size = sizeof(msg),
@@ -1071,7 +1103,7 @@ 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),
+                       .min_version = WIMLIB_MAKEVERSION(1, 2, 1),
                        .cur_version = WIMLIB_VERSION_CODE,
                        .msg_type = MSG_TYPE_UNMOUNT_FINISHED,
                        .msg_size = sizeof(msg),
@@ -1082,26 +1114,52 @@ static void send_unmount_finished_msg(mqd_t mq, int status)
                ERROR_WITH_ERRNO("Failed to send status to unmount process");
 }
 
-static int msg_unmount_request_handler(const void *_msg,
-                                      struct msg_handler_context *handler_ctx)
+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, 1),
+                               .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 progress information "
+                                        "to unmount process");
+               }
+       }
+       return 0;
+}
+
+static int msg_unmount_request_handler(const void *_msg, void *_handler_ctx)
 {
-       const struct msg_unmount_request *msg;
+       const struct msg_unmount_request *msg = _msg;
+       struct daemon_msg_handler_context *handler_ctx = _handler_ctx;
        struct wimfs_context *wimfs_ctx;
        int status = 0;
        int ret;
        int unmount_flags;
+       wimlib_progress_func_t progress_func;
 
        DEBUG("Handling unmount request msg");
 
-       msg = _msg;
-       wimfs_ctx = handler_ctx->daemon.wimfs_ctx;
-
+       wimfs_ctx = handler_ctx->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
+               progress_func = NULL;
 
        ret = send_daemon_info_msg(wimfs_ctx->daemon_to_unmount_mq, getpid(),
                                   wimfs_ctx->mount_flags);
@@ -1119,7 +1177,8 @@ static int msg_unmount_request_handler(const void *_msg,
                                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);
+                       status = rebuild_wim(wimfs_ctx, write_flags,
+                                            progress_func);
                }
        } else {
                DEBUG("Read-only mount");
@@ -1139,51 +1198,65 @@ out:
        return MSG_BREAK_LOOP;
 }
 
-static int msg_daemon_info_handler(const void *_msg,
-                                  struct msg_handler_context *handler_ctx)
+static int msg_daemon_info_handler(const void *_msg, void *_handler_ctx)
 {
        const struct msg_daemon_info *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
+
        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;
+       handler_ctx->daemon_pid = msg->daemon_pid;
+       handler_ctx->mount_flags = msg->mount_flags;
+       handler_ctx->hdr.timeout_seconds = 1;
        DEBUG("pid of daemon is %d; mount flags were %#x",
-             handler_ctx->unmount.daemon_pid,
-             handler_ctx->unmount.mount_flags);
+             handler_ctx->daemon_pid,
+             handler_ctx->mount_flags);
        return 0;
 }
 
-static int msg_unmount_finished_handler(const void *_msg,
-                                       struct msg_handler_context *handler_ctx)
+static int msg_write_streams_progress_handler(const void *_msg,
+                                             void *_handler_ctx)
+{
+       const struct msg_write_streams_progress *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
+
+       if (msg->hdr.msg_size < sizeof(*msg))
+               return WIMLIB_ERR_INVALID_UNMOUNT_MESSAGE;
+       if (handler_ctx->progress_func) {
+               handler_ctx->progress_func(WIMLIB_PROGRESS_MSG_WRITE_STREAMS,
+                                          &msg->info);
+       }
+       return 0;
+}
+
+static int msg_unmount_finished_handler(const void *_msg, void *_handler_ctx)
 {
        const struct msg_unmount_finished *msg = _msg;
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
+
        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);
+       handler_ctx->status = msg->status;
+       DEBUG("status is %d", handler_ctx->status);
        return MSG_BREAK_LOOP;
 }
 
-static int unmount_timed_out_cb(struct msg_handler_context *handler_ctx)
+static int unmount_timed_out_cb(void *_handler_ctx)
 {
-       if (handler_ctx->unmount.daemon_pid == 0) {
+       struct unmount_msg_handler_context *handler_ctx = _handler_ctx;
+
+       if (handler_ctx->daemon_pid == 0) {
                goto out_crashed;
        } else {
-               errno = 0;
-               kill(handler_ctx->unmount.daemon_pid, 0);
+               kill(handler_ctx->daemon_pid, 0);
                if (errno == ESRCH) {
                        goto out_crashed;
-               } else if (errno != 0) {
-                       ERROR_WITH_ERRNO("Error determining state of "
-                                        "filesystem daemon");
-                       return WIMLIB_ERR_MQUEUE;
                } else {
                        DEBUG("Filesystem daemon is still alive... "
-                             "Waiting another %d seconds",
-                             handler_ctx->timeout_seconds);
+                             "Waiting another %d seconds\n",
+                             handler_ctx->hdr.timeout_seconds);
                        return 0;
                }
        }
@@ -1193,17 +1266,17 @@ out_crashed:
        return WIMLIB_ERR_FILESYSTEM_DAEMON_CRASHED;
 }
 
-static int daemon_timed_out_cb(struct msg_handler_context *handler_ctx)
+static int daemon_timed_out_cb(void *_handler_ctx)
 {
-       DEBUG("Timed out");
+       ERROR("Timed out waiting for unmount request! "
+             "Changes to the mounted WIM will not be committed.");
        return WIMLIB_ERR_TIMEOUT;
 }
 
-typedef int (*msg_handler_t)(const void *_msg,
-                            struct msg_handler_context *handler_ctx);
+typedef int (*msg_handler_t)(const void *_msg, void *_handler_ctx);
 
 struct msg_handler_callbacks {
-       int (*timed_out)(struct msg_handler_context *);
+       int (*timed_out)(void * _handler_ctx);
        msg_handler_t msg_handlers[MSG_TYPE_MAX];
 };
 
@@ -1211,6 +1284,7 @@ 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,
        },
 };
@@ -1222,7 +1296,8 @@ static const struct msg_handler_callbacks daemon_msg_handler_callbacks = {
        },
 };
 
-static int receive_message(mqd_t mq, struct msg_handler_context *handler_ctx,
+static int receive_message(mqd_t mq,
+                          struct msg_handler_context_hdr *handler_ctx,
                           const msg_handler_t msg_handlers[],
                           long mailbox_size, void *mailbox)
 {
@@ -1240,11 +1315,12 @@ static int receive_message(mqd_t mq, struct msg_handler_context *handler_ctx,
                                         mailbox_size, NULL, &timeout);
        hdr = mailbox;
        if (bytes_received == -1) {
-               ERROR_WITH_ERRNO("mq_timedreceive()");
-               if (errno == ETIMEDOUT)
+               if (errno == ETIMEDOUT) {
                        ret = WIMLIB_ERR_TIMEOUT;
-               else
+               } 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;
@@ -1267,7 +1343,7 @@ static int receive_message(mqd_t mq, struct msg_handler_context *handler_ctx,
 
 static int message_loop(mqd_t mq,
                        const struct msg_handler_callbacks *callbacks,
-                       struct msg_handler_context *handler_ctx)
+                       struct msg_handler_context_hdr *handler_ctx)
 {
        static const size_t MAX_MSG_SIZE = 512;
        long msgsize;
@@ -1334,46 +1410,53 @@ static int execute_fusermount(const char *dir)
        }
 
        /* Parent */
-       ret = wait(&status);
+       ret = waitpid(pid, &status, 0);
        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);
+       if (!WIFEXITED(status)) {
+               ERROR("'fusermount' did not terminate normally!");
+               return WIMLIB_ERR_FUSERMOUNT;
+       }
 
-               /* 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. */
+       status = WEXITSTATUS(status);
 
-               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);
-               }
+       if (status == 0)
+               return 0;
 
-               /* Parent */
-               ret = wait(&status);
-               if (ret == -1) {
-                       ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
-                                        "terminate");
-                       return WIMLIB_ERR_FUSERMOUNT;
-               }
-               if (status != 0) {
-                       ERROR("`umount' exited with failure status");
-                       return WIMLIB_ERR_FUSERMOUNT;
-               }
+       if (status != WIMLIB_ERR_FUSERMOUNT)
+               return WIMLIB_ERR_FUSERMOUNT;
+
+       /* 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. */
+       ERROR("Falling back to 'umount'.  Note: you may need to be "
+             "root for this to work");
+       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 = waitpid(pid, &status, 0);
+       if (ret == -1) {
+               ERROR_WITH_ERRNO("Failed to wait for `umount' process to "
+                                "terminate");
+               return WIMLIB_ERR_FUSERMOUNT;
+       }
+       if (status != 0) {
+               ERROR("`umount' did not successfully complete");
+               return WIMLIB_ERR_FUSERMOUNT;
        }
        return 0;
 }
@@ -1386,9 +1469,9 @@ static int wimfs_access(const char *path, int mask)
 
 static int wimfs_chmod(const char *path, mode_t mask)
 {
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        struct wimfs_context *ctx = wimfs_get_context();
-       struct inode *inode;
+       struct wim_inode *inode;
        struct stat stbuf;
        int ret;
 
@@ -1408,33 +1491,26 @@ static int wimfs_chmod(const char *path, mode_t mask)
 /* Called when the filesystem is unmounted. */
 static void wimfs_destroy(void *p)
 {
-       struct wimfs_context *wimfs_ctx;
-
-       wimfs_ctx = wimfs_get_context();
-
-       if (open_message_queues(wimfs_ctx, true))
-               return;
-
-       struct msg_handler_context handler_ctx = {
-               .is_daemon = true,
-               .timeout_seconds = 5,
-               .daemon = {
+       struct wimfs_context *wimfs_ctx = wimfs_get_context();
+       if (open_message_queues(wimfs_ctx, true) == 0) {
+               struct daemon_msg_handler_context handler_ctx = {
+                       .hdr = {
+                               .timeout_seconds = 5,
+                       },
                        .wimfs_ctx = wimfs_ctx,
-               },
-       };
-
-       message_loop(wimfs_ctx->unmount_to_daemon_mq,
-                    &daemon_msg_handler_callbacks,
-                    &handler_ctx);
-
-       close_message_queues(wimfs_ctx);
+               };
+               message_loop(wimfs_ctx->unmount_to_daemon_mq,
+                            &daemon_msg_handler_callbacks,
+                            &handler_ctx.hdr);
+               close_message_queues(wimfs_ctx);
+       }
 }
 
 #if 0
 static int wimfs_fallocate(const char *path, int mode,
                           off_t offset, off_t len, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        wimlib_assert(fd->staging_fd != -1);
        return fallocate(fd->staging_fd, mode, offset, len);
 }
@@ -1444,14 +1520,14 @@ static int wimfs_fallocate(const char *path, int mode,
 static int wimfs_fgetattr(const char *path, struct stat *stbuf,
                          struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        return inode_to_stbuf(fd->f_inode, fd->f_lte, stbuf);
 }
 
 static int wimfs_ftruncate(const char *path, off_t size,
                           struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        int ret = ftruncate(fd->staging_fd, size);
        if (ret != 0)
                return -errno;
@@ -1465,8 +1541,8 @@ static int wimfs_ftruncate(const char *path, off_t size,
  */
 static int wimfs_getattr(const char *path, struct stat *stbuf)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        struct wimfs_context *ctx = wimfs_get_context();
 
@@ -1484,10 +1560,10 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
                          size_t size)
 {
        int ret;
-       struct inode *inode;
-       struct ads_entry *ads_entry;
+       struct wim_inode *inode;
+       struct wim_ads_entry *ads_entry;
        size_t res_size;
-       struct lookup_table_entry *lte;
+       struct wim_lookup_table_entry *lte;
        struct wimfs_context *ctx = wimfs_get_context();
 
        if (!(ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR))
@@ -1499,7 +1575,7 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        ads_entry = inode_get_ads_entry(inode, name, NULL);
        if (!ads_entry)
@@ -1526,21 +1602,21 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
 /* Create a hard link */
 static int wimfs_link(const char *to, const char *from)
 {
-       struct dentry *from_dentry, *from_dentry_parent;
+       struct wim_dentry *from_dentry, *from_dentry_parent;
        const char *link_name;
-       struct inode *inode;
-       struct lookup_table_entry *lte;
+       struct wim_inode *inode;
+       struct wim_lookup_table_entry *lte;
        WIMStruct *w = wimfs_get_WIMStruct();
        u16 i;
 
        inode = wim_pathname_to_inode(w, to);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
-       if (inode->attributes & FILE_ATTRIBUTE_REPARSE_POINT)
+       if (inode->i_attributes & FILE_ATTRIBUTE_REPARSE_POINT)
                return -EEXIST;
 
-       if (inode->attributes & FILE_ATTRIBUTE_DIRECTORY)
+       if (inode->i_attributes & FILE_ATTRIBUTE_DIRECTORY)
                return -EPERM;
 
        from_dentry_parent = get_parent_dentry(w, from);
@@ -1554,13 +1630,13 @@ static int wimfs_link(const char *to, const char *from)
                return -EEXIST;
        from_dentry = new_dentry(link_name);
        if (!from_dentry)
-               return -ENOMEM;
+               return -errno;
 
        inode_add_dentry(from_dentry, inode);
        from_dentry->d_inode = inode;
-       inode->link_count++;
+       inode->i_nlink++;
 
-       for (i = 0; i <= inode->num_ads; i++) {
+       for (i = 0; i <= inode->i_num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte->refcnt++;
@@ -1573,7 +1649,7 @@ static int wimfs_link(const char *to, const char *from)
 static int wimfs_listxattr(const char *path, char *list, size_t size)
 {
        size_t needed_size;
-       struct inode *inode;
+       struct wim_inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
        u16 i;
        char *p;
@@ -1585,21 +1661,21 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        if (size == 0) {
                needed_size = 0;
-               for (i = 0; i < inode->num_ads; i++)
-                       needed_size += inode->ads_entries[i].stream_name_utf8_len + 6;
+               for (i = 0; i < inode->i_num_ads; i++)
+                       needed_size += inode->i_ads_entries[i].stream_name_utf8_len + 6;
                return needed_size;
        } else {
                p = list;
-               for (i = 0; i < inode->num_ads; i++) {
-                       needed_size = inode->ads_entries[i].stream_name_utf8_len + 6;
+               for (i = 0; i < inode->i_num_ads; i++) {
+                       needed_size = inode->i_ads_entries[i].stream_name_utf8_len + 6;
                        if (needed_size > size)
                                return -ERANGE;
                        p += sprintf(p, "user.%s",
-                                    inode->ads_entries[i].stream_name_utf8) + 1;
+                                    inode->i_ads_entries[i].stream_name_utf8) + 1;
                        size -= needed_size;
                }
                return p - list;
@@ -1612,12 +1688,12 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
  * @mode is currently ignored.  */
 static int wimfs_mkdir(const char *path, mode_t mode)
 {
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        int ret;
 
        ret = create_dentry(wimfs_get_context(), path, &dentry);
        if (ret == 0)
-               dentry->d_inode->attributes = FILE_ATTRIBUTE_DIRECTORY;
+               dentry->d_inode->i_attributes = FILE_ATTRIBUTE_DIRECTORY;
        return ret;
 }
 
@@ -1630,8 +1706,8 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
        if ((ctx->mount_flags & WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)
             && (stream_name = path_stream_name(path))) {
                /* Make an alternate data stream */
-               struct ads_entry *new_entry;
-               struct inode *inode;
+               struct wim_ads_entry *new_entry;
+               struct wim_inode *inode;
 
                char *p = (char*)stream_name - 1;
                wimlib_assert(*p == ':');
@@ -1639,8 +1715,8 @@ 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->attributes &
+                       return -errno;
+               if (inode->i_attributes &
                    (FILE_ATTRIBUTE_REPARSE_POINT | FILE_ATTRIBUTE_DIRECTORY))
                        return -ENOENT;
                if (inode_get_ads_entry(inode, stream_name, NULL))
@@ -1650,13 +1726,13 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
                        return -ENOMEM;
                return 0;
        } else {
-               struct dentry *dentry;
+               struct wim_dentry *dentry;
                int ret;
 
                /* Make a normal file (not an alternate data stream) */
                ret = create_dentry(ctx, path, &dentry);
                if (ret == 0)
-                       dentry->d_inode->attributes = FILE_ATTRIBUTE_NORMAL;
+                       dentry->d_inode->i_attributes = FILE_ATTRIBUTE_NORMAL;
                return ret;
        }
 }
@@ -1665,11 +1741,11 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 /* Open a file.  */
 static int wimfs_open(const char *path, struct fuse_file_info *fi)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
-       struct wimlib_fd *fd;
-       struct inode *inode;
+       struct wimfs_fd *fd;
+       struct wim_inode *inode;
        u16 stream_idx;
        u32 stream_id;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -1684,7 +1760,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
        if (stream_idx == 0)
                stream_id = 0;
        else
-               stream_id = inode->ads_entries[stream_idx - 1].stream_id;
+               stream_id = inode->i_ads_entries[stream_idx - 1].stream_id;
 
        /* The file resource may be in the staging directory (read-write mounts
         * only) or in the WIM.  If it's in the staging directory, we need to
@@ -1702,7 +1778,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
                        return ret;
        }
 
-       ret = alloc_wimlib_fd(inode, stream_id, lte, &fd,
+       ret = alloc_wimfs_fd(inode, stream_id, lte, &fd,
                              wimfs_ctx_readonly(ctx));
        if (ret != 0)
                return ret;
@@ -1711,7 +1787,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
                fd->staging_fd = open(lte->staging_file_name, fi->flags);
                if (fd->staging_fd == -1) {
                        int errno_save = errno;
-                       close_wimlib_fd(fd);
+                       close_wimfs_fd(fd);
                        return -errno_save;
                }
        }
@@ -1722,18 +1798,18 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
 /* Opens a directory. */
 static int wimfs_opendir(const char *path, struct fuse_file_info *fi)
 {
-       struct inode *inode;
+       struct wim_inode *inode;
        int ret;
-       struct wimlib_fd *fd = NULL;
+       struct wimfs_fd *fd = NULL;
        struct wimfs_context *ctx = wimfs_get_context();
        WIMStruct *w = ctx->wim;
 
        inode = wim_pathname_to_inode(w, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
        if (!inode_is_directory(inode))
                return -ENOTDIR;
-       ret = alloc_wimlib_fd(inode, 0, NULL, &fd, wimfs_ctx_readonly(ctx));
+       ret = alloc_wimfs_fd(inode, 0, NULL, &fd, wimfs_ctx_readonly(ctx));
        fi->fh = (uintptr_t)fd;
        return ret;
 }
@@ -1745,7 +1821,7 @@ static int wimfs_opendir(const char *path, struct fuse_file_info *fi)
 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;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        ssize_t ret;
 
        if (!fd)
@@ -1760,7 +1836,7 @@ 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);
 
-               DEBUG("Seek to offset %zu", offset);
+               DEBUG("Seek to offset %"PRIu64, offset);
 
                if (lseek(fd->staging_fd, offset, SEEK_SET) == -1)
                        return -errno;
@@ -1787,7 +1863,7 @@ struct fill_params {
        fuse_fill_dir_t filler;
 };
 
-static int dentry_fuse_fill(struct dentry *dentry, void *arg)
+static int dentry_fuse_fill(struct wim_dentry *dentry, void *arg)
 {
        struct fill_params *fill_params = arg;
        return fill_params->filler(fill_params->buf, dentry->file_name_utf8,
@@ -1799,8 +1875,8 @@ static int dentry_fuse_fill(struct dentry *dentry, void *arg)
 static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
                         off_t offset, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       struct inode *inode;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       struct wim_inode *inode;
 
        if (!fd)
                return -EBADF;
@@ -1815,7 +1891,7 @@ static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
        filler(buf, ".", NULL, 0);
        filler(buf, "..", NULL, 0);
 
-       return for_dentry_in_rbtree(inode->children.rb_node,
+       return for_dentry_in_rbtree(inode->i_children.rb_node,
                                    dentry_fuse_fill, &fill_params);
 }
 
@@ -1823,10 +1899,10 @@ static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
 static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
 {
        struct wimfs_context *ctx = wimfs_get_context();
-       struct inode *inode = wim_pathname_to_inode(ctx->wim, path);
+       struct wim_inode *inode = wim_pathname_to_inode(ctx->wim, path);
        int ret;
        if (!inode)
-               return -ENOENT;
+               return -errno;
        if (!inode_is_symlink(inode))
                return -EINVAL;
 
@@ -1840,23 +1916,23 @@ static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
 /* Close a file. */
 static int wimfs_release(const char *path, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       return close_wimlib_fd(fd);
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       return close_wimfs_fd(fd);
 }
 
 /* Close a directory */
 static int wimfs_releasedir(const char *path, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       return close_wimlib_fd(fd);
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
+       return close_wimfs_fd(fd);
 }
 
 #ifdef ENABLE_XATTR
 /* Remove an alternate data stream through the XATTR interface */
 static int wimfs_removexattr(const char *path, const char *name)
 {
-       struct inode *inode;
-       struct ads_entry *ads_entry;
+       struct wim_inode *inode;
+       struct wim_ads_entry *ads_entry;
        u16 ads_idx;
        struct wimfs_context *ctx = wimfs_get_context();
 
@@ -1869,7 +1945,7 @@ static int wimfs_removexattr(const char *path, const char *name)
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        ads_entry = inode_get_ads_entry(inode, name, &ads_idx);
        if (!ads_entry)
@@ -1882,11 +1958,9 @@ static int wimfs_removexattr(const char *path, const char *name)
 /* Renames a file or directory.  See rename (3) */
 static int wimfs_rename(const char *from, const char *to)
 {
-       struct dentry *src;
-       struct dentry *dst;
-       struct dentry *parent_of_dst;
-       char *file_name_utf16 = NULL, *file_name_utf8 = NULL;
-       u16 file_name_utf16_len, file_name_utf8_len;
+       struct wim_dentry *src;
+       struct wim_dentry *dst;
+       struct wim_dentry *parent_of_dst;
        WIMStruct *w = wimfs_get_WIMStruct();
        int ret;
 
@@ -1895,16 +1969,10 @@ static int wimfs_rename(const char *from, const char *to)
 
        src = get_dentry(w, from);
        if (!src)
-               return -ENOENT;
+               return -errno;
 
        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));
-       if (ret != 0)
-               return -ENOMEM;
-
        if (dst) {
                /* Destination file exists */
 
@@ -1924,24 +1992,21 @@ static int wimfs_rename(const char *from, const char *to)
                                return -ENOTEMPTY;
                }
                parent_of_dst = dst->parent;
-               remove_dentry(dst, w->lookup_table);
        } else {
                /* Destination does not exist */
                parent_of_dst = get_parent_dentry(w, to);
                if (!parent_of_dst)
-                       return -ENOENT;
+                       return -errno;
 
                if (!dentry_is_directory(parent_of_dst))
                        return -ENOTDIR;
        }
 
-       FREE(src->file_name);
-       FREE(src->file_name_utf8);
-       src->file_name          = file_name_utf16;
-       src->file_name_utf8     = file_name_utf8;
-       src->file_name_len      = file_name_utf16_len;
-       src->file_name_utf8_len = file_name_utf8_len;
-
+       ret = set_dentry_name(src, path_basename(to));
+       if (ret != 0)
+               return -ENOMEM;
+       if (dst)
+               remove_dentry(dst, w->lookup_table);
        unlink_dentry(src);
        dentry_add_child(parent_of_dst, src);
        return 0;
@@ -1950,14 +2015,17 @@ static int wimfs_rename(const char *from, const char *to)
 /* Remove a directory */
 static int wimfs_rmdir(const char *path)
 {
-       struct dentry *dentry;
+       struct wim_dentry *parent, *dentry;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
+
+       if (!dentry_is_directory(dentry))
+               return -ENOTDIR;
 
-       if (!dentry_is_empty_directory(dentry))
+       if (dentry_has_children(dentry))
                return -ENOTEMPTY;
 
        remove_dentry(dentry, w->lookup_table);
@@ -1969,11 +2037,11 @@ static int wimfs_rmdir(const char *path)
 static int wimfs_setxattr(const char *path, const char *name,
                          const char *value, size_t size, int flags)
 {
-       struct ads_entry *existing_ads_entry;
-       struct ads_entry *new_ads_entry;
-       struct lookup_table_entry *existing_lte;
-       struct lookup_table_entry *lte;
-       struct inode *inode;
+       struct wim_ads_entry *existing_ads_entry;
+       struct wim_ads_entry *new_ads_entry;
+       struct wim_lookup_table_entry *existing_lte;
+       struct wim_lookup_table_entry *lte;
+       struct wim_inode *inode;
        u8 value_hash[SHA1_HASH_SIZE];
        u16 ads_idx;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -1987,7 +2055,7 @@ static int wimfs_setxattr(const char *path, const char *name,
 
        inode = wim_pathname_to_inode(ctx->wim, path);
        if (!inode)
-               return -ENOENT;
+               return -errno;
 
        existing_ads_entry = inode_get_ads_entry(inode, name, &ads_idx);
        if (existing_ads_entry) {
@@ -2036,13 +2104,13 @@ static int wimfs_setxattr(const char *path, const char *name,
 static int wimfs_symlink(const char *to, const char *from)
 {
        struct wimfs_context *ctx = wimfs_get_context();
-       struct dentry *dentry;
+       struct wim_dentry *dentry;
        int ret;
 
        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;
+               dentry->d_inode->i_attributes = FILE_ATTRIBUTE_REPARSE_POINT;
+               dentry->d_inode->i_reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
                if (inode_set_symlink(dentry->d_inode, to,
                                      ctx->wim->lookup_table, NULL))
                {
@@ -2058,12 +2126,12 @@ static int wimfs_symlink(const char *to, const char *from)
 /* Reduce the size of a file */
 static int wimfs_truncate(const char *path, off_t size)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        u16 stream_idx;
        u32 stream_id;
-       struct inode *inode;
+       struct wim_inode *inode;
        struct wimfs_context *ctx = wimfs_get_context();
 
        ret = lookup_resource(ctx->wim, path, get_lookup_flags(ctx),
@@ -2079,7 +2147,7 @@ static int wimfs_truncate(const char *path, off_t size)
        if (stream_idx == 0)
                stream_id = 0;
        else
-               stream_id = inode->ads_entries[stream_idx - 1].stream_id;
+               stream_id = inode->i_ads_entries[stream_idx - 1].stream_id;
 
        if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
                ret = truncate(lte->staging_file_name, size);
@@ -2097,8 +2165,8 @@ static int wimfs_truncate(const char *path, off_t size)
 /* Unlink a non-directory or alternate data stream */
 static int wimfs_unlink(const char *path)
 {
-       struct dentry *dentry;
-       struct lookup_table_entry *lte;
+       struct wim_dentry *dentry;
+       struct wim_lookup_table_entry *lte;
        int ret;
        u16 stream_idx;
        struct wimfs_context *ctx = wimfs_get_context();
@@ -2125,43 +2193,43 @@ static int wimfs_unlink(const char *path)
  */
 static int wimfs_utimens(const char *path, const struct timespec tv[2])
 {
-       struct dentry *dentry;
-       struct inode *inode;
+       struct wim_dentry *dentry;
+       struct wim_inode *inode;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
        inode = dentry->d_inode;
 
        if (tv[0].tv_nsec != UTIME_OMIT) {
                if (tv[0].tv_nsec == UTIME_NOW)
-                       inode->last_access_time = get_wim_timestamp();
+                       inode->i_last_access_time = get_wim_timestamp();
                else
-                       inode->last_access_time = timespec_to_wim_timestamp(&tv[0]);
+                       inode->i_last_access_time = timespec_to_wim_timestamp(&tv[0]);
        }
        if (tv[1].tv_nsec != UTIME_OMIT) {
                if (tv[1].tv_nsec == UTIME_NOW)
-                       inode->last_write_time = get_wim_timestamp();
+                       inode->i_last_write_time = get_wim_timestamp();
                else
-                       inode->last_write_time = timespec_to_wim_timestamp(&tv[1]);
+                       inode->i_last_write_time = timespec_to_wim_timestamp(&tv[1]);
        }
        return 0;
 }
 #else
 static int wimfs_utime(const char *path, struct utimbuf *times)
 {
-       struct dentry *dentry;
-       struct inode *inode;
+       struct wim_dentry *dentry;
+       struct wim_inode *inode;
        WIMStruct *w = wimfs_get_WIMStruct();
 
        dentry = get_dentry(w, path);
        if (!dentry)
-               return -ENOENT;
+               return -errno;
        inode = dentry->d_inode;
 
-       inode->last_write_time = unix_timestamp_to_wim(times->modtime);
-       inode->last_access_time = unix_timestamp_to_wim(times->actime);
+       inode->i_last_write_time = unix_timestamp_to_wim(times->modtime);
+       inode->i_last_access_time = unix_timestamp_to_wim(times->actime);
        return 0;
 }
 #endif
@@ -2172,7 +2240,7 @@ static int wimfs_utime(const char *path, struct utimbuf *times)
 static int wimfs_write(const char *path, const char *buf, size_t size,
                       off_t offset, struct fuse_file_info *fi)
 {
-       struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
+       struct wimfs_fd *fd = (struct wimfs_fd*)(uintptr_t)fi->fh;
        int ret;
        u64 now;
 
@@ -2194,8 +2262,8 @@ static int wimfs_write(const char *path, const char *buf, size_t size,
                return -errno;
 
        now = get_wim_timestamp();
-       fd->f_inode->last_write_time = now;
-       fd->f_inode->last_access_time = now;
+       fd->f_inode->i_last_write_time = now;
+       fd->f_inode->i_last_access_time = now;
        return ret;
 }
 
@@ -2243,6 +2311,9 @@ static struct fuse_operations wimfs_operations = {
 #endif
        .write       = wimfs_write,
 
+       /* wimfs keeps file descriptor structures (struct wimfs_fd), so there is
+        * no need to have the file path provided on operations such as read()
+        * where only the file descriptor is needed. */
 #if FUSE_MAJOR_VERSION > 2 || (FUSE_MAJOR_VERSION == 2 && FUSE_MINOR_VERSION >= 8)
        .flag_nullpath_ok = 1,
 #endif
@@ -2263,11 +2334,11 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        char *argv[16];
        int ret;
        char *dir_copy;
-       struct lookup_table *joined_tab, *wim_tab_save;
-       struct image_metadata *imd;
+       struct wim_lookup_table *joined_tab, *wim_tab_save;
+       struct wim_image_metadata *imd;
        struct wimfs_context ctx;
        struct hlist_node *cur_node;
-       struct inode *inode;
+       struct wim_inode *inode;
 
        DEBUG("Mount: wim = %p, image = %d, dir = %s, flags = %d, ",
              wim, image, dir, mount_flags);
@@ -2315,6 +2386,9 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                goto out;
        }
 
+       if (imd->inode_list.first) /* May be unneeded? */
+               imd->inode_list.first->pprev = &imd->inode_list.first;
+
        if (imd->modified) {
                ERROR("Cannot mount image that was added "
                      "with wimlib_add_image()");
@@ -2328,6 +2402,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                        goto out;
        }
 
+       /* Use default stream interface if one was not specified */
        if (!(mount_flags & (WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_NONE |
                       WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR |
                       WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_WINDOWS)))
@@ -2366,11 +2441,8 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
                argv[argc++] = "-d";
 
        /*
-        * We provide the use_ino option because we are going to assign inode
-        * numbers oursides.  The inodes will be given unique numbers in the
-        * assign_inode_numbers() function, and the static variable @next_ino is
-        * set to the next available inode number.
-        */
+        * We provide the use_ino option to the FUSE mount because we are going
+        * to assign inode numbers ourselves. */
        char optstring[256] =
                "use_ino"
                ",subtype=wimfs"
@@ -2414,13 +2486,11 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        /* 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 = 1;
-       hlist_for_each_entry(inode, cur_node, &imd->inode_list, hlist) {
+       hlist_for_each_entry(inode, cur_node, &imd->inode_list, i_hlist) {
                inode_resolve_ltes(inode, wim->lookup_table);
-               inode->ino = ctx.next_ino++;
+               inode->i_ino = ctx.next_ino++;
        }
-       /*ctx.next_ino = assign_inode_numbers(&imd->inode_list);*/
        DEBUG("(next_ino = %"PRIu64")", ctx.next_ino);
 
        DEBUG("Calling fuse_main()");
@@ -2428,6 +2498,7 @@ WIMLIBAPI int wimlib_mount_image(WIMStruct *wim, int image, const char *dir,
        ret = fuse_main(argc, argv, &wimfs_operations, &ctx);
 
        DEBUG("Returned from fuse_main() (ret = %d)", ret);
+
        if (ret)
                ret = WIMLIB_ERR_FUSE;
 out_free_dir_copy:
@@ -2465,7 +2536,8 @@ WIMLIBAPI int wimlib_unmount_image(const char *dir, int unmount_flags,
                goto out_free_message_queue_names;
 
        ret = send_unmount_request_msg(wimfs_ctx.unmount_to_daemon_mq,
-                                      unmount_flags);
+                                      unmount_flags,
+                                      progress_func != NULL);
        if (ret != 0)
                goto out_close_message_queues;
 
@@ -2473,19 +2545,19 @@ WIMLIBAPI int wimlib_unmount_image(const char *dir, int unmount_flags,
        if (ret != 0)
                goto out_close_message_queues;
 
-       struct msg_handler_context handler_ctx = {
-               .is_daemon = false,
-               .timeout_seconds = 5,
-               .unmount = {
-                       .daemon_pid = 0,
+       struct unmount_msg_handler_context handler_ctx = {
+               .hdr = {
+                       .timeout_seconds = 5,
                },
+               .daemon_pid = 0,
+               .progress_func = progress_func,
        };
 
        ret = message_loop(wimfs_ctx.daemon_to_unmount_mq,
                           &unmount_msg_handler_callbacks,
-                          &handler_ctx);
+                          &handler_ctx.hdr);
        if (ret == 0)
-               ret = handler_ctx.unmount.status;
+               ret = handler_ctx.status;
 out_close_message_queues:
        close_message_queues(&wimfs_ctx);
 out_free_message_queue_names: