]> wimlib.net Git - wimlib/blobdiff - src/mount.c
update_lte_of_staging_file() fix
[wimlib] / src / mount.c
index b67f40c6faacaec36cbb8e969fabcce36899bb21..0da8eeeacd5c589b611c45aa36d3c81421b02b55 100644 (file)
@@ -53,8 +53,8 @@
 
 /* File descriptor to a file open on the WIM filesystem. */
 struct wimlib_fd {
-       struct inode *inode;
-       struct lookup_table_entry *lte;
+       struct inode *f_inode;
+       struct lookup_table_entry *f_lte;
        int staging_fd;
        u16 idx;
        u32 stream_id;
@@ -100,6 +100,16 @@ static inline int flags_writable(int open_flags)
        return open_flags & (O_RDWR | O_WRONLY);
 }
 
+/*
+ * Allocate a file descriptor for a stream.
+ *
+ * @inode:     inode containing the stream we're opening
+ * @stream_id: ID of the stream we're opening
+ * @lte:       Lookup table entry for the stream (may be NULL)
+ * @fd_ret:    Return the allocated file descriptor if successful.
+ *
+ * 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,
@@ -108,16 +118,22 @@ static int alloc_wimlib_fd(struct inode *inode,
        static const u16 fds_per_alloc = 8;
        static const u16 max_fds = 0xffff;
 
+       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);
+
        if (inode->num_opened_fds == inode->num_allocated_fds) {
                struct wimlib_fd **fds;
                u16 num_new_fds;
 
                if (inode->num_allocated_fds == max_fds)
                        return -EMFILE;
-               num_new_fds = min(fds_per_alloc, max_fds - inode->num_allocated_fds);
+               num_new_fds = min(fds_per_alloc,
+                                 max_fds - inode->num_allocated_fds);
                
-               fds = REALLOC(inode->fds, (inode->num_allocated_fds + num_new_fds) *
-                              sizeof(inode->fds[0]));
+               fds = REALLOC(inode->fds,
+                             (inode->num_allocated_fds + num_new_fds) *
+                               sizeof(inode->fds[0]));
                if (!fds)
                        return -ENOMEM;
                memset(&fds[inode->num_allocated_fds], 0,
@@ -130,15 +146,17 @@ static int alloc_wimlib_fd(struct inode *inode,
                        struct wimlib_fd *fd = CALLOC(1, sizeof(*fd));
                        if (!fd)
                                return -ENOMEM;
-                       fd->inode      = inode;
-                       fd->lte        = lte;
+                       fd->f_inode    = inode;
+                       fd->f_lte      = lte;
                        fd->staging_fd = -1;
                        fd->idx        = i;
                        fd->stream_id  = stream_id;
                        *fd_ret        = fd;
                        inode->fds[i]  = fd;
                        inode->num_opened_fds++;
-                       lte->num_opened_fds++;
+                       if (lte)
+                               lte->num_opened_fds++;
+                       DEBUG("Allocated fd");
                        return 0;
                }
        }
@@ -148,7 +166,7 @@ static void inode_put_fd(struct inode *inode, struct wimlib_fd *fd)
 {
        wimlib_assert(fd);
        wimlib_assert(inode);
-       wimlib_assert(fd->inode == inode);
+       wimlib_assert(fd->f_inode == inode);
        wimlib_assert(inode->num_opened_fds);
        wimlib_assert(fd->idx < inode->num_opened_fds);
        wimlib_assert(inode->fds[fd->idx] == fd);
@@ -162,52 +180,61 @@ 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);
-       wimlib_assert(fd->lte == lte);
+       wimlib_assert(fd->f_lte == lte);
 
        if (!lte) /* Empty stream with no lookup table entry */
                return 0;
 
+       /* Close staging file descriptor if needed. */
        wimlib_assert(lte->num_opened_fds);
 
-       if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
-               wimlib_assert(lte->staging_file_name);
-               wimlib_assert(fd->staging_fd != -1);
-               if (close(fd->staging_fd) != 0)
+       if (lte->resource_location == RESOURCE_IN_STAGING_FILE
+            && fd->staging_fd != -1)
+       {
+               if (close(fd->staging_fd) != 0) {
+                       ERROR_WITH_ERRNO("Failed to close staging file");
                        return -errno;
+               }
        }
        lte_decrement_num_opened_fds(lte, w->lookup_table);
        return 0;
 }
 
+/* Close a file descriptor. */
 static int close_wimlib_fd(struct wimlib_fd *fd)
 {
        int ret;
-       ret = lte_put_fd(fd->lte, fd);
+       wimlib_assert(fd);
+       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);
+       ret = lte_put_fd(fd->f_lte, fd);
        if (ret != 0)
                return ret;
 
