Changeset d510ac01 in mainline for uspace/lib/ext4/libext4_extent.c

Timestamp:

2012-06-23T19:53:49Z (12 years ago)

Author:

Frantisek Princ <frantisek.princ@…>

Branches:

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

Children:

02020dc

Parents:

380553c

Message:

Fixed bug with different sizes in writing file on extents

File:

• uspace/lib/ext4/libext4_extent.c (modified) (7 diffs)

uspace/lib/ext4/libext4_extent.c

@@ -742 +742 @@
                 uint32_t iblock)
 {
-        EXT4FS_DBG("iblock = \%u", iblock);
 
         int rc;
@@ -968 +967 @@
  */
 int ext4_extent_append_block(ext4_inode_ref_t *inode_ref,
-                uint32_t *iblock, uint32_t *fblock)
+                uint32_t *iblock, uint32_t *fblock, bool update_size)
 {
         int rc = EOK;
@@ -1026 +1025 @@
 
                         /* Update i-node */
-                        ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
-                        inode_ref->dirty = true;
+                        if (update_size) {
+                                ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
+                                inode_ref->dirty = true;
+                        }
 
                         path_ptr->block->dirty = true;
@@ -1056 +1057 @@
 
                         /* Update i-node */
-                        ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
-                        inode_ref->dirty = true;
+                        if (update_size) {
+                                ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
+                                inode_ref->dirty = true;
+                        }
 
                         path_ptr->block->dirty = true;
@@ -1067 +1070 @@
 /* Append new extent to the tree */
 append_extent:
-
-        /* Append extent for new block (includes tree splitting if needed) */
-        rc = ext4_extent_append_extent(inode_ref, path, &path_ptr, new_block_idx);
-        if (rc != EOK) {
-                goto finish;
-        }
 
         phys_block = 0;
@@ -1083 +1080 @@
         }
 
+        /* Append extent for new block (includes tree splitting if needed) */
+        rc = ext4_extent_append_extent(inode_ref, path, &path_ptr, new_block_idx);
+        if (rc != EOK) {
+                ext4_balloc_free_block(inode_ref, phys_block);
+                goto finish;
+        }
+
         /* Initialize newly created extent */
         ext4_extent_set_block_count(path_ptr->extent, 1);
@@ -1089 +1093 @@
 
         /* Update i-node */
-        ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
-        inode_ref->dirty = true;
+        if (update_size) {
+                ext4_inode_set_size(inode_ref->inode, inode_size + block_size);
+                inode_ref->dirty = true;
+        }
 
         path_ptr->block->dirty = true;