Changeset 12b4a7f in mainline for uspace/srv/fs/ext4fs/ext4fs_ops.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/srv/fs/ext4fs/ext4fs_ops.c (modified) (4 diffs)

uspace/srv/fs/ext4fs/ext4fs_ops.c

@@ -954 +954 @@
         fs = enode->instance->filesystem;
 
+
+        if (! ext4_inode_can_truncate(inode_ref->inode)) {
+                // Unable to truncate
+                return EINVAL;
+        }
+
         old_size = ext4_inode_get_size(fs->superblock, inode_ref->inode);
-
-        printf("old size = \%llu, new size = \%llu\n", old_size, new_size);
 
         if (old_size == new_size) {
@@ -962 +966 @@
         } else {
 
-                //int rc;
                 uint32_t block_size;
                 uint32_t blocks_count, total_blocks;
@@ -970 +973 @@
 
                 if (old_size < new_size) {
-                        // TODO don't return immediately
-                        EXT4FS_DBG("expand the file");
+                        // Currently not supported to expand the file
+                        // TODO
+                        EXT4FS_DBG("trying to expand the file");
                         return EINVAL;
                 }
-
-                EXT4FS_DBG("cut the end of the file !");
 
                 size_diff = old_size - new_size;
@@ -990 +992 @@
                 inode_ref->dirty = true;
 
-                for (i = 0; i< blocks_count; ++i) {
+                // starting from 1 because of logical blocks are numbered from 0
+                for (i = 1; i <= blocks_count; ++i) {
                         // TODO check retval
+                        // TODO decrement inode->blocks_count
+
                         ext4_filesystem_release_inode_block(fs, inode_ref, total_blocks - i);
                 }