-       inode_put_fd(fd->inode, fd);
+       inode_put_fd(fd->f_inode, fd);
        return 0;
 }
 
-/* Remove a dentry; i.e. remove a reference to the inode.
+/* Remove a dentry; i.e. remove a reference to the corresponding inode.
  *
- * If there are no remaining references to the inode either through detnries or
- * open file descriptors, the inode is freed.
+ * 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
+ * removed, but the dentry is unlinked and freed.
  *
- * 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.
- *
- * Otherwise, the inode is not removed, but the dentry is unlinked and freed. */
+ * 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.
+ */
 static void remove_dentry(struct dentry *dentry,
                          struct lookup_table *lookup_table)
 {
-       struct inode *inode = dentry->inode;
+       struct inode *inode = dentry->d_inode;
        struct lookup_table_entry *lte;
        unsigned i;
 
-       for (i = 0; <= inode->num_ads; i++) {
+       for (i = 0; <= inode->num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte_decrement_refcnt(lte, lookup_table);
@@ -216,76 +243,6 @@ static void remove_dentry(struct dentry *dentry,
        put_dentry(dentry);
 }
 
-/* Remove an alternate data stream from the inode  */
-static void inode_remove_ads(struct inode *inode, u16 idx,
-                            struct lookup_table *lookup_table)
-{
-       struct ads_entry *ads_entry;
-       struct lookup_table_entry *lte;
-
-       ads_entry = inode->ads_entries[idx];
-
-       wimlib_assert(ads_entry);
-
-       lte = ads_entry->lte;
-
-       if (lte)
-               lte_decrement_refcnt(lte, lookup_table);
-
-       free_ads_entry(ads_entry);
-
-       wimlib_assert(inode->num_ads);
-       memcpy(&inode->ads_entries[idx],
-              &inode->ads_entries[idx + 1],
-              (inode->num_ads - idx - 1) * sizeof(inode->ads_entries[0]));
-       inode->num_ads--;
-}
-
-/* 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. */
-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 | 0644;
-
-       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) {
-                       wimlib_assert(mount_flags & WIMLIB_MOUNT_FLAG_READWRITE);
-                       wimlib_assert(lte->staging_file_name);
-                       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 = lte->resource_entry.original_size;
-               }
-       } 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.
  *
@@ -346,6 +303,9 @@ static int create_staging_file(char **name_ret, int open_flags)
  *
  * @inode:  Inode that contains the stream we are extracting
  *
+ * @stream_id: Identifier for the stream (it stays constant even if the indices
+ * of the stream entries are changed)
+ *
  * @lte: Pointer to pointer to the lookup table entry for the stream we need to
  * extract, or NULL if there was no lookup table entry present for the stream
  *
@@ -380,63 +340,104 @@ static int extract_resource_to_staging_dir(struct inode *inode,
                goto out_delete_staging_file;
        }
 
-       if (old_lte) {
-               wimlib_assert(old_lte->resource_location == RESOURCE_IN_WIM);
-               if (inode->link_count == old_lte->refcnt) {
-                       /* The reference count of the existing lookup table
-                        * entry is the same as the size of the hard link group
-                        * associated with the dentry; 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(w->lookup_table, old_lte);
-                       new_lte = old_lte;
-               } else {
+       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.  */
+               DEBUG("Re-using lookup table entry");
+               lookup_table_unlink(w->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
+                        * 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 = %zu, lte refcnt = %u)",
+                             "(inode->link_count = %zu, old_lte->refcnt = %u)",
                              inode->link_count, old_lte->refcnt);
 
-                       wimlib_assert(old_lte->refcnt > inode->link_count);
+               }
 
-                       new_lte = new_lookup_table_entry();
-                       if (!new_lte) {
-                               ret = -ENOMEM;
-                               goto out_delete_staging_file;
-                       }
-                       old_lte->refcnt -= inode->link_count;
+               new_lte = new_lookup_table_entry();
+               if (!new_lte) {
+                       ret = -ENOMEM;
+                       goto out_delete_staging_file;
+               }
 
-                       for (u16 i = 0, j = 0; j < inode->num_opened_fds; i++) {
-                               if (inode->fds[i]) {
-                                       if (inode->fds[i]->stream_id == stream_id) {
-                                               wimlib_assert(inode->fds[i]->lte == old_lte);
-                                               inode->fds[i]->lte = new_lte;
+               /* There may already be open file descriptors to this stream if
+                * it's previously been opened read-only, but just now we're
+                * opening it read-write.  Identify those file descriptors and
+                * change their lookup table entry pointers to point to the new
+                * lookup table entry, and open staging file descriptors for
+                * them. 
+                *
+                * At the same time, we need to count the number of these opened
+                * 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];
+                       if (fd) {
+                               if (fd->stream_id == stream_id) {
+                                       wimlib_assert(fd->f_lte == old_lte);
+                                       wimlib_assert(fd->staging_fd == -1);
+                                       fd->f_lte = new_lte;
+                                       new_lte->num_opened_fds++;
+                                       fd->staging_fd = open(staging_file_name, O_RDONLY);
+                                       if (fd->staging_fd == -1) {
+                                               ret = -errno;
+                                               goto out_revert_fd_changes;
                                        }
-                                       j++;
                                }
+                               j++;
                        }
                }
-       } else {
-               /* No old_lte was supplied, so the resource had no lookup table
-                * entry before (it must be an empty resource) */
-               wimlib_assert(inode->num_opened_fds == 0);
-               new_lte = new_lookup_table_entry();
-               if (!new_lte) {
-                       ret = -ENOMEM;
-                       goto out_delete_staging_file;
+               DEBUG("%zu fd's were already opened to the file we extracted",
+                     new_lte->num_opened_fds);
+               if (old_lte) {
+                       old_lte->num_opened_fds -= new_lte->num_opened_fds;
+                       old_lte->refcnt -= inode->link_count;
                }
        }
+
        new_lte->resource_entry.original_size = size;
-       new_lte->refcnt = inode->link_count;
+       new_lte->refcnt                       = inode->link_count;
+       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);
-       new_lte->staging_file_name = staging_file_name;
-       new_lte->resource_location = RESOURCE_IN_STAGING_FILE;
+
+       if (stream_id == 0)
+               inode->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;
 
        lookup_table_insert(w->lookup_table, new_lte);
        list_add(&new_lte->staging_list, &staging_list);
        *lte = new_lte;
        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];
+               if (fd && fd->stream_id == stream_id && fd->f_lte == new_lte) {
+                       fd->f_lte = old_lte;
+                       if (fd->staging_fd != -1) {
+                               close(fd->staging_fd);
+                               fd->staging_fd = -1;
+                       }
+                       j++;
+               }
+       }
+out_free_new_lte:
+       free_lookup_table_entry(new_lte);
 out_delete_staging_file:
        unlink(staging_file_name);
        FREE(staging_file_name);
@@ -669,54 +670,25 @@ static void close_message_queues()
 
 static int wimfs_access(const char *path, int mask)
 {
-       /* XXX Permissions not implemented */
+       /* Permissions not implemented */
        return 0;
 }
 
-#if 0
-/* Closes the staging file descriptor associated with the lookup table entry, if
- * it is opened. */
-static int close_lte_fds(struct lookup_table_entry *lte)
+static void inode_update_lte_ptr(struct inode *inode,
+                                struct lookup_table_entry *old_lte,
+                                struct lookup_table_entry *new_lte)
 {
-       for (u16 i = 0, j = 0; j < lte->num_opened_fds; i++) {
-               if (lte->fds[i] && lte->fds[i]->staging_fd != -1) {
-                       wimlib_assert(lte->resource_location == RESOURCE_IN_STAGING_FILE);
-                       wimlib_assert(lte->staging_file_name);
-                       if (close(lte->fds[i]->staging_fd) != 0) {
-                               ERROR_WITH_ERRNO("Failed close file `%s'",
-                                                lte->staging_file_name);
-                               return WIMLIB_ERR_WRITE;
+       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;
                        }
-                       j++;
-               }
-       }
-       return 0;
-}
-
-static void lte_list_change_lte_ptr(struct lookup_table_entry *lte,
-                                   struct lookup_table_entry *newptr)
-{
-       struct list_head *pos;
-       struct stream_list_head *head;
-       list_for_each(pos, &lte->lte_group_list) {
-               head = container_of(pos, struct stream_list_head, list);
-               if (head->type == STREAM_TYPE_ADS) {
-                       struct ads_entry *ads_entry;
-                       ads_entry = container_of(head, struct ads_entry, lte_group_list);
-
-                       ads_entry->lte = newptr;
-               } else {
-                       wimlib_assert(head->type == STREAM_TYPE_NORMAL);
-
-                       struct dentry *dentry;
-                       dentry = container_of(head, struct dentry, lte_group_list);
-
-                       dentry->lte = newptr;
                }
        }
 }
-#endif
-
 
 static int update_lte_of_staging_file(struct lookup_table_entry *lte,
                                      struct lookup_table *table)
@@ -739,30 +711,45 @@ static int update_lte_of_staging_file(struct lookup_table_entry *lte,
 
        if (duplicate_lte) {
                /* Merge duplicate lookup table entries */
-
-#if 0
-               lte_list_change_lte_ptr(lte, duplicate_lte);
-#endif
                duplicate_lte->refcnt += lte->refcnt;
-#if 0
-               list_splice(&lte->lte_group_list,
-                           &duplicate_lte->lte_group_list);
-#endif
-
+               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;
                }
-               wimlib_assert(&lte->file_on_disk == &lte->staging_file_name);
-               lte->resource_location = RESOURCE_IN_FILE_ON_DISK;
-               copy_hash(lte->hash, hash);
-               lte->resource_entry.original_size = stbuf.st_size;
-               lte->resource_entry.size = stbuf.st_size;
-               lookup_table_insert(table, lte);
+               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);
+               }
        }
