Changeset 12b4a7f in mainline for uspace/lib/ext4/libext4_filesystem.c

Timestamp:

2011-11-07T16:23:30Z (14 years ago)

Author:

Frantisek Princ <frantisek.princ@…>

Branches:

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

Children:

43a9968

Parents:

052e82d

Message:

functional truncate operation (only to lower size actually), except extent files

File:

• uspace/lib/ext4/libext4_filesystem.c (modified) (5 diffs)

uspace/lib/ext4/libext4_filesystem.c

@@ -152 +152 @@
         }
 
-        //EXT4FS_DBG("desc size = \%u", (uint32_t)ext4_superblock_get_desc_size(fs->superblock));
-
         descriptors_per_block = ext4_superblock_get_block_size(fs->superblock)
             / ext4_superblock_get_desc_size(fs->superblock);
@@ -171 +169 @@
 
         newref->block_group = newref->block->data + offset;
+        newref->dirty = false;
 
         *ref = newref;
@@ -180 +179 @@
 {
         int rc;
+
+        if (ref->dirty) {
+                ref->block->dirty = true;
+        }
 
         rc = block_put(ref->block);
@@ -375 +378 @@
         /* TODO handle extents */
 
+
         /* Handle simple case when we are dealing with direct reference */
         if (iblock < EXT4_INODE_DIRECT_BLOCK_COUNT) {
-                EXT4FS_DBG("direct block");
-                fblock = ext4_inode_get_direct_block(inode, (uint32_t)iblock);
+                fblock = ext4_inode_get_direct_block(inode, iblock);
                 // Sparse file
                 if (fblock == 0) {
@@ -386 +389 @@
                 ext4_inode_set_direct_block(inode, iblock, 0);
                 return ext4_bitmap_free_block(fs, fblock);
-        } else {
-
-                /* Determine the indirection level needed to get the desired block */
-                level = -1;
-                for (i = 1; i < 4; i++) {
-                        if (iblock < fs->inode_block_limits[i]) {
-                                level = i;
-                                break;
-                        }
-                }
-
-                if (level == -1) {
-                        return EIO;
-                }
-
-                /* Compute offsets for the topmost level */
-                block_offset_in_level = iblock - fs->inode_block_limits[level-1];
-                current_block = ext4_inode_get_indirect_block(inode, level-1);
+        }
+
+
+        /* Determine the indirection level needed to get the desired block */
+        level = -1;
+        for (i = 1; i < 4; i++) {
+                if (iblock < fs->inode_block_limits[i]) {
+                        level = i;
+                        break;
+                }
+        }
+
+        if (level == -1) {
+                return EIO;
+        }
+
+        /* Compute offsets for the topmost level */
+        block_offset_in_level = iblock - fs->inode_block_limits[level-1];
+        current_block = ext4_inode_get_indirect_block(inode, level-1);
+        offset_in_block = block_offset_in_level / fs->inode_blocks_per_level[level-1];
+
+        /* Navigate through other levels, until we find the block number
+         * or find null reference meaning we are dealing with sparse file
+         */
+        while (level > 0) {
+                rc = block_get(&block, fs->device, current_block, 0);
+                if (rc != EOK) {
+                        return rc;
+                }
+
+                current_block = uint32_t_le2host(((uint32_t*)block->data)[offset_in_block]);
+
+                // Set zero
+                if (level == 1) {
+                        ((uint32_t*)block->data)[offset_in_block] = host2uint32_t_le(0);
+                        block->dirty = true;
+                }
+
+                rc = block_put(block);
+                if (rc != EOK) {
+                        return rc;
+                }
+
+                level -= 1;
+
+                /* If we are on the last level, break here as
+                 * there is no next level to visit
+                 */
+                if (level == 0) {
+                        break;
+                }
+
+                /* Visit the next level */
+                block_offset_in_level %= fs->inode_blocks_per_level[level];
                 offset_in_block = block_offset_in_level / fs->inode_blocks_per_level[level-1];
-
-                /* Navigate through other levels, until we find the block number
-                 * or find null reference meaning we are dealing with sparse file
-                 */
-                while (level > 0) {
-                        rc = block_get(&block, fs->device, current_block, 0);
-                        if (rc != EOK) {
-                                return rc;
-                        }
-
-                        current_block = uint32_t_le2host(((uint32_t*)block->data)[offset_in_block]);
-
-                        // Set zero
-                        if (level == 1) {
-                                ((uint32_t*)block->data)[offset_in_block] = host2uint32_t_le(0);
-                                block->dirty = true;
-                        }
-
-                        rc = block_put(block);
-                        if (rc != EOK) {
-                                return rc;
-                        }
-
-                        level -= 1;
-
-                        /* If we are on the last level, break here as
-                         * there is no next level to visit
-                         */
-                        if (level == 0) {
-                                break;
-                        }
-
-                        /* Visit the next level */
-                        block_offset_in_level %= fs->inode_blocks_per_level[level];
-                        offset_in_block = block_offset_in_level / fs->inode_blocks_per_level[level-1];
-                }
-
-                fblock = current_block;
-
-                if (fblock == 0) {
-                        return EOK;
-                }
+        }
+
+        fblock = current_block;
+
+        if (fblock == 0) {
+                return EOK;
         }