Changeset d0d7afb in mainline for uspace/lib/ext4/libext4_directory_index.c

Timestamp:

2012-03-01T19:56:31Z (12 years ago)

Author:

Frantisek Princ <frantisek.princ@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c30a015

Parents:

d8269dc

Message:

Code refactorization (entry insertion)

File:

• uspace/lib/ext4/libext4_directory_index.c (modified) (8 diffs)

uspace/lib/ext4/libext4_directory_index.c

@@ -503 +503 @@
                 block_t *old_data_block, ext4_directory_dx_block_t *index_block, block_t **new_data_block)
 {
-        int rc;
+        int rc = EOK;
 
         uint32_t block_size = ext4_superblock_get_block_size(fs->superblock);
@@ -652 +652 @@
 
 int ext4_directory_dx_add_entry(ext4_filesystem_t *fs,
-                ext4_inode_ref_t *parent, ext4_inode_ref_t *child,
-                size_t name_size, const char *name)
-{
-        int rc;
+                ext4_inode_ref_t *parent, ext4_inode_ref_t *child, const char *name)
+{
+        int rc = EOK;
+        int rc2;
 
         // get direct block 0 (index root)
@@ -670 +670 @@
         }
 
+        uint32_t name_len = strlen(name);
         ext4_hash_info_t hinfo;
-        rc = ext4_directory_hinfo_init(&hinfo, root_block, fs->superblock, name_size, name);
+        rc = ext4_directory_hinfo_init(&hinfo, root_block, fs->superblock, name_len, name);
         if (rc != EOK) {
                 block_put(root_block);
@@ -702 +703 @@
         }
 
-        uint32_t block_size = ext4_superblock_get_block_size(fs->superblock);
-        uint16_t required_len = 8 + name_size + (4 - name_size % 4);
-
-        ext4_directory_entry_ll_t *de = target_block->data;
-        ext4_directory_entry_ll_t *stop = target_block->data + block_size;
-
-        while (de < stop) {
-
-                uint32_t de_inode = ext4_directory_entry_ll_get_inode(de);
-                uint16_t de_rec_len = ext4_directory_entry_ll_get_entry_length(de);
-
-                if ((de_inode == 0) && (de_rec_len >= required_len)) {
-                        ext4_directory_write_entry(fs->superblock, de, de_rec_len,
-                                child, name, name_size);
-
-                                // TODO cleanup
-                                target_block->dirty = true;
-                                rc = block_put(target_block);
-                                if (rc != EOK) {
-                                        return EXT4_ERR_BAD_DX_DIR;
-                                }
-                                return EOK;
-                }
-
-                if (de_inode != 0) {
-                        uint16_t used_name_len = ext4_directory_entry_ll_get_name_length(
-                                        fs->superblock, de);
-
-                        uint16_t used_space = 8 + used_name_len;
-                        if ((used_name_len % 4) != 0) {
-                                used_space += 4 - (used_name_len % 4);
-                        }
-                        uint16_t free_space = de_rec_len - used_space;
-
-                        if (free_space >= required_len) {
-
-                                // Cut tail of current entry
-                                ext4_directory_entry_ll_set_entry_length(de, used_space);
-                                ext4_directory_entry_ll_t *new_entry = (void *)de + used_space;
-                                ext4_directory_write_entry(fs->superblock, new_entry,
-                                        free_space, child, name, name_size);
-
-                                // TODO cleanup
-                                target_block->dirty = true;
-                                rc = block_put(target_block);
-                                if (rc != EOK) {
-                                        return EXT4_ERR_BAD_DX_DIR;
-                                }
-                                return EOK;
-
-                        }
-
-                }
-
-                de = (void *)de + de_rec_len;
+        rc = ext4_directory_try_insert_entry(fs->superblock, target_block, child, name, name_len);
+        if (rc == EOK) {
+                goto cleanup;
+
         }
 
     EXT4FS_DBG("no free space found");
+
+    uint32_t block_size = ext4_superblock_get_block_size(fs->superblock);
 
         ext4_directory_dx_entry_t *entries = ((ext4_directory_dx_node_t *) dx_block->block->data)->entries;
@@ -786 +738 @@
                 rc =  ext4_directory_append_block(fs, parent, &new_fblock, &new_iblock);
                 if (rc != EOK) {
-                        // TODO error
+                        goto cleanup;
                 }
 
@@ -793 +745 @@
                 rc = block_get(&new_block, fs->device, new_fblock, BLOCK_FLAGS_NOREAD);
                 if (rc != EOK) {
-                        // TODO error
+                        goto cleanup;
                 }
 
@@ -873 +825 @@
         }
 
-        // TODO Where to save new entry
+        // Where to save new entry
         uint32_t new_block_hash = ext4_directory_dx_entry_get_hash(dx_block->position + 1);
         if (hinfo.hash >= new_block_hash) {
-                de = new_block->data;
-                stop = new_block->data + block_size;
+                rc = ext4_directory_try_insert_entry(fs->superblock, new_block, child, name, name_len);
         } else {
-                de = target_block->data;
-                stop = target_block->data + block_size;
-        }
-
-        while (de < stop) {
-
-                uint32_t de_inode = ext4_directory_entry_ll_get_inode(de);
-                uint16_t de_rec_len = ext4_directory_entry_ll_get_entry_length(de);
-
-                if ((de_inode == 0) && (de_rec_len >= required_len)) {
-                        ext4_directory_write_entry(fs->superblock, de, de_rec_len,
-                                child, name, name_size);
-                                goto success;
-                }
-
-                if (de_inode != 0) {
-                        uint16_t used_name_len = ext4_directory_entry_ll_get_name_length(
-                                        fs->superblock, de);
-
-                        uint16_t used_space = 8 + used_name_len;
-                        if ((used_name_len % 4) != 0) {
-                                used_space += 4 - (used_name_len % 4);
-                        }
-                        uint16_t free_space = de_rec_len - used_space;
-
-                        if (free_space >= required_len) {
-
-                                // Cut tail of current entry
-                                ext4_directory_entry_ll_set_entry_length(de, used_space);
-                                ext4_directory_entry_ll_t *new_entry = (void *)de + used_space;
-                                ext4_directory_write_entry(fs->superblock, new_entry,
-                                        free_space, child, name, name_size);
-
-                                goto success;
-                        }
-
-                }
-
-                de = (void *)de + de_rec_len;
-        }
-
-success:
+                rc = ext4_directory_try_insert_entry(fs->superblock, target_block, child, name, name_len);
+        }
+
+        if (rc != EOK) {
+                goto terminate;
+        }
+
+
+terminate:
+        rc = block_put(new_block);
+        if (rc != EOK) {
+                EXT4FS_DBG("error writing new block");
+        }
+
+
+cleanup:
+
+        rc2 = rc;
 
         rc = block_put(target_block);
         if (rc != EOK) {
-                EXT4FS_DBG("error writing target block");
-        }
-        rc = block_put(new_block);
-        if (rc != EOK) {
-                EXT4FS_DBG("error writing new block");
+                return rc;
         }
 
@@ -936 +859 @@
                 rc = block_put(dx_it->block);
                 if (rc != EOK) {
-                        EXT4FS_DBG("error writing index block \%u", (uint32_t)dx_it->block->pba);
+                        return rc;
                 }
                 dx_it++;
         }
 
-        return EOK;
+        return rc2;
 }