+       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;
 }
 
@@ -772,18 +759,14 @@ static int rebuild_wim(WIMStruct *w, bool check_integrity)
        int ret;
        struct lookup_table_entry *lte, *tmp;
 
-       /* Close all the staging file descriptors. */
-#if 0
+
        DEBUG("Closing all staging file descriptors.");
        list_for_each_entry(lte, &staging_list, staging_list) {
-               ret = close_lte_fds(lte);
+               ret = inode_close_fds(lte->lte_inode);
                if (ret != 0)
                        return ret;
        }
-#endif
 
-       /* Calculate SHA1 checksums for all staging files, and merge unnecessary
-        * lookup table entries. */
        DEBUG("Calculating SHA1 checksums for all new staging files.");
        list_for_each_entry_safe(lte, tmp, &staging_list, staging_list) {
                ret = update_lte_of_staging_file(lte, w->lookup_table);
@@ -888,13 +871,14 @@ static int wimfs_fallocate(const char *path, int mode,
        return fallocate(fd->staging_fd, mode, offset, len);
 }
 
+#endif
+
 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;
-       return dentry_to_stbuf(fd->dentry, stbuf);
+       return inode_to_stbuf(fd->f_inode, fd->f_lte, stbuf);
 }
-#endif
 
 static int wimfs_ftruncate(const char *path, off_t size,
                           struct fuse_file_info *fi)
@@ -903,7 +887,8 @@ static int wimfs_ftruncate(const char *path, off_t size,
        int ret = ftruncate(fd->staging_fd, size);
        if (ret != 0)
                return ret;
-       fd->lte->resource_entry.original_size = size;
+       if (fd->f_lte && size < fd->f_lte->resource_entry.original_size)
+               fd->f_lte->resource_entry.original_size = size;
        return 0;
 }
 
@@ -921,7 +906,7 @@ static int wimfs_getattr(const char *path, struct stat *stbuf)
                              &dentry, &lte, NULL);
        if (ret != 0)
                return ret;
-       return inode_to_stbuf(dentry->inode, lte, stbuf);
+       return inode_to_stbuf(dentry->d_inode, lte, stbuf);
 }
 
 #ifdef ENABLE_XATTR
@@ -945,7 +930,7 @@ static int wimfs_getxattr(const char *path, const char *name, char *value,
        dentry = get_dentry(w, path);
        if (!dentry)
                return -ENOENT;
-       ads_entry = inode_get_ads_entry(dentry->inode, name, NULL);
+       ads_entry = inode_get_ads_entry(dentry->d_inode, name, NULL);
        if (!ads_entry)
                return -ENOATTR;
 
@@ -966,18 +951,15 @@ 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 *to_dentry, *from_dentry, *from_dentry_parent;
+       struct dentry *from_dentry, *from_dentry_parent;
        const char *link_name;
        struct inode *inode;
-       unsigned i;
        struct lookup_table_entry *lte;
 
-       to_dentry = get_dentry(w, to);
-       if (!to_dentry)
+       inode = wim_pathname_to_inode(w, to);
+       if (!inode)
                return -ENOENT;
 
-       inode = to_dentry->inode;
-
        if (!inode_is_regular_file(inode))
                return -EPERM;
 
@@ -996,10 +978,10 @@ static int wimfs_link(const char *to, const char *from)
 
 
        inode_add_dentry(from_dentry, inode);
-       from_dentry->inode = inode;
+       from_dentry->d_inode = inode;
        inode->link_count++;
 
-       for (i = 0; i <= inode->num_ads; i++) {
+       for (unsigned i = 0; i <= inode->num_ads; i++) {
                lte = inode_stream_lte_resolved(inode, i);
                if (lte)
                        lte->refcnt++;
@@ -1026,20 +1008,20 @@ static int wimfs_listxattr(const char *path, char *list, size_t size)
        ret = lookup_resource(w, path, get_lookup_flags(), &dentry, NULL, NULL);
        if (ret != 0)
                return ret;
-       inode = dentry->inode;
+       inode = dentry->d_inode;
 
        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;
+                       needed_size += inode->ads_entries[i].stream_name_utf8_len + 6;
                return needed_size;
        } else {
                for (i = 0; i < inode->num_ads; i++) {
-                       needed_size = inode->ads_entries[i]->stream_name_utf8_len + 6;
+                       needed_size = inode->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->ads_entries[i].stream_name_utf8) + 1;
                        size -= needed_size;
                }
                return p - list;
@@ -1069,14 +1051,13 @@ static int wimfs_mkdir(const char *path, mode_t mode)
                return -EEXIST;
 
        newdir = new_dentry_with_inode(basename);
-       newdir->inode->attributes |= FILE_ATTRIBUTE_DIRECTORY;
-       newdir->inode->resolved = true;
-       newdir->inode->ino = next_ino++;
+       newdir->d_inode->attributes |= FILE_ATTRIBUTE_DIRECTORY;
+       newdir->d_inode->resolved = true;
+       newdir->d_inode->ino = next_ino++;
        link_dentry(newdir, parent);
        return 0;
 }
 
-
 /* Creates a regular file. */
 static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
 {
@@ -1094,9 +1075,9 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
                dentry = get_dentry(w, path);
                if (!dentry || !dentry_is_regular_file(dentry))
                        return -ENOENT;
-               if (inode_get_ads_entry(dentry->inode, stream_name, NULL))
+               if (inode_get_ads_entry(dentry->d_inode, stream_name, NULL))
                        return -EEXIST;
-               new_entry = inode_add_ads(dentry->inode, stream_name);
+               new_entry = inode_add_ads(dentry->d_inode, stream_name);
                if (!new_entry)
                        return -ENOENT;
        } else {
@@ -1120,8 +1101,8 @@ static int wimfs_mknod(const char *path, mode_t mode, dev_t rdev)
                dentry = new_dentry_with_inode(basename);
                if (!dentry)
                        return -ENOMEM;
-               dentry->inode->resolved = true;
-               dentry->inode->ino = next_ino++;
+               dentry->d_inode->resolved = true;
+               dentry->d_inode->ino = next_ino++;
                link_dentry(dentry, parent);
        }
        return 0;
@@ -1144,12 +1125,12 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
        if (ret != 0)
                return ret;
 
-       inode = dentry->inode;
+       inode = dentry->d_inode;
 
        if (stream_idx == 0)
                stream_id = 0;
        else
-               stream_id = inode->ads_entries[stream_idx - 1]->stream_id;
+               stream_id = inode->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
@@ -1157,11 +1138,12 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
         * we can read the file resource directly from the WIM file if we are
         * opening it read-only, but we need to extract the resource to the
         * staging directory if we are opening it writable. */
+
        if (flags_writable(fi->flags) &&
-             lte->resource_location != RESOURCE_IN_STAGING_FILE) {
+            (!lte || lte->resource_location != RESOURCE_IN_STAGING_FILE)) {
+               u64 size = (lte) ? wim_resource_size(lte) : 0;
                ret = extract_resource_to_staging_dir(inode, stream_id,
-                                                     &lte,
-                                                     wim_resource_size(lte));
+                                                     &lte, size);
                if (ret != 0)
                        return ret;
        }
@@ -1170,7 +1152,7 @@ static int wimfs_open(const char *path, struct fuse_file_info *fi)
        if (ret != 0)
                return ret;
 
-       if (lte->resource_location == RESOURCE_IN_STAGING_FILE) {
+       if (lte && lte->resource_location == RESOURCE_IN_STAGING_FILE) {
                fd->staging_fd = open(lte->staging_file_name, fi->flags);
                if (fd->staging_fd == -1) {
                        close_wimlib_fd(fd);
@@ -1184,17 +1166,13 @@ 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 dentry *dentry;
        struct inode *inode;
        int ret;
        struct wimlib_fd *fd = NULL;
        
-       dentry = get_dentry(w, path);
-       if (!dentry)
+       inode = wim_pathname_to_inode(w, path);
+       if (!inode)
                return -ENOENT;
-
-       inode = dentry->inode;
-
        if (!inode_is_directory(inode))
                return -ENOTDIR;
        ret = alloc_wimlib_fd(inode, 0, NULL, &fd);
@@ -1214,13 +1192,13 @@ static int wimfs_read(const char *path, char *buf, size_t size,
        if (!fd)
                return -EBADF;
 
-       if (!fd->lte) /* Empty stream with no lookup table entry */
+       if (!fd->f_lte) /* Empty stream with no lookup table entry */
                return 0;
 
-       if (fd->lte->resource_location == RESOURCE_IN_STAGING_FILE) {
+       if (fd->f_lte->resource_location == RESOURCE_IN_STAGING_FILE) {
                /* Read from staging file */
 
-               wimlib_assert(fd->lte->staging_file_name);
+               wimlib_assert(fd->f_lte->staging_file_name);
                wimlib_assert(fd->staging_fd != -1);
 
                ssize_t ret;
@@ -1235,14 +1213,14 @@ static int wimfs_read(const char *path, char *buf, size_t size,
        } else {
                /* Read from WIM */
 
-               u64 res_size = wim_resource_size(fd->lte);
+               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->lte, (u8*)buf,
+               if (read_wim_resource(fd->f_lte, (u8*)buf,
                                      size, offset, false) != 0)
                        return -EIO;
                return size;
@@ -1261,7 +1239,7 @@ static int wimfs_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
        if (!fd)
                return -EBADF;
 
-       inode = fd->inode;
+       inode = fd->f_inode;
 
        filler(buf, ".", NULL, 0);
        filler(buf, "..", NULL, 0);
@@ -1289,7 +1267,7 @@ static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
        if (!dentry_is_symlink(dentry))
                return -EINVAL;
 
-       ret = inode_readlink(dentry->inode, buf, buf_len, w);
+       ret = inode_readlink(dentry->d_inode, buf, buf_len, w);
        if (ret > 0)
                ret = 0;
        return ret;
@@ -1299,14 +1277,13 @@ static int wimfs_readlink(const char *path, char *buf, size_t buf_len)
 static int wimfs_release(const char *path, struct fuse_file_info *fi)
 {
        struct wimlib_fd *fd = (struct wimlib_fd*)(uintptr_t)fi->fh;
-       wimlib_assert(fd);
        return close_wimlib_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;
-       wimlib_assert(fd);
        return close_wimlib_fd(fd);
 }
 
@@ -1328,10 +1305,10 @@ static int wimfs_removexattr(const char *path, const char *name)
        if (!dentry)
                return -ENOENT;
 
-       ads_entry = inode_get_ads_entry(dentry->inode, name, &ads_idx);
+       ads_entry = inode_get_ads_entry(dentry->d_inode, name, &ads_idx);
        if (!ads_entry)
                return -ENOATTR;
-       inode_remove_ads(dentry->inode, ads_idx, w->lookup_table);
+       inode_remove_ads(dentry->d_inode, ads_idx, w->lookup_table);
        return 0;
 }
 #endif
@@ -1377,7 +1354,7 @@ static int wimfs_rename(const char *from, const char *to)
                         * directory */
                        if (!dentry_is_directory(dst))
                                return -ENOTDIR;
-                       if (dst->inode->children != NULL)
+                       if (dst->d_inode->children != NULL)
                                return -ENOTEMPTY;
                }
                parent_of_dst = dst->parent;
@@ -1425,7 +1402,6 @@ 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 dentry *dentry;
        struct ads_entry *existing_ads_entry;
        struct ads_entry *new_ads_entry;
        struct lookup_table_entry *existing_lte;
@@ -1441,21 +1417,20 @@ static int wimfs_setxattr(const char *path, const char *name,
                return -ENOATTR;
        name += 5;
 
-       dentry = get_dentry(w, path);
-       if (!dentry)
+       inode = wim_pathname_to_inode(w, path);
+       if (!inode)
                return -ENOENT;
-       inode = dentry->inode;
 
        existing_ads_entry = inode_get_ads_entry(inode, name, &ads_idx);
        if (existing_ads_entry) {
                if (flags & XATTR_CREATE)
                        return -EEXIST;
-               inode_remove_ads(dentry->inode, ads_idx, w->lookup_table);
+               inode_remove_ads(inode, ads_idx, w->lookup_table);
        } else {
                if (flags & XATTR_REPLACE)
                        return -ENOATTR;
        }
-       new_ads_entry = inode_add_ads(dentry, name);
+       new_ads_entry = inode_add_ads(inode, name);
        if (!new_ads_entry)
                return -ENOMEM;
 
@@ -1494,7 +1469,7 @@ static int wimfs_symlink(const char *to, const char *from)
 {
        struct dentry *dentry_parent, *dentry;
        const char *link_name;
-       struct lookup_table_entry *lte;
+       struct inode *inode;
        
        dentry_parent = get_parent_dentry(w, from);
        if (!dentry_parent)
@@ -1506,24 +1481,19 @@ static int wimfs_symlink(const char *to, const char *from)
 
        if (get_dentry_child_with_name(dentry_parent, link_name))
                return -EEXIST;
-       dentry = new_dentry(link_name);
+       dentry = new_dentry_with_inode(link_name);
        if (!dentry)
                return -ENOMEM;
+       inode = dentry->d_inode;
 
-       dentry->attributes = FILE_ATTRIBUTE_REPARSE_POINT;
-       dentry->reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
-       dentry->link_group_id = next_link_group_id++;
+       inode->attributes  = FILE_ATTRIBUTE_REPARSE_POINT;
+       inode->reparse_tag = WIM_IO_REPARSE_TAG_SYMLINK;
+       inode->ino         = next_ino++;
+       inode->resolved    = true;
 
-       if (dentry_set_symlink(dentry, to, w->lookup_table, &lte) != 0)
+       if (inode_set_symlink(inode, to, w->lookup_table, NULL) != 0)
                goto out_free_dentry;
 
-       wimlib_assert(lte);
-
-       dentry->ads_entries[1].lte_group_list.type = STREAM_TYPE_ADS;
-       list_add(&dentry->ads_entries[1].lte_group_list.list,
-                &lte->lte_group_list);
-       wimlib_assert(dentry->resolved);
-
        link_dentry(dentry, dentry_parent);
        return 0;
 out_free_dentry:
@@ -1538,7 +1508,9 @@ static int wimfs_truncate(const char *path, off_t size)
        struct dentry *dentry;
        struct lookup_table_entry *lte;
        int ret;
-       unsigned stream_idx;
+       u16 stream_idx;
+       u32 stream_id;
+       struct inode *inode;
        
        ret = lookup_resource(w, path, get_lookup_flags(), &dentry,
                              &lte, &stream_idx);
@@ -1549,6 +1521,13 @@ static int wimfs_truncate(const char *path, off_t size)
        if (!lte) /* Already a zero-length file */
                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);
@@ -1559,7 +1538,7 @@ static int wimfs_truncate(const char *path, off_t size)
                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(dentry, stream_idx,
+               ret = extract_resource_to_staging_dir(inode, stream_id,
                                                      &lte, size);
        }
        return ret;
@@ -1572,7 +1551,7 @@ static int wimfs_unlink(const char *path)
        struct lookup_table_entry *lte;
        struct inode *inode;
        int ret;
-       unsigned stream_idx;
+       u16 stream_idx;
        unsigned i;
        
        ret = lookup_resource(w, path, get_lookup_flags(), &dentry,
@@ -1584,7 +1563,7 @@ static int wimfs_unlink(const char *path)
        if (stream_idx == 0)
                remove_dentry(dentry, w->lookup_table);
        else
-               inode_remove_ads(dentry->inode, stream_idx - 1, w->lookup_table);
+               inode_remove_ads(dentry->d_inode, stream_idx - 1, w->lookup_table);
        return 0;
 }
 
@@ -1601,7 +1580,7 @@ static int wimfs_utimens(const char *path, const struct timespec tv[2])
        dentry = get_dentry(w, path);
        if (!dentry)
                return -ENOENT;
-       inode = dentry->inode;
+       inode = dentry->d_inode;
 
        if (tv[0].tv_nsec != UTIME_OMIT) {
                if (tv[0].tv_nsec == UTIME_NOW)
@@ -1625,7 +1604,7 @@ static int wimfs_utime(const char *path, struct utimbuf *times)
        dentry = get_dentry(w, path);
        if (!dentry)
                return -ENOENT;
-       inode = dentry->inode;
+       inode = dentry->d_inode;
 
        inode->last_write_time = unix_timestamp_to_wim(times->modtime);
        inode->last_access_time = unix_timestamp_to_wim(times->actime);
@@ -1646,10 +1625,10 @@ static int wimfs_write(const char *path, const char *buf, size_t size,
        if (!fd)
                return -EBADF;
 
-       wimlib_assert(fd->lte);
-       wimlib_assert(fd->lte->staging_file_name);
+       wimlib_assert(fd->f_lte);
+       wimlib_assert(fd->f_lte->staging_file_name);
        wimlib_assert(fd->staging_fd != -1);
-       wimlib_assert(fd->inode);
+       wimlib_assert(fd->f_inode);
 
        /* Seek to the requested position */
        if (lseek(fd->staging_fd, offset, SEEK_SET) == -1)
@@ -1661,8 +1640,8 @@ static int wimfs_write(const char *path, const char *buf, size_t size,
                return -errno;
 
        now = get_wim_timestamp();
-       fd->inode->last_write_time = now;
-       fd->inode->last_access_time = now;
+       fd->f_inode->last_write_time = now;
+       fd->f_inode->last_access_time = now;
        return ret;
 }
 
@@ -1671,8 +1650,8 @@ static struct fuse_operations wimfs_operations = {
        .destroy     = wimfs_destroy,
 #if 0
        .fallocate   = wimfs_fallocate,
-       .fgetattr    = wimfs_fgetattr,
 #endif
+       .fgetattr    = wimfs_fgetattr,
        .ftruncate   = wimfs_ftruncate,
        .getattr     = wimfs_getattr,
 #ifdef ENABLE_XATTR
@@ -1711,46 +1690,6 @@ static struct fuse_operations wimfs_operations = {
 };
 
 
-static int check_lte_refcnt(struct lookup_table_entry *lte, void *ignore)
-{
-       size_t lte_group_size = 0;
-       struct list_head *cur;
-       list_for_each(cur, &lte->lte_group_list)
-               lte_group_size++;
-       if (lte_group_size > lte->refcnt) {
-#ifdef ENABLE_ERROR_MESSAGES
-               struct dentry *example_dentry;
-               struct list_head *next;
-               struct stream_list_head *head;
-               WARNING("The following lookup table entry has a reference count "
-                     "of %u, but", lte->refcnt);
-               WARNING("We found %zu references to it", lte_group_size);
-               next = lte->lte_group_list.next;
-               head = container_of(next, struct stream_list_head, list);
-               if (head->type == STREAM_TYPE_NORMAL) {
-                       example_dentry = container_of(head, struct dentry,
-                                                     lte_group_list);
-                       WARNING("(One dentry referencing it is at `%s')",
-                               example_dentry->full_path_utf8);
-               }
-               print_lookup_table_entry(lte);
-#endif
-               /* Guess what!  install.wim for Windows 8 contains a stream with
-                * 2 dentries referencing it, but the lookup table entry has
-                * reference count of 1.  So we will need to handle this case
-                * and not just make it be an error...  I'm just setting the
-                * reference count to the number of references we found. */
-
-               #if 1
-               lte->refcnt = lte_group_size;
-               WARNING("Fixing reference count");
-               #else
-               return WIMLIB_ERR_INVALID_DENTRY;
-               #endif
-       }
-       return 0;
-}
-
 /* Mounts a WIM file. */
 WIMLIBAPI int wimlib_mount(WIMStruct *wim, int image, const char *dir, 
                           int flags, WIMStruct **additional_swms,
@@ -1761,6 +1700,7 @@ WIMLIBAPI int wimlib_mount(WIMStruct *wim, int image, const char *dir,
        int ret;
        char *p;
        struct lookup_table *joined_tab, *wim_tab_save;
+       struct image_metadata *imd;
 
        DEBUG("Mount: wim = %p, image = %d, dir = %s, flags = %d, ",
                        wim, image, dir, flags);
@@ -1787,23 +1727,21 @@ WIMLIBAPI int wimlib_mount(WIMStruct *wim, int image, const char *dir,
        if (ret != 0)
                goto out;
 
+       imd = &wim->image_metadata[image - 1];
+
        DEBUG("Selected image %d", image);
 
-       next_link_group_id = assign_link_group_ids(wim->image_metadata[image - 1].lgt);
+       next_ino = assign_inode_numbers(&imd->inode_list);
+
+       DEBUG("(next_ino = %"PRIu64")", next_ino);
 
-       DEBUG("Resolving lookup table entries");
        /* Resolve all the lookup table entries of the dentry tree */
-       for_dentry_in_tree(wim_root_dentry(wim), dentry_resolve_ltes,
+       DEBUG("Resolving lookup table entries");
+       for_dentry_in_tree(imd->root_dentry, dentry_resolve_ltes,
                           wim->lookup_table);
 
-       DEBUG("Checking lookup table entry reference counts");
-
-       ret = for_lookup_table_entry(wim->lookup_table, check_lte_refcnt, NULL);
-       if (ret != 0)
-               goto out;
-
        if (flags & WIMLIB_MOUNT_FLAG_READWRITE)
-               wim_get_current_image_metadata(wim)->modified = true;
+               imd->modified = true;
 
        if (!(flags & (WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_NONE |
                       WIMLIB_MOUNT_FLAG_STREAM_INTERFACE_XATTR |
@@ -1847,12 +1785,11 @@ WIMLIBAPI int wimlib_mount(WIMStruct *wim, int image, const char *dir,
 
        /* 
         * We provide the use_ino option because we are going to assign inode
-        * numbers oursides.  We've already numbered the hard link groups with
-        * unique numbers with the assign_link_groups() function, and the static
-        * variable next_link_group_id is set to the next available link group
-        * ID that we will assign to new dentries.
+        * numbers oursides.  We've already numbered the inodes with unique
+        * numbers in the assign_inode_numbers() function, and the static
+        * variable next_ino is set to the next available inode number.
         */
-       char optstring[256] = "use_ino";
+       char optstring[256] = "use_ino,subtype=wimfs,attr_timeout=0";
        argv[argc++] = "-o";
        argv[argc++] = optstring;
        if ((flags & WIMLIB_MOUNT_FLAG_READWRITE)) {
@@ -1867,7 +1804,6 @@ WIMLIBAPI int wimlib_mount(WIMStruct *wim, int image, const char *dir,
                /* Read-only mount */
                strcat(optstring, ",ro");
        }
-       strcat(optstring, ",subtype=wimfs,attr_timeout=0");
        argv[argc] = NULL;
 
 #ifdef ENABLE_DEBUG