Numorous changes. See below:
[lwext4.git] / lwext4 / ext4.c
index 9ffffb39960b8fe2a8b8ae5bdf0c0337a867a6cf..1684edb0f3511f7bf2f880507f158432ea850dea 100644 (file)
@@ -35,6 +35,7 @@
  */
 
 #include "ext4_config.h"
+#include "ext4.h"
 #include "ext4_blockdev.h"
 #include "ext4_types.h"
 #include "ext4_debug.h"
@@ -45,7 +46,8 @@
 #include "ext4_super.h"
 #include "ext4_dir_idx.h"
 #include "ext4_xattr.h"
-#include "ext4.h"
+#include "ext4_journal.h"
+
 
 #include <stdlib.h>
 #include <string.h>
@@ -81,6 +83,9 @@ struct ext4_mountpoint {
 
        /**@brief   Dynamic allocation cache flag.*/
        bool cache_dynamic;
+
+       struct jbd_fs jbd_fs;
+       struct jbd_journal jbd_journal;
 };
 
 /**@brief   Block devices descriptor.*/
@@ -137,27 +142,26 @@ static bool ext4_is_dots(const uint8_t *name, size_t name_size)
 
 static int ext4_has_children(bool *has_children, struct ext4_inode_ref *enode)
 {
-       struct ext4_fs *fs = enode->fs;
+       struct ext4_sblock *sb = &enode->fs->sb;
 
        /* Check if node is directory */
-       if (!ext4_inode_is_type(&fs->sb, enode->inode,
-                               EXT4_INODE_MODE_DIRECTORY)) {
+       if (!ext4_inode_is_type(sb, enode->inode, EXT4_INODE_MODE_DIRECTORY)) {
                *has_children = false;
                return EOK;
        }
 
-       struct ext4_directory_iterator it;
+       struct ext4_dir_iter it;
        int rc = ext4_dir_iterator_init(&it, enode, 0);
        if (rc != EOK)
                return rc;
 
        /* Find a non-empty directory entry */
        bool found = false;
-       while (it.current != NULL) {
-               if (ext4_dir_entry_ll_get_inode(it.current) != 0) {
-                       uint16_t name_size = ext4_dir_entry_ll_get_name_length(
-                           &fs->sb, it.current);
-                       if (!ext4_is_dots(it.current->name, name_size)) {
+       while (it.curr != NULL) {
+               if (ext4_dir_en_get_inode(it.curr) != 0) {
+                       uint16_t nsize;
+                       nsize = ext4_dir_en_get_name_len(sb, it.curr);
+                       if (!ext4_is_dots(it.curr->name, nsize)) {
                                found = true;
                                break;
                        }
@@ -180,111 +184,105 @@ static int ext4_has_children(bool *has_children, struct ext4_inode_ref *enode)
 }
 
 static int ext4_link(struct ext4_mountpoint *mp, struct ext4_inode_ref *parent,
-                    struct ext4_inode_ref *child, const char *name,
-                    uint32_t name_len, bool rename)
+                    struct ext4_inode_ref *ch, const char *n,
+                    uint32_t len, bool rename)
 {
        /* Check maximum name length */
-       if (name_len > EXT4_DIRECTORY_FILENAME_LEN)
+       if (len > EXT4_DIRECTORY_FILENAME_LEN)
                return EINVAL;
 
        /* Add entry to parent directory */
-       int rc = ext4_dir_add_entry(parent, name, name_len, child);
-       if (rc != EOK)
-               return rc;
+       int r = ext4_dir_add_entry(parent, n, len, ch);
+       if (r != EOK)
+               return r;
 
        /* Fill new dir -> add '.' and '..' entries.
         * Also newly allocated inode should have 0 link count.
         */
-       if (ext4_inode_is_type(&mp->fs.sb, child->inode,
-                              EXT4_INODE_MODE_DIRECTORY) &&
-           !rename) {
-               rc = ext4_dir_add_entry(child, ".", strlen("."), child);
-               if (rc != EOK) {
-                       ext4_dir_remove_entry(parent, name, strlen(name));
-                       return rc;
-               }
-
-               rc = ext4_dir_add_entry(child, "..", strlen(".."), parent);
-               if (rc != EOK) {
-                       ext4_dir_remove_entry(parent, name, strlen(name));
-                       ext4_dir_remove_entry(child, ".", strlen("."));
-                       return rc;
-               }
 
-               /*New empty directory. Two links (. and ..) */
-               ext4_inode_set_links_count(child->inode, 2);
+       bool is_dir = ext4_inode_is_type(&mp->fs.sb, ch->inode,
+                              EXT4_INODE_MODE_DIRECTORY);
+       if (is_dir && !rename) {
 
 #if CONFIG_DIR_INDEX_ENABLE
                /* Initialize directory index if supported */
-               if (ext4_sb_has_feature_compatible(
-                       &mp->fs.sb, EXT4_FEATURE_COMPAT_DIR_INDEX)) {
-                       rc = ext4_dir_dx_init(child);
-                       if (rc != EOK)
-                               return rc;
-
-                       ext4_inode_set_flag(child->inode,
-                                           EXT4_INODE_FLAG_INDEX);
-                       child->dirty = true;
-               }
+               if (ext4_sb_feature_com(&mp->fs.sb, EXT4_FCOM_DIR_INDEX)) {
+                       r = ext4_dir_dx_init(ch, parent);
+                       if (r != EOK)
+                               return r;
+
+                       ext4_inode_set_flag(ch->inode, EXT4_INODE_FLAG_INDEX);
+                       ch->dirty = true;
+               } else
 #endif
+               {
+                       r = ext4_dir_add_entry(ch, ".", strlen("."), ch);
+                       if (r != EOK) {
+                               ext4_dir_remove_entry(parent, n, strlen(n));
+                               return r;
+                       }
 
+                       r = ext4_dir_add_entry(ch, "..", strlen(".."), parent);
+                       if (r != EOK) {
+                               ext4_dir_remove_entry(parent, n, strlen(n));
+                               ext4_dir_remove_entry(ch, ".", strlen("."));
+                               return r;
+                       }
+               }
+
+               /*New empty directory. Two links (. and ..) */
+               ext4_inode_set_links_cnt(ch->inode, 2);
                ext4_fs_inode_links_count_inc(parent);
-               child->dirty = true;
+               ch->dirty = true;
                parent->dirty = true;
-       } else {
-               /*
-                * In case we want to rename a directory,
-                * we reset the original '..' pointer.
-                */
-               if (ext4_inode_is_type(&mp->fs.sb, child->inode,
-                                       EXT4_INODE_MODE_DIRECTORY)) {
-                       int has_flag_index =
-                               ext4_inode_has_flag(child->inode,
-                                                   EXT4_INODE_FLAG_INDEX);
-                       struct ext4_directory_search_result result;
-                       if (!has_flag_index) {
-                               rc = ext4_dir_find_entry(&result,
-                                                        child, "..",
-                                                        strlen(".."));
-                               if (rc != EOK)
-                                       return EIO;
-
-                               ext4_dir_entry_ll_set_inode(result.dentry,
-                                                           parent->index);
-                               result.block.dirty = true;
-                               rc = ext4_dir_destroy_result(child, &result);
-                               if (rc != EOK)
-                                       return rc;
+               return r;
+       }
+       /*
+        * In case we want to rename a directory,
+        * we reset the original '..' pointer.
+        */
+       if (is_dir) {
+               bool idx;
+               idx = ext4_inode_has_flag(ch->inode, EXT4_INODE_FLAG_INDEX);
+               struct ext4_dir_search_result res;
+               if (!idx) {
+                       r = ext4_dir_find_entry(&res, ch, "..", strlen(".."));
+                       if (r != EOK)
+                               return EIO;
 
-                       } else {
+                       ext4_dir_en_set_inode(res.dentry, parent->index);
+                       ext4_trans_set_block_dirty(res.block.buf);
+                       r = ext4_dir_destroy_result(ch, &res);
+                       if (r != EOK)
+                               return r;
+
+               } else {
 #if CONFIG_DIR_INDEX_ENABLE
-                               rc = ext4_dir_dx_reset_parent_inode(child,
-                                               parent->index);
-                               if (rc != EOK)
-                                       return rc;
+                       r = ext4_dir_dx_reset_parent_inode(ch, parent->index);
+                       if (r != EOK)
+                               return r;
 
 #endif
-                       }
-
-                       ext4_fs_inode_links_count_inc(parent);
-                       parent->dirty = true;
-               }
-               if (!rename) {
-                       ext4_fs_inode_links_count_inc(child);
-                       child->dirty = true;
                }
+
+               ext4_fs_inode_links_count_inc(parent);
+               parent->dirty = true;
+       }
+       if (!rename) {
+               ext4_fs_inode_links_count_inc(ch);
+               ch->dirty = true;
        }
 
-       return rc;
+       return r;
 }
 
 static int ext4_unlink(struct ext4_mountpoint *mp,
                       struct ext4_inode_ref *parent,
-                      struct ext4_inode_ref *child_inode_ref, const char *name,
+                      struct ext4_inode_ref *child, const char *name,
                       uint32_t name_len)
 {
        bool has_children;
-       int rc = ext4_has_children(&has_children, child_inode_ref);
+       int rc = ext4_has_children(&has_children, child);
        if (rc != EOK)
                return rc;
 
@@ -297,13 +295,11 @@ static int ext4_unlink(struct ext4_mountpoint *mp,
        if (rc != EOK)
                return rc;
 
-       bool is_dir = ext4_inode_is_type(&mp->fs.sb, child_inode_ref->inode,
+       bool is_dir = ext4_inode_is_type(&mp->fs.sb, child->inode,
                                         EXT4_INODE_MODE_DIRECTORY);
 
        /* If directory - handle links from parent */
        if (is_dir) {
-               // ext4_assert(ext4_inode_get_links_count(child_inode_ref->inode)
-               // == 1);
                ext4_fs_inode_links_count_dec(parent);
                parent->dirty = true;
        }
@@ -320,12 +316,12 @@ static int ext4_unlink(struct ext4_mountpoint *mp,
        /*
         * TODO: Update timestamp for inode.
         *
-        * ext4_inode_set_change_inode_time(child_inode_ref->inode,
+        * ext4_inode_set_change_inode_time(child->inode,
         *     (uint32_t) now);
         */
-       if (ext4_inode_get_links_count(child_inode_ref->inode)) {
-               ext4_fs_inode_links_count_dec(child_inode_ref);
-               child_inode_ref->dirty = true;
+       if (ext4_inode_get_links_cnt(child->inode)) {
+               ext4_fs_inode_links_count_dec(child);
+               child->dirty = true;
        }
 
        return EOK;
@@ -410,6 +406,7 @@ int ext4_mount(const char *dev_name, const char *mount_point)
        /*Bind block cache to block device*/
        r = ext4_block_bind_bcache(bd, bc);
        if (r != EOK) {
+               ext4_bcache_cleanup(bc);
                ext4_block_fini(bd);
                if (mp->cache_dynamic) {
                        ext4_bcache_fini_dynamic(bc);
@@ -417,10 +414,12 @@ int ext4_mount(const char *dev_name, const char *mount_point)
                }
                return r;
        }
+       bd->fs = &mp->fs;
 
        return r;
 }
 
+
 int ext4_umount(const char *mount_point)
 {
        int i;
@@ -439,16 +438,19 @@ int ext4_umount(const char *mount_point)
 
        r = ext4_fs_fini(&mp->fs);
        if (r != EOK)
-               return r;
+               goto Finish;
 
        mp->mounted = 0;
 
+       ext4_bcache_cleanup(mp->fs.bdev->bc);
        if (mp->cache_dynamic) {
                ext4_bcache_fini_dynamic(mp->fs.bdev->bc);
                free(mp->fs.bdev->bc);
        }
-
-       return ext4_block_fini(mp->fs.bdev);
+       r = ext4_block_fini(mp->fs.bdev);
+Finish:
+       mp->fs.bdev->fs = NULL;
+       return r;
 }
 
 static struct ext4_mountpoint *ext4_get_mount(const char *path)
@@ -465,6 +467,136 @@ static struct ext4_mountpoint *ext4_get_mount(const char *path)
        return 0;
 }
 
+int ext4_journal_start(const char *mount_point)
+{
+       int r = EOK;
+       struct ext4_mountpoint *mp = ext4_get_mount(mount_point);
+       if (!mp)
+               return ENOENT;
+
+       if (ext4_sb_feature_com(&mp->fs.sb,
+                               EXT4_FCOM_HAS_JOURNAL)) {
+               r = jbd_get_fs(&mp->fs, &mp->jbd_fs);
+               if (r != EOK)
+                       goto Finish;
+
+               r = jbd_journal_start(&mp->jbd_fs, &mp->jbd_journal);
+               if (r != EOK) {
+                       mp->jbd_fs.dirty = false;
+                       jbd_put_fs(&mp->jbd_fs);
+                       goto Finish;
+               }
+               mp->fs.jbd_fs = &mp->jbd_fs;
+               mp->fs.jbd_journal = &mp->jbd_journal;
+       }
+Finish:
+       return r;
+}
+
+int ext4_journal_stop(const char *mount_point)
+{
+       int r = EOK;
+       struct ext4_mountpoint *mp = ext4_get_mount(mount_point);
+       if (!mp)
+               return ENOENT;
+
+       if (ext4_sb_feature_com(&mp->fs.sb,
+                               EXT4_FCOM_HAS_JOURNAL)) {
+               r = jbd_journal_stop(&mp->jbd_journal);
+               if (r != EOK) {
+                       mp->jbd_fs.dirty = false;
+                       jbd_put_fs(&mp->jbd_fs);
+                       mp->fs.jbd_journal = NULL;
+                       mp->fs.jbd_fs = NULL;
+                       goto Finish;
+               }
+
+               r = jbd_put_fs(&mp->jbd_fs);
+               if (r != EOK) {
+                       mp->fs.jbd_journal = NULL;
+                       mp->fs.jbd_fs = NULL;
+                       goto Finish;
+               }
+
+               mp->fs.jbd_journal = NULL;
+               mp->fs.jbd_fs = NULL;
+       }
+Finish:
+       return r;
+}
+
+int ext4_recover(const char *mount_point)
+{
+       struct ext4_mountpoint *mp = ext4_get_mount(mount_point);
+       if (!mp)
+               return ENOENT;
+
+       int r = ENOTSUP;
+       EXT4_MP_LOCK(mp);
+       if (ext4_sb_feature_com(&mp->fs.sb, EXT4_FCOM_HAS_JOURNAL)) {
+               struct jbd_fs *jbd_fs = calloc(1, sizeof(struct jbd_fs));
+               if (!jbd_fs) {
+                        r = ENOMEM;
+                        goto Finish;
+               }
+
+
+               r = jbd_get_fs(&mp->fs, jbd_fs);
+               if (r != EOK) {
+                       free(jbd_fs);
+                       goto Finish;
+               }
+
+               r = jbd_recover(jbd_fs);
+               jbd_put_fs(jbd_fs);
+               free(jbd_fs);
+       }
+
+
+Finish:
+       EXT4_MP_UNLOCK(mp);
+       return r;
+}
+
+static int ext4_trans_start(struct ext4_mountpoint *mp)
+{
+       int r = EOK;
+       if (mp->fs.jbd_journal && !mp->fs.curr_trans) {
+               struct jbd_journal *journal = mp->fs.jbd_journal;
+               struct jbd_trans *trans;
+               trans = jbd_journal_new_trans(journal);
+               if (!trans) {
+                       r = ENOMEM;
+                       goto Finish;
+               }
+               mp->fs.curr_trans = trans;
+       }
+Finish:
+       return r;
+}
+
+static int ext4_trans_stop(struct ext4_mountpoint *mp)
+{
+       int r = EOK;
+       if (mp->fs.jbd_journal && mp->fs.curr_trans) {
+               struct jbd_journal *journal = mp->fs.jbd_journal;
+               struct jbd_trans *trans = mp->fs.curr_trans;
+               r = jbd_journal_commit_trans(journal, trans);
+               mp->fs.curr_trans = NULL;
+       }
+       return r;
+}
+
+static void ext4_trans_abort(struct ext4_mountpoint *mp)
+{
+       if (mp->fs.jbd_journal && mp->fs.curr_trans) {
+               struct jbd_journal *journal = mp->fs.jbd_journal;
+               struct jbd_trans *trans = mp->fs.curr_trans;
+               jbd_journal_free_trans(journal, trans, true);
+               mp->fs.curr_trans = NULL;
+       }
+}
+
 int ext4_mount_point_stats(const char *mount_point,
                           struct ext4_mount_stats *stats)
 {
@@ -572,21 +704,93 @@ static bool ext4_parse_flags(const char *flags, uint32_t *file_flags)
        return false;
 }
 
+static int ext4_trunc_inode(struct ext4_mountpoint *mp,
+                           uint32_t index, uint64_t new_size)
+{
+       int r = EOK;
+       struct ext4_fs *const fs = &mp->fs;
+       struct ext4_inode_ref inode_ref;
+       uint64_t inode_size;
+       bool has_trans = mp->fs.jbd_journal && mp->fs.curr_trans;
+       r = ext4_fs_get_inode_ref(fs, index, &inode_ref);
+       if (r != EOK)
+               return r;
+
+       inode_size = ext4_inode_get_size(&fs->sb, inode_ref.inode);
+       ext4_fs_put_inode_ref(&inode_ref);
+       if (has_trans)
+               ext4_trans_stop(mp);
+
+       while (inode_size > new_size + CONFIG_MAX_TRUNCATE_SIZE) {
+
+               inode_size -= CONFIG_MAX_TRUNCATE_SIZE;
+
+               ext4_trans_start(mp);
+               r = ext4_fs_get_inode_ref(fs, index, &inode_ref);
+               if (r != EOK) {
+                       ext4_trans_abort(mp);
+                       break;
+               }
+               r = ext4_fs_truncate_inode(&inode_ref, inode_size);
+               if (r != EOK)
+                       ext4_fs_put_inode_ref(&inode_ref);
+               else
+                       r = ext4_fs_put_inode_ref(&inode_ref);
+
+               if (r != EOK) {
+                       ext4_trans_abort(mp);
+                       goto Finish;
+               } else
+                       ext4_trans_stop(mp);
+       }
+
+       if (inode_size > new_size) {
+
+               inode_size = new_size;
+
+               ext4_trans_start(mp);
+               r = ext4_fs_get_inode_ref(fs, index, &inode_ref);
+               if (r != EOK) {
+                       ext4_trans_abort(mp);
+                       goto Finish;
+               }
+               r = ext4_fs_truncate_inode(&inode_ref, inode_size);
+               if (r != EOK)
+                       ext4_fs_put_inode_ref(&inode_ref);
+               else
+                       r = ext4_fs_put_inode_ref(&inode_ref);
+
+               if (r != EOK)
+                       ext4_trans_abort(mp);
+               else
+                       ext4_trans_stop(mp);
+
+       }
+
+Finish:
+
+       if (has_trans)
+               ext4_trans_start(mp);
+
+       return r;
+}
+
 /*
  * NOTICE: if filetype is equal to EXT4_DIRENTRY_UNKNOWN,
  * any filetype of the target dir entry will be accepted.
  */
 static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
-                             int filetype, uint32_t *parent_inode,
+                             int ftype, uint32_t *parent_inode,
                              uint32_t *name_off)
 {
        bool is_goal = false;
-       uint8_t inode_type = EXT4_DIRENTRY_UNKNOWN;
+       uint32_t imode = EXT4_INODE_MODE_DIRECTORY;
        uint32_t next_inode;
 
        int r;
+       int len;
        struct ext4_mountpoint *mp = ext4_get_mount(path);
-       struct ext4_directory_search_result result;
+       struct ext4_dir_search_result result;
        struct ext4_inode_ref ref;
 
        f->mp = 0;
@@ -594,6 +798,9 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
        if (!mp)
                return ENOENT;
 
+       struct ext4_fs *const fs = &mp->fs;
+       struct ext4_sblock *const sb = &mp->fs.sb;
+
        f->flags = flags;
 
        /*Skip mount point*/
@@ -603,26 +810,25 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
                *name_off = strlen(mp->name);
 
        /*Load root*/
-       r = ext4_fs_get_inode_ref(&mp->fs, EXT4_INODE_ROOT_INDEX, &ref);
-
+       r = ext4_fs_get_inode_ref(fs, EXT4_INODE_ROOT_INDEX, &ref);
        if (r != EOK)
                return r;
 
        if (parent_inode)
                *parent_inode = ref.index;
 
-       int len = ext4_path_check(path, &is_goal);
+       if (flags & O_CREAT)
+               ext4_trans_start(mp);
 
+       len = ext4_path_check(path, &is_goal);
        while (1) {
 
                len = ext4_path_check(path, &is_goal);
-
                if (!len) {
                        /*If root open was request.*/
-                       if (is_goal &&
-                           ((filetype == EXT4_DIRENTRY_DIR) ||
-                            (filetype == EXT4_DIRENTRY_UNKNOWN)))
-                               break;
+                       if (ftype == EXT4_DE_DIR || ftype == EXT4_DE_UNKNOWN)
+                               if (is_goal)
+                                       break;
 
                        r = ENOENT;
                        break;
@@ -633,7 +839,6 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
 
                        /*Destroy last result*/
                        ext4_dir_destroy_result(&ref, &result);
-
                        if (r != ENOENT)
                                break;
 
@@ -642,10 +847,8 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
 
                        /*O_CREAT allows create new entry*/
                        struct ext4_inode_ref child_ref;
-                       r = ext4_fs_alloc_inode(
-                           &mp->fs, &child_ref,
-                           is_goal ? filetype
-                                   : EXT4_DIRENTRY_DIR);
+                       r = ext4_fs_alloc_inode(fs, &child_ref,
+                                       is_goal ? ftype : EXT4_DE_DIR);
                        if (r != EOK)
                                break;
 
@@ -663,28 +866,39 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
                        }
 
                        ext4_fs_put_inode_ref(&child_ref);
-
                        continue;
                }
 
                if (parent_inode)
                        *parent_inode = ref.index;
 
-               next_inode = ext4_dir_entry_ll_get_inode(result.dentry);
-               inode_type =
-                   ext4_dir_entry_ll_get_inode_type(&mp->fs.sb, result.dentry);
+               next_inode = ext4_dir_en_get_inode(result.dentry);
+               if (ext4_sb_feature_incom(sb, EXT4_FINCOM_FILETYPE)) {
+                       uint8_t t;
+                       t = ext4_dir_en_get_inode_type(sb, result.dentry);
+                       imode = ext4_fs_correspond_inode_mode(t);
+               } else {
+                       struct ext4_inode_ref child_ref;
+                       r = ext4_fs_get_inode_ref(fs, next_inode, &child_ref);
+                       if (r != EOK)
+                               break;
+
+                       imode = ext4_inode_type(sb, child_ref.inode);
+                       ext4_fs_put_inode_ref(&child_ref);
+               }
 
                r = ext4_dir_destroy_result(&ref, &result);
                if (r != EOK)
                        break;
 
                /*If expected file error*/
-               if (inode_type != EXT4_DIRENTRY_DIR && !is_goal) {
+               if (imode != EXT4_INODE_MODE_DIRECTORY && !is_goal) {
                        r = ENOENT;
                        break;
                }
-               if (filetype != EXT4_DIRENTRY_UNKNOWN) {
-                       if ((inode_type != filetype) && is_goal) {
+               if (ftype != EXT4_DE_UNKNOWN) {
+                       bool df = imode != ext4_fs_correspond_inode_mode(ftype);
+                       if (df && is_goal) {
                                r = ENOENT;
                                break;
                        }
@@ -694,7 +908,7 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
                if (r != EOK)
                        break;
 
-               r = ext4_fs_get_inode_ref(&mp->fs, next_inode, &ref);
+               r = ext4_fs_get_inode_ref(fs, next_inode, &ref);
                if (r != EOK)
                        break;
 
@@ -714,10 +928,8 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
 
        if (is_goal) {
 
-               if ((f->flags & O_TRUNC) &&
-                   (inode_type == EXT4_DIRENTRY_REG_FILE)) {
-
-                       r = ext4_fs_truncate_inode(&ref, 0);
+               if ((f->flags & O_TRUNC) && (imode == EXT4_INODE_MODE_FILE)) {
+                       r = ext4_trunc_inode(mp, ref.index, 0);
                        if (r != EOK) {
                                ext4_fs_put_inode_ref(&ref);
                                return r;
@@ -725,37 +937,24 @@ static int ext4_generic_open2(ext4_file *f, const char *path, int flags,
                }
 
                f->mp = mp;
-               f->fsize = ext4_inode_get_size(&f->mp->fs.sb, ref.inode);
+               f->fsize = ext4_inode_get_size(sb, ref.inode);
                f->inode = ref.index;
                f->fpos = 0;
 
                if (f->flags & O_APPEND)
                        f->fpos = f->fsize;
 
-               /* FIXME: Debugging code on EA. */
-               {
-                       int private_ret;
-                       struct ext4_xattr_ref xattr_ref;
-                       private_ret = ext4_fs_get_xattr_ref(&f->mp->fs, &ref,
-                                                         &xattr_ref);
-                       if (private_ret == EOK) {
-#define EXT4_XATTR_TEST_DATA_SIZE 20
-#define EXT4_XATTR_TEST_NAME "bad_boy"
-                               static char test_data[EXT4_XATTR_TEST_DATA_SIZE] = {'a'};
-                               ext4_dmask_set(EXT4_DEBUG_ALL);
-                               ext4_fs_set_xattr(&xattr_ref,
-                                                 EXT4_XATTR_INDEX_USER,
-                                                 EXT4_XATTR_TEST_NAME,
-                                                 strlen(EXT4_XATTR_TEST_NAME),
-                                                 test_data,
-                                                 EXT4_XATTR_TEST_DATA_SIZE,
-                                                 0);
-                               ext4_fs_put_xattr_ref(&xattr_ref);
-                       }
-               }
        }
 
        r = ext4_fs_put_inode_ref(&ref);
+       if (flags & O_CREAT) {
+               if (r == EOK)
+                       ext4_trans_stop(mp);
+               else
+                       ext4_trans_abort(mp);
+
+       }
+
        return r;
 }
 
@@ -771,51 +970,48 @@ static int ext4_generic_open(ext4_file *f, const char *path, const char *flags,
                return EINVAL;
 
        if (file_expect == true)
-               filetype = EXT4_DIRENTRY_REG_FILE;
+               filetype = EXT4_DE_REG_FILE;
        else
-               filetype = EXT4_DIRENTRY_DIR;
+               filetype = EXT4_DE_DIR;
 
        return ext4_generic_open2(f, path, iflags, filetype, parent_inode,
                                  name_off);
 }
 
-static int __ext4_create_hardlink(const char *path,
-               struct ext4_inode_ref *child_ref,
-               bool rename)
+static int ext4_create_hardlink(const char *path,
+               struct ext4_inode_ref *child_ref, bool rename)
 {
        bool is_goal = false;
-       uint8_t inode_type = EXT4_DIRENTRY_DIR;
+       uint32_t inode_mode = EXT4_INODE_MODE_DIRECTORY;
        uint32_t next_inode;
 
        int r;
+       int len;
        struct ext4_mountpoint *mp = ext4_get_mount(path);
-       struct ext4_directory_search_result result;
+       struct ext4_dir_search_result result;
        struct ext4_inode_ref ref;
 
        if (!mp)
                return ENOENT;
 
+       struct ext4_fs *const fs = &mp->fs;
+       struct ext4_sblock *const sb = &mp->fs.sb;
+
        /*Skip mount point*/
        path += strlen(mp->name);
 
        /*Load root*/
-       r = ext4_fs_get_inode_ref(&mp->fs, EXT4_INODE_ROOT_INDEX, &ref);
-
+       r = ext4_fs_get_inode_ref(fs, EXT4_INODE_ROOT_INDEX, &ref);
        if (r != EOK)
                return r;
 
-       int len = ext4_path_check(path, &is_goal);
-
+       len = ext4_path_check(path, &is_goal);
        while (1) {
 
                len = ext4_path_check(path, &is_goal);
-
                if (!len) {
                        /*If root open was request.*/
-                       if (is_goal)
-                               r = EINVAL;
-                       else
-                               r = ENOENT;
+                       r = is_goal ? EINVAL : ENOENT;
                        break;
                }
 
@@ -839,19 +1035,26 @@ static int __ext4_create_hardlink(const char *path,
                }
 
                next_inode = result.dentry->inode;
-               inode_type =
-                       ext4_dir_entry_ll_get_inode_type(&mp->fs.sb, result.dentry);
+               if (ext4_sb_feature_incom(sb, EXT4_FINCOM_FILETYPE)) {
+                       uint8_t t;
+                       t = ext4_dir_en_get_inode_type(sb, result.dentry);
+                       inode_mode = ext4_fs_correspond_inode_mode(t);
+               } else {
+                       struct ext4_inode_ref child_ref;
+                       r = ext4_fs_get_inode_ref(fs, next_inode, &child_ref);
+                       if (r != EOK)
+                               break;
+
+                       inode_mode = ext4_inode_type(sb, child_ref.inode);
+                       ext4_fs_put_inode_ref(&child_ref);
+               }
 
                r = ext4_dir_destroy_result(&ref, &result);
                if (r != EOK)
                        break;
 
-               if (inode_type == EXT4_DIRENTRY_REG_FILE) {
-                       if (is_goal)
-                               r = EEXIST;
-                       else
-                               r = ENOENT;
-
+               if (inode_mode != EXT4_INODE_MODE_DIRECTORY) {
+                       r = is_goal ? EEXIST : ENOENT;
                        break;
                }
 
@@ -859,7 +1062,7 @@ static int __ext4_create_hardlink(const char *path,
                if (r != EOK)
                        break;
 
-               r = ext4_fs_get_inode_ref(&mp->fs, next_inode, &ref);
+               r = ext4_fs_get_inode_ref(fs, next_inode, &ref);
                if (r != EOK)
                        break;
 
@@ -878,8 +1081,7 @@ static int __ext4_create_hardlink(const char *path,
        return r;
 }
 
-static int ext4_remove_orig_reference(const char *path,
-                                     uint32_t name_off,
+static int ext4_remove_orig_reference(const char *path, uint32_t name_off,
                                      struct ext4_inode_ref *parent_ref,
                                      struct ext4_inode_ref *child_ref)
 {
@@ -929,9 +1131,9 @@ int ext4_flink(const char *path, const char *hardlink_path)
                return EINVAL;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDONLY,
-                              EXT4_DIRENTRY_UNKNOWN,
+       r = ext4_generic_open2(&f, path, O_RDONLY, EXT4_DE_UNKNOWN,
                               &parent_inode, &name_off);
        if (r != EOK)
                goto Finish;
@@ -953,12 +1155,17 @@ int ext4_flink(const char *path, const char *hardlink_path)
                goto Finish;
        }
 
-       r = __ext4_create_hardlink(hardlink_path, &child_ref, false);
+       r = ext4_create_hardlink(hardlink_path, &child_ref, false);
 
 Finish:
        if (child_loaded)
                ext4_fs_put_inode_ref(&child_ref);
 
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 
@@ -978,10 +1185,10 @@ int ext4_frename(const char *path, const char *new_path)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDONLY,
-                       EXT4_DIRENTRY_UNKNOWN,
-                       &parent_inode, &name_off);
+       r = ext4_generic_open2(&f, path, O_RDONLY, EXT4_DE_UNKNOWN,
+                               &parent_inode, &name_off);
        if (r != EOK)
                goto Finish;
 
@@ -1002,12 +1209,11 @@ int ext4_frename(const char *path, const char *new_path)
 
        child_loaded = true;
 
-       r = __ext4_create_hardlink(new_path, &child_ref, true);
+       r = ext4_create_hardlink(new_path, &child_ref, true);
        if (r != EOK)
                goto Finish;
 
-       r = ext4_remove_orig_reference(path, name_off,
-                                      &parent_ref, &child_ref);
+       r = ext4_remove_orig_reference(path, name_off, &parent_ref, &child_ref);
        if (r != EOK)
                goto Finish;
 
@@ -1018,6 +1224,11 @@ Finish:
        if (child_loaded)
                ext4_fs_put_inode_ref(&child_ref);
 
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 
@@ -1065,10 +1276,12 @@ int ext4_fremove(const char *path)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
-       r = ext4_generic_open2(&f, path, O_RDWR,
-                              EXT4_DIRENTRY_UNKNOWN,
+       ext4_trans_start(mp);
+
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN,
                               &parent_inode, &name_off);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1076,6 +1289,7 @@ int ext4_fremove(const char *path)
        /*Load parent*/
        r = ext4_fs_get_inode_ref(&mp->fs, parent_inode, &parent);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1084,10 +1298,34 @@ int ext4_fremove(const char *path)
        r = ext4_fs_get_inode_ref(&mp->fs, f.inode, &child);
        if (r != EOK) {
                ext4_fs_put_inode_ref(&parent);
+               ext4_trans_abort(mp);
+               EXT4_MP_UNLOCK(mp);
+               return r;
+       }
+       /* We do not allow opening files here. */
+       if (ext4_inode_type(&mp->fs.sb, child.inode) ==
+           EXT4_INODE_MODE_DIRECTORY) {
+               ext4_fs_put_inode_ref(&parent);
+               ext4_fs_put_inode_ref(&child);
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
 
+       /*Link count will be zero, the inode should be freed. */
+       if (ext4_inode_get_links_cnt(child.inode) == 1) {
+               ext4_block_cache_write_back(mp->fs.bdev, 1);
+               r = ext4_trunc_inode(mp, child.index, 0);
+               if (r != EOK) {
+                       ext4_fs_put_inode_ref(&parent);
+                       ext4_fs_put_inode_ref(&child);
+                       ext4_trans_abort(mp);
+                       EXT4_MP_UNLOCK(mp);
+                       return r;
+               }
+               ext4_block_cache_write_back(mp->fs.bdev, 0);
+       }
+
        /*Set path*/
        path += name_off;
 
@@ -1099,16 +1337,8 @@ int ext4_fremove(const char *path)
                goto Finish;
 
        /*Link count is zero, the inode should be freed. */
-       if (!ext4_inode_get_links_count(child.inode)) {
-               ext4_inode_set_deletion_time(child.inode, 0xFFFFFFFF);
-               /*Turncate*/
-               ext4_block_cache_write_back(mp->fs.bdev, 1);
-               /*Truncate may be IO heavy. Do it writeback cache mode.*/
-               r = ext4_fs_truncate_inode(&child, 0);
-               ext4_block_cache_write_back(mp->fs.bdev, 0);
-
-               if (r != EOK)
-                       goto Finish;
+       if (!ext4_inode_get_links_cnt(child.inode)) {
+               ext4_inode_set_del_time(child.inode, -1L);
 
                r = ext4_fs_free_inode(&child);
                if (r != EOK)
@@ -1118,12 +1348,17 @@ int ext4_fremove(const char *path)
 Finish:
        ext4_fs_put_inode_ref(&child);
        ext4_fs_put_inode_ref(&parent);
+
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
 
-int ext4_fill_raw_inode(const char *path,
-                       uint32_t *ret_ino,
+int ext4_fill_raw_inode(const char *path, uint32_t *ret_ino,
                        struct ext4_inode *inode)
 {
        int r;
@@ -1137,9 +1372,7 @@ int ext4_fill_raw_inode(const char *path,
 
        EXT4_MP_LOCK(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDONLY,
-                              EXT4_DIRENTRY_UNKNOWN,
-                              NULL, NULL);
+       r = ext4_generic_open2(&f, path, O_RDONLY, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
                EXT4_MP_UNLOCK(mp);
                return r;
@@ -1156,7 +1389,6 @@ int ext4_fill_raw_inode(const char *path,
        }
 
        memcpy(inode, inode_ref.inode, sizeof(struct ext4_inode));
-
        ext4_fs_put_inode_ref(&inode_ref);
        EXT4_MP_UNLOCK(mp);
 
@@ -1175,9 +1407,11 @@ int ext4_fopen(ext4_file *f, const char *path, const char *flags)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+
        ext4_block_cache_write_back(mp->fs.bdev, 1);
        r = ext4_generic_open(f, path, flags, true, 0, 0);
        ext4_block_cache_write_back(mp->fs.bdev, 0);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1191,12 +1425,14 @@ int ext4_fopen2(ext4_file *f, const char *path, int flags)
        if (!mp)
                return ENOENT;
 
-        filetype = EXT4_DIRENTRY_REG_FILE;
+        filetype = EXT4_DE_REG_FILE;
 
        EXT4_MP_LOCK(mp);
+
        ext4_block_cache_write_back(mp->fs.bdev, 1);
-       r = ext4_generic_open2(f, path, flags, filetype, 0, 0);
+       r = ext4_generic_open2(f, path, flags, filetype, NULL, NULL);
        ext4_block_cache_write_back(mp->fs.bdev, 0);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1237,7 +1473,7 @@ static int ext4_ftruncate_no_lock(ext4_file *f, uint64_t size)
        if (r != EOK)
                goto Finish;
 
-       r = ext4_fs_truncate_inode(&ref, size);
+       r = ext4_trunc_inode(f->mp, ref.index, size);
        if (r != EOK)
                goto Finish;
 
@@ -1267,7 +1503,12 @@ int ext4_ftruncate(ext4_file *f, uint64_t size)
 
        EXT4_MP_LOCK(f->mp);
 
+       ext4_trans_start(f->mp);
        r = ext4_ftruncate_no_lock(f, size);
+       if (r != EOK)
+               ext4_trans_abort(f->mp);
+       else
+               ext4_trans_stop(f->mp);
 
        EXT4_MP_UNLOCK(f->mp);
        return r;
@@ -1275,16 +1516,17 @@ int ext4_ftruncate(ext4_file *f, uint64_t size)
 
 int ext4_fread(ext4_file *f, void *buf, size_t size, size_t *rcnt)
 {
-       uint32_t u;
-       uint32_t fblock;
-       uint32_t fblock_start;
-       uint32_t fblock_cnt;
-       uint32_t sblock;
-       uint32_t sblock_end;
+       uint32_t unalg;
+       uint32_t iblock_idx;
+       uint32_t iblock_last;
        uint32_t block_size;
+
+       ext4_fsblk_t fblock;
+       ext4_fsblk_t fblock_start;
+       uint32_t fblock_count;
+
        uint8_t *u8_buf = buf;
        int r;
-       struct ext4_block b;
        struct ext4_inode_ref ref;
 
        ext4_assert(f && f->mp);
@@ -1297,41 +1539,42 @@ int ext4_fread(ext4_file *f, void *buf, size_t size, size_t *rcnt)
 
        EXT4_MP_LOCK(f->mp);
 
+       struct ext4_fs *const fs = &f->mp->fs;
+       struct ext4_sblock *const sb = &f->mp->fs.sb;
+
        if (rcnt)
                *rcnt = 0;
 
-       r = ext4_fs_get_inode_ref(&f->mp->fs, f->inode, &ref);
+       r = ext4_fs_get_inode_ref(fs, f->inode, &ref);
        if (r != EOK) {
                EXT4_MP_UNLOCK(f->mp);
                return r;
        }
 
        /*Sync file size*/
-       f->fsize = ext4_inode_get_size(&f->mp->fs.sb, ref.inode);
+       f->fsize = ext4_inode_get_size(sb, ref.inode);
 
-       block_size = ext4_sb_get_block_size(&f->mp->fs.sb);
+       block_size = ext4_sb_get_block_size(sb);
        size = size > (f->fsize - f->fpos) ? (f->fsize - f->fpos) : size;
-       sblock = (f->fpos) / block_size;
-       sblock_end = (f->fpos + size) / block_size;
-       u = (f->fpos) % block_size;
+
+       iblock_idx = (f->fpos) / block_size;
+       iblock_last = (f->fpos + size) / block_size;
+       unalg = (f->fpos) % block_size;
 
        /*If the size of symlink is smaller than 60 bytes*/
-       if (ext4_inode_is_type(&f->mp->fs.sb, ref.inode, EXT4_INODE_MODE_SOFTLINK)
-                       && f->fsize < sizeof(ref.inode->blocks)
-                       && !ext4_inode_get_blocks_count(&f->mp->fs.sb, ref.inode)) {
+       bool softlink;
+       softlink = ext4_inode_is_type(sb, ref.inode, EXT4_INODE_MODE_SOFTLINK);
+       if (softlink && f->fsize < sizeof(ref.inode->blocks)
+                    && !ext4_inode_get_blocks_count(sb, ref.inode)) {
+
                char *content = (char *)ref.inode->blocks;
                if (f->fpos < f->fsize) {
-                       r = (u + size > f->fsize)
-                               ?(f->fsize - u)
-                               :(size);
-                       memcpy(buf, content + u, r);
-                       if (rcnt)
-                               *rcnt = r;
-
-               } else {
-                       r = 0;
+                       size_t len = size;
+                       if (unalg + size > f->fsize)
+                               len = f->fsize - unalg;
+                       memcpy(buf, content + unalg, len);
                        if (rcnt)
-                               *rcnt = 0;
+                               *rcnt = len;
 
                }
 
@@ -1339,83 +1582,81 @@ int ext4_fread(ext4_file *f, void *buf, size_t size, size_t *rcnt)
                goto Finish;
        }
 
-       if (u) {
-
-               uint32_t ll = size > (block_size - u) ? (block_size - u) : size;
-
-               r = ext4_fs_get_inode_data_block_index(&ref, sblock, &fblock);
-               if (r != EOK)
-                       goto Finish;
+       if (unalg) {
+               size_t len =  size;
+               if (size > (block_size - unalg))
+                       len = block_size - unalg;
 
-               r = ext4_block_get(f->mp->fs.bdev, &b, fblock);
+               r = ext4_fs_get_inode_dblk_idx(&ref, iblock_idx, &fblock, true);
                if (r != EOK)
                        goto Finish;
 
-               memcpy(u8_buf, b.data + u, ll);
+               /* Do we get an unwritten range? */
+               if (fblock != 0) {
+                       uint64_t off = fblock * block_size + unalg;
+                       r = ext4_block_readbytes(f->mp->fs.bdev, off, u8_buf, len);
+                       if (r != EOK)
+                               goto Finish;
 
-               r = ext4_block_set(f->mp->fs.bdev, &b);
-               if (r != EOK)
-                       goto Finish;
+               } else {
+                       /* Yes, we do. */
+                       memset(u8_buf, 0, len);
+               }
 
-               u8_buf += ll;
-               size -= ll;
-               f->fpos += ll;
+               u8_buf += len;
+               size -= len;
+               f->fpos += len;
 
                if (rcnt)
-                       *rcnt += ll;
+                       *rcnt += len;
 
-               sblock++;
+               iblock_idx++;
        }
 
        fblock_start = 0;
-       fblock_cnt = 0;
+       fblock_count = 0;
        while (size >= block_size) {
-               while (sblock < sblock_end) {
-                       r = ext4_fs_get_inode_data_block_index(&ref, sblock,
-                                                              &fblock);
+               while (iblock_idx < iblock_last) {
+                       r = ext4_fs_get_inode_dblk_idx(&ref, iblock_idx,
+                                                      &fblock, true);
                        if (r != EOK)
                                goto Finish;
 
-                       sblock++;
+                       iblock_idx++;
 
-                       if (!fblock_start) {
+                       if (!fblock_start)
                                fblock_start = fblock;
-                       }
 
-                       if ((fblock_start + fblock_cnt) != fblock)
+                       if ((fblock_start + fblock_count) != fblock)
                                break;
 
-                       fblock_cnt++;
+                       fblock_count++;
                }
 
                r = ext4_blocks_get_direct(f->mp->fs.bdev, u8_buf, fblock_start,
-                                          fblock_cnt);
+                                          fblock_count);
                if (r != EOK)
                        goto Finish;
 
-               size -= block_size * fblock_cnt;
-               u8_buf += block_size * fblock_cnt;
-               f->fpos += block_size * fblock_cnt;
+               size -= block_size * fblock_count;
+               u8_buf += block_size * fblock_count;
+               f->fpos += block_size * fblock_count;
 
                if (rcnt)
-                       *rcnt += block_size * fblock_cnt;
+                       *rcnt += block_size * fblock_count;
 
                fblock_start = fblock;
-               fblock_cnt = 1;
+               fblock_count = 1;
        }
 
        if (size) {
-               r = ext4_fs_get_inode_data_block_index(&ref, sblock, &fblock);
-               if (r != EOK)
-                       goto Finish;
-
-               r = ext4_block_get(f->mp->fs.bdev, &b, fblock);
+               uint64_t off;
+               r = ext4_fs_get_inode_dblk_idx(&ref, iblock_idx, &fblock, true);
                if (r != EOK)
                        goto Finish;
 
-               memcpy(u8_buf, b.data, size);
-
-               r = ext4_block_set(f->mp->fs.bdev, &b);
+               off = fblock * block_size;
+               r = ext4_block_readbytes(f->mp->fs.bdev, off, u8_buf, size);
                if (r != EOK)
                        goto Finish;
 
@@ -1433,20 +1674,19 @@ Finish:
 
 int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
 {
-       uint32_t u;
-       uint32_t fblock;
-
-       uint32_t sblock;
-       uint32_t sblock_end;
-       uint32_t file_blocks;
+       uint32_t unalg;
+       uint32_t iblk_idx;
+       uint32_t iblock_last;
+       uint32_t ifile_blocks;
        uint32_t block_size;
-       uint32_t fblock_start;
-       uint32_t fblock_cnt;
 
-       struct ext4_block b;
+       uint32_t fblock_count;
+       ext4_fsblk_t fblk;
+       ext4_fsblk_t fblock_start;
+
        struct ext4_inode_ref ref;
        const uint8_t *u8_buf = buf;
-       int r;
+       int r, rr = EOK;
 
        ext4_assert(f && f->mp);
 
@@ -1457,58 +1697,54 @@ int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
                return EOK;
 
        EXT4_MP_LOCK(f->mp);
+       ext4_trans_start(f->mp);
+
+       struct ext4_fs *const fs = &f->mp->fs;
+       struct ext4_sblock *const sb = &f->mp->fs.sb;
 
        if (wcnt)
                *wcnt = 0;
 
-       r = ext4_fs_get_inode_ref(&f->mp->fs, f->inode, &ref);
+       r = ext4_fs_get_inode_ref(fs, f->inode, &ref);
        if (r != EOK) {
+               ext4_trans_abort(f->mp);
                EXT4_MP_UNLOCK(f->mp);
                return r;
        }
 
        /*Sync file size*/
-       f->fsize = ext4_inode_get_size(&f->mp->fs.sb, ref.inode);
-
-       block_size = ext4_sb_get_block_size(&f->mp->fs.sb);
-
-       sblock_end = (f->fpos + size) > f->fsize ? (f->fpos + size) : f->fsize;
-       sblock_end /= block_size;
-       file_blocks = (f->fsize / block_size);
+       f->fsize = ext4_inode_get_size(sb, ref.inode);
+       block_size = ext4_sb_get_block_size(sb);
 
-       if (f->fsize % block_size)
-               file_blocks++;
+       iblock_last = (f->fpos + size) / block_size;
+       iblk_idx = (f->fpos) / block_size;
+       ifile_blocks = (f->fsize + block_size - 1) / block_size;
 
-       sblock = (f->fpos) / block_size;
+       unalg = (f->fpos) % block_size;
 
-       u = (f->fpos) % block_size;
+       if (unalg) {
+               size_t len =  size;
+               uint64_t off;
+               if (size > (block_size - unalg))
+                       len = block_size - unalg;
 
-       if (u) {
-               uint32_t ll = size > (block_size - u) ? (block_size - u) : size;
-
-               r = ext4_fs_get_inode_data_block_index(&ref, sblock, &fblock);
-               if (r != EOK)
-                       goto Finish;
-
-               r = ext4_block_get(f->mp->fs.bdev, &b, fblock);
+               r = ext4_fs_init_inode_dblk_idx(&ref, iblk_idx, &fblk);
                if (r != EOK)
                        goto Finish;
 
-               memcpy(b.data + u, u8_buf, ll);
-               b.dirty = true;
-
-               r = ext4_block_set(f->mp->fs.bdev, &b);
+               off = fblk * block_size + unalg;
+               r = ext4_block_writebytes(f->mp->fs.bdev, off, u8_buf, len);
                if (r != EOK)
                        goto Finish;
 
-               u8_buf += ll;
-               size -= ll;
-               f->fpos += ll;
+               u8_buf += len;
+               size -= len;
+               f->fpos += len;
 
                if (wcnt)
-                       *wcnt += ll;
+                       *wcnt += len;
 
-               sblock++;
+               iblk_idx++;
        }
 
        /*Start write back cache mode.*/
@@ -1517,48 +1753,60 @@ int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
                goto Finish;
 
        fblock_start = 0;
-       fblock_cnt = 0;
+       fblock_count = 0;
        while (size >= block_size) {
 
-               while (sblock < sblock_end) {
-                       if (sblock < file_blocks) {
-                               r = ext4_fs_get_inode_data_block_index(
-                                   &ref, sblock, &fblock);
+               while (iblk_idx < iblock_last) {
+                       if (iblk_idx < ifile_blocks) {
+                               r = ext4_fs_init_inode_dblk_idx(&ref, iblk_idx,
+                                                               &fblk);
                                if (r != EOK)
-                                       break;
+                                       goto Finish;
                        } else {
-                               r = ext4_fs_append_inode_block(&ref, &fblock,
-                                                              &sblock);
-                               if (r != EOK)
+                               rr = ext4_fs_append_inode_dblk(&ref, &fblk,
+                                                              &iblk_idx);
+                               if (rr != EOK) {
+                                       /* Unable to append more blocks. But
+                                        * some block might be allocated already
+                                        * */
                                        break;
+                               }
                        }
 
-                       sblock++;
+                       iblk_idx++;
 
                        if (!fblock_start) {
-                               fblock_start = fblock;
+                               fblock_start = fblk;
                        }
 
-                       if ((fblock_start + fblock_cnt) != fblock)
+                       if ((fblock_start + fblock_count) != fblk)
                                break;
 
-                       fblock_cnt++;
+                       fblock_count++;
                }
 
                r = ext4_blocks_set_direct(f->mp->fs.bdev, u8_buf, fblock_start,
-                                          fblock_cnt);
+                                          fblock_count);
                if (r != EOK)
                        break;
 
-               size -= block_size * fblock_cnt;
-               u8_buf += block_size * fblock_cnt;
-               f->fpos += block_size * fblock_cnt;
+               size -= block_size * fblock_count;
+               u8_buf += block_size * fblock_count;
+               f->fpos += block_size * fblock_count;
 
                if (wcnt)
-                       *wcnt += block_size * fblock_cnt;
+                       *wcnt += block_size * fblock_count;
 
-               fblock_start = fblock;
-               fblock_cnt = 1;
+               fblock_start = fblk;
+               fblock_count = 1;
+
+               if (rr != EOK) {
+                       /*ext4_fs_append_inode_block has failed and no
+                        * more blocks might be written. But node size
+                        * should be updated.*/
+                       r = rr;
+                       goto out_fsize;
+               }
        }
 
        /*Stop write back cache mode*/
@@ -1568,25 +1816,20 @@ int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
                goto Finish;
 
        if (size) {
-               if (sblock < file_blocks) {
-                       r = ext4_fs_get_inode_data_block_index(&ref, sblock,
-                                                              &fblock);
+               uint64_t off;
+               if (iblk_idx < ifile_blocks) {
+                       r = ext4_fs_init_inode_dblk_idx(&ref, iblk_idx, &fblk);
                        if (r != EOK)
                                goto Finish;
                } else {
-                       r = ext4_fs_append_inode_block(&ref, &fblock, &sblock);
+                       r = ext4_fs_append_inode_dblk(&ref, &fblk, &iblk_idx);
                        if (r != EOK)
-                               goto Finish;
+                               /*Node size sholud be updated.*/
+                               goto out_fsize;
                }
 
-               r = ext4_block_get(f->mp->fs.bdev, &b, fblock);
-               if (r != EOK)
-                       goto Finish;
-
-               memcpy(b.data, u8_buf, size);
-               b.dirty = true;
-
-               r = ext4_block_set(f->mp->fs.bdev, &b);
+               off = fblk * block_size;
+               r = ext4_block_writebytes(f->mp->fs.bdev, off, u8_buf, size);
                if (r != EOK)
                        goto Finish;
 
@@ -1596,6 +1839,7 @@ int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
                        *wcnt += size;
        }
 
+out_fsize:
        if (f->fpos > f->fsize) {
                f->fsize = f->fpos;
                ext4_inode_set_size(ref.inode, f->fsize);
@@ -1603,7 +1847,13 @@ int ext4_fwrite(ext4_file *f, const void *buf, size_t size, size_t *wcnt)
        }
 
 Finish:
-       ext4_fs_put_inode_ref(&ref);
+       r = ext4_fs_put_inode_ref(&ref);
+
+       if (r != EOK)
+               ext4_trans_abort(f->mp);
+       else
+               ext4_trans_stop(f->mp);
+
        EXT4_MP_UNLOCK(f->mp);
        return r;
 }
@@ -1633,9 +1883,15 @@ int ext4_fseek(ext4_file *f, uint64_t offset, uint32_t origin)
        return EINVAL;
 }
 
-uint64_t ext4_ftell(ext4_file *f) { return f->fpos; }
+uint64_t ext4_ftell(ext4_file *f)
+{
+       return f->fpos;
+}
 
-uint64_t ext4_fsize(ext4_file *f) { return f->fsize; }
+uint64_t ext4_fsize(ext4_file *f)
+{
+       return f->fsize;
+}
 
 int ext4_chmod(const char *path, uint32_t mode)
 {
@@ -1650,9 +1906,11 @@ int ext4_chmod(const char *path, uint32_t mode)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DIRENTRY_UNKNOWN, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1661,6 +1919,7 @@ int ext4_chmod(const char *path, uint32_t mode)
        ext4_fclose(&f);
        r = ext4_fs_get_inode_ref(&mp->fs, ino, &inode_ref);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1668,7 +1927,12 @@ int ext4_chmod(const char *path, uint32_t mode)
        ext4_inode_set_mode(sb, inode_ref.inode, mode);
        inode_ref.dirty = true;
 
-       ext4_fs_put_inode_ref(&inode_ref);
+       r = ext4_fs_put_inode_ref(&inode_ref);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1685,9 +1949,11 @@ int ext4_chown(const char *path, uint32_t uid, uint32_t gid)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DIRENTRY_UNKNOWN, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1695,6 +1961,7 @@ int ext4_chown(const char *path, uint32_t uid, uint32_t gid)
        ext4_fclose(&f);
        r = ext4_fs_get_inode_ref(&mp->fs, ino, &inode_ref);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1703,7 +1970,12 @@ int ext4_chown(const char *path, uint32_t uid, uint32_t gid)
        ext4_inode_set_gid(inode_ref.inode, gid);
        inode_ref.dirty = true;
 
-       ext4_fs_put_inode_ref(&inode_ref);
+       r = ext4_fs_put_inode_ref(&inode_ref);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1720,9 +1992,11 @@ int ext4_file_set_atime(const char *path, uint32_t atime)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DIRENTRY_UNKNOWN, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1730,6 +2004,7 @@ int ext4_file_set_atime(const char *path, uint32_t atime)
        ext4_fclose(&f);
        r = ext4_fs_get_inode_ref(&mp->fs, ino, &inode_ref);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1737,7 +2012,12 @@ int ext4_file_set_atime(const char *path, uint32_t atime)
        ext4_inode_set_access_time(inode_ref.inode, atime);
        inode_ref.dirty = true;
 
-       ext4_fs_put_inode_ref(&inode_ref);
+       r = ext4_fs_put_inode_ref(&inode_ref);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1754,9 +2034,11 @@ int ext4_file_set_mtime(const char *path, uint32_t mtime)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DIRENTRY_UNKNOWN, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1764,14 +2046,20 @@ int ext4_file_set_mtime(const char *path, uint32_t mtime)
        ext4_fclose(&f);
        r = ext4_fs_get_inode_ref(&mp->fs, ino, &inode_ref);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
 
-       ext4_inode_set_modification_time(inode_ref.inode, mtime);
+       ext4_inode_set_modif_time(inode_ref.inode, mtime);
        inode_ref.dirty = true;
 
-       ext4_fs_put_inode_ref(&inode_ref);
+       r = ext4_fs_put_inode_ref(&inode_ref);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1788,9 +2076,11 @@ int ext4_file_set_ctime(const char *path, uint32_t ctime)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
-       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DIRENTRY_UNKNOWN, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1798,6 +2088,7 @@ int ext4_file_set_ctime(const char *path, uint32_t ctime)
        ext4_fclose(&f);
        r = ext4_fs_get_inode_ref(&mp->fs, ino, &inode_ref);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1805,16 +2096,21 @@ int ext4_file_set_ctime(const char *path, uint32_t ctime)
        ext4_inode_set_change_inode_time(inode_ref.inode, ctime);
        inode_ref.dirty = true;
 
-       ext4_fs_put_inode_ref(&inode_ref);
+       r = ext4_fs_put_inode_ref(&inode_ref);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
 
 static int ext4_fsymlink_set(ext4_file *f, const void *buf, uint32_t size)
 {
-       struct ext4_block b;
        struct ext4_inode_ref ref;
-       uint32_t sblock, fblock;
+       uint32_t sblock;
+       ext4_fsblk_t fblock;
        uint32_t block_size;
        int r;
 
@@ -1824,10 +2120,8 @@ static int ext4_fsymlink_set(ext4_file *f, const void *buf, uint32_t size)
                return EOK;
 
        r = ext4_fs_get_inode_ref(&f->mp->fs, f->inode, &ref);
-       if (r != EOK) {
-               EXT4_MP_UNLOCK(f->mp);
+       if (r != EOK)
                return r;
-       }
 
        /*Sync file size*/
        block_size = ext4_sb_get_block_size(&f->mp->fs.sb);
@@ -1846,25 +2140,19 @@ static int ext4_fsymlink_set(ext4_file *f, const void *buf, uint32_t size)
 
        /*If the size of symlink is smaller than 60 bytes*/
        if (size < sizeof(ref.inode->blocks)) {
-               char *content = (char *)ref.inode->blocks;
-               memset(content, 0, sizeof(ref.inode->blocks));
-               memcpy(content, buf, size);
+               memset(ref.inode->blocks, 0, sizeof(ref.inode->blocks));
+               memcpy(ref.inode->blocks, buf, size);
                ext4_inode_clear_flag(ref.inode, EXT4_INODE_FLAG_EXTENTS);
        } else {
                ext4_fs_inode_blocks_init(&f->mp->fs, &ref);
-               r = ext4_fs_append_inode_block(&ref, &fblock, &sblock);
+               r = ext4_fs_append_inode_dblk(&ref, &fblock, &sblock);
                if (r != EOK)
                        goto Finish;
 
-               r = ext4_block_get(f->mp->fs.bdev, &b, fblock);
+               r = ext4_block_writebytes(f->mp->fs.bdev, 0, buf, size);
                if (r != EOK)
                        goto Finish;
 
-               memcpy(b.data, buf, size);
-               b.dirty = true;
-               r = ext4_block_set(f->mp->fs.bdev, &b);
-               if (r != EOK)
-                       goto Finish;
        }
 
        /*Stop write back cache mode*/
@@ -1895,11 +2183,13 @@ int ext4_fsymlink(const char *target, const char *path)
        if (!mp)
                return ENOENT;
 
-       filetype = EXT4_DIRENTRY_SYMLINK;
+       filetype = EXT4_DE_SYMLINK;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
+
        ext4_block_cache_write_back(mp->fs.bdev, 1);
-       r = ext4_generic_open2(&f, path, O_RDWR|O_CREAT, filetype, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDWR|O_CREAT, filetype, NULL, NULL);
        if (r == EOK)
                r = ext4_fsymlink_set(&f, target, strlen(target));
        else
@@ -1909,6 +2199,12 @@ int ext4_fsymlink(const char *target, const char *path)
 
 Finish:
        ext4_block_cache_write_back(mp->fs.bdev, 0);
+
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -1926,13 +2222,11 @@ int ext4_readlink(const char *path, char *buf, size_t bufsize, size_t *rcnt)
        if (!buf)
                return EINVAL;
 
-       memset(buf, 0, sizeof(bufsize));
-
-       filetype = EXT4_DIRENTRY_SYMLINK;
+       filetype = EXT4_DE_SYMLINK;
 
        EXT4_MP_LOCK(mp);
        ext4_block_cache_write_back(mp->fs.bdev, 1);
-       r = ext4_generic_open2(&f, path, O_RDONLY, filetype, 0, 0);
+       r = ext4_generic_open2(&f, path, O_RDONLY, filetype, NULL, NULL);
        if (r == EOK)
                r = ext4_fread(&f, buf, bufsize, rcnt);
        else
@@ -1942,10 +2236,259 @@ int ext4_readlink(const char *path, char *buf, size_t bufsize, size_t *rcnt)
 
 Finish:
        ext4_block_cache_write_back(mp->fs.bdev, 0);
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
        EXT4_MP_UNLOCK(mp);
        return r;
 }
 
+int ext4_setxattr(const char *path, const char *name, size_t name_len,
+                 const void *data, size_t data_size, bool replace)
+{
+       int r = EOK;
+       ext4_file f;
+       uint32_t inode;
+       uint8_t name_index;
+       const char *dissected_name = NULL;
+       size_t dissected_len = 0;
+       struct ext4_xattr_ref xattr_ref;
+       struct ext4_inode_ref inode_ref;
+       struct ext4_mountpoint *mp = ext4_get_mount(path);
+       if (!mp)
+               return ENOENT;
+
+       dissected_name = ext4_extract_xattr_name(name, name_len,
+                               &name_index, &dissected_len);
+       if (!dissected_len)
+               return EINVAL;
+
+       EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
+
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
+       if (r != EOK)
+               goto Finish;
+       inode = f.inode;
+       ext4_fclose(&f);
+
+       r = ext4_fs_get_inode_ref(&mp->fs, inode, &inode_ref);
+       if (r != EOK)
+               goto Finish;
+
+       r = ext4_fs_get_xattr_ref(&mp->fs, &inode_ref, &xattr_ref);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&inode_ref);
+               goto Finish;
+       }
+
+       r = ext4_fs_set_xattr(&xattr_ref, name_index, dissected_name,
+                       dissected_len, data, data_size, replace);
+
+       ext4_fs_put_xattr_ref(&xattr_ref);
+       ext4_fs_put_inode_ref(&inode_ref);
+Finish:
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
+       EXT4_MP_UNLOCK(mp);
+       return r;
+}
+
+int ext4_getxattr(const char *path, const char *name, size_t name_len,
+                 void *buf, size_t buf_size, size_t *data_size)
+{
+       int r = EOK;
+       ext4_file f;
+       uint32_t inode;
+       uint8_t name_index;
+       const char *dissected_name = NULL;
+       size_t dissected_len = 0;
+       struct ext4_xattr_ref xattr_ref;
+       struct ext4_inode_ref inode_ref;
+       struct ext4_mountpoint *mp = ext4_get_mount(path);
+       if (!mp)
+               return ENOENT;
+
+       dissected_name = ext4_extract_xattr_name(name, name_len,
+                               &name_index, &dissected_len);
+       if (!dissected_len)
+               return EINVAL;
+
+       EXT4_MP_LOCK(mp);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
+       if (r != EOK)
+               goto Finish;
+       inode = f.inode;
+       ext4_fclose(&f);
+
+       r = ext4_fs_get_inode_ref(&mp->fs, inode, &inode_ref);
+       if (r != EOK)
+               goto Finish;
+
+       r = ext4_fs_get_xattr_ref(&mp->fs, &inode_ref, &xattr_ref);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&inode_ref);
+               goto Finish;
+       }
+
+       r = ext4_fs_get_xattr(&xattr_ref, name_index, dissected_name,
+                               dissected_len, buf, buf_size, data_size);
+
+       ext4_fs_put_xattr_ref(&xattr_ref);
+       ext4_fs_put_inode_ref(&inode_ref);
+Finish:
+       EXT4_MP_UNLOCK(mp);
+       return r;
+}
+
+struct ext4_listxattr_iterator {
+       char *list;
+       char *list_ptr;
+       size_t size;
+       size_t ret_size;
+       bool list_too_small;
+       bool get_required_size;
+};
+
+static int ext4_iterate_ea_list(struct ext4_xattr_ref *ref,
+                               struct ext4_xattr_item *item)
+{
+       struct ext4_listxattr_iterator *lxi;
+       lxi = ref->iter_arg;
+       if (!lxi->get_required_size) {
+               size_t plen;
+               const char *prefix;
+               prefix = ext4_get_xattr_name_prefix(item->name_index, &plen);
+               if (lxi->ret_size + plen + item->name_len + 1 > lxi->size) {
+                       lxi->list_too_small = 1;
+                       return EXT4_XATTR_ITERATE_STOP;
+               }
+               if (prefix) {
+                       memcpy(lxi->list_ptr, prefix, plen);
+                       lxi->list_ptr += plen;
+                       lxi->ret_size += plen;
+               }
+               memcpy(lxi->list_ptr, item->name, item->name_len);
+               lxi->list_ptr[item->name_len] = 0;
+               lxi->list_ptr += item->name_len + 1;
+       }
+       lxi->ret_size += item->name_len + 1;
+       return EXT4_XATTR_ITERATE_CONT;
+}
+
+int ext4_listxattr(const char *path, char *list, size_t size, size_t *ret_size)
+{
+       int r = EOK;
+       ext4_file f;
+       uint32_t inode;
+       struct ext4_xattr_ref xattr_ref;
+       struct ext4_inode_ref inode_ref;
+       struct ext4_listxattr_iterator lxi;
+       struct ext4_mountpoint *mp = ext4_get_mount(path);
+       if (!mp)
+               return ENOENT;
+
+       lxi.list = list;
+       lxi.list_ptr = list;
+       lxi.size = size;
+       lxi.ret_size = 0;
+       lxi.list_too_small = false;
+       lxi.get_required_size = (!size) ? true : false;
+
+       EXT4_MP_LOCK(mp);
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
+       if (r != EOK)
+               goto Finish;
+       inode = f.inode;
+       ext4_fclose(&f);
+
+       r = ext4_fs_get_inode_ref(&mp->fs, inode, &inode_ref);
+       if (r != EOK)
+               goto Finish;
+
+       r = ext4_fs_get_xattr_ref(&mp->fs, &inode_ref, &xattr_ref);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&inode_ref);
+               goto Finish;
+       }
+
+       xattr_ref.iter_arg = &lxi;
+       ext4_fs_xattr_iterate(&xattr_ref, ext4_iterate_ea_list);
+       if (lxi.list_too_small)
+               r = ERANGE;
+
+       if (r == EOK) {
+               if (ret_size)
+                       *ret_size = lxi.ret_size;
+
+       }
+       ext4_fs_put_xattr_ref(&xattr_ref);
+       ext4_fs_put_inode_ref(&inode_ref);
+Finish:
+       EXT4_MP_UNLOCK(mp);
+       return r;
+
+}
+
+int ext4_removexattr(const char *path, const char *name, size_t name_len)
+{
+       int r = EOK;
+       ext4_file f;
+       uint32_t inode;
+       uint8_t name_index;
+       const char *dissected_name = NULL;
+       size_t dissected_len = 0;
+       struct ext4_xattr_ref xattr_ref;
+       struct ext4_inode_ref inode_ref;
+       struct ext4_mountpoint *mp = ext4_get_mount(path);
+       if (!mp)
+               return ENOENT;
+
+       dissected_name = ext4_extract_xattr_name(name, name_len,
+                                               &name_index, &dissected_len);
+       if (!dissected_len)
+               return EINVAL;
+
+       EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
+
+       r = ext4_generic_open2(&f, path, O_RDWR, EXT4_DE_UNKNOWN, NULL, NULL);
+       if (r != EOK)
+               goto Finish;
+       inode = f.inode;
+       ext4_fclose(&f);
+
+       r = ext4_fs_get_inode_ref(&mp->fs, inode, &inode_ref);
+       if (r != EOK)
+               goto Finish;
+
+       r = ext4_fs_get_xattr_ref(&mp->fs, &inode_ref, &xattr_ref);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&inode_ref);
+               goto Finish;
+       }
+
+       r = ext4_fs_remove_xattr(&xattr_ref, name_index, dissected_name,
+                               dissected_len);
+
+       ext4_fs_put_xattr_ref(&xattr_ref);
+       ext4_fs_put_inode_ref(&inode_ref);
+Finish:
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
+
+       EXT4_MP_UNLOCK(mp);
+       return r;
+
+}
+
 /*********************************DIRECTORY OPERATION************************/
 
 int ext4_dir_rm(const char *path)
@@ -1955,18 +2498,13 @@ int ext4_dir_rm(const char *path)
        ext4_file f;
 
        struct ext4_mountpoint *mp = ext4_get_mount(path);
-       struct ext4_inode_ref current;
-       struct ext4_inode_ref child;
-       struct ext4_directory_iterator it;
+       struct ext4_inode_ref act, parent;
 
        uint32_t name_off;
-       uint32_t inode_up;
-       uint32_t inode_current;
-       uint32_t depth = 1;
+       uint32_t inode_up, inode_current;
 
-       bool has_children;
+       bool has_children = false;
        bool is_goal;
-       bool dir_end;
 
        if (!mp)
                return ENOENT;
@@ -1976,6 +2514,7 @@ int ext4_dir_rm(const char *path)
        /*Check if exist.*/
        r = ext4_generic_open(&f, path, "r", false, &inode_up, &name_off);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -1984,169 +2523,81 @@ int ext4_dir_rm(const char *path)
        len = ext4_path_check(path, &is_goal);
 
        inode_current = f.inode;
-       dir_end = false;
 
        ext4_block_cache_write_back(mp->fs.bdev, 1);
 
-       do {
-               /*Load directory node.*/
-               r = ext4_fs_get_inode_ref(&f.mp->fs, inode_current, &current);
-               if (r != EOK) {
-                       break;
-               }
-
-               /*Initialize iterator.*/
-               r = ext4_dir_iterator_init(&it, &current, 0);
-               if (r != EOK) {
-                       ext4_fs_put_inode_ref(&current);
-                       break;
-               }
-
-               while (r == EOK) {
-
-                       if (!it.current) {
-                               dir_end = true;
-                               break;
-                       }
-
-                       /*Get up directory inode when ".." entry*/
-                       if ((it.current->name_length == 2) &&
-                           ext4_is_dots(it.current->name,
-                                        it.current->name_length)) {
-                               inode_up = ext4_dir_entry_ll_get_inode(it.current);
-                       }
-
-                       /*If directory or file entry,  but not "." ".." entry*/
-                       if (!ext4_is_dots(it.current->name,
-                                         it.current->name_length)) {
-
-                               /*Get child inode reference do unlink
-                                * directory/file.*/
-                               r = ext4_fs_get_inode_ref(&f.mp->fs,
-                                       ext4_dir_entry_ll_get_inode(it.current),
-                                       &child);
-                               if (r != EOK)
-                                       break;
-
-                               /*If directory with no leaf children*/
-                               r = ext4_has_children(&has_children, &child);
-                               if (r != EOK) {
-                                       ext4_fs_put_inode_ref(&child);
-                                       break;
-                               }
-
-                               if (has_children) {
-                                       /*Has directory children. Go into this
-                                        * directory.*/
-                                       inode_up = inode_current;
-                                       inode_current = ext4_dir_entry_ll_get_inode(it.current);
-                                       depth++;
-                                       ext4_fs_put_inode_ref(&child);
-                                       break;
-                               }
-
-                               /*No children in child directory or file. Just
-                                * unlink.*/
-                               r = ext4_unlink(f.mp, &current, &child,
-                                               (char *)it.current->name,
-                                               it.current->name_length);
-                               if (r != EOK) {
-                                       ext4_fs_put_inode_ref(&child);
-                                       break;
-                               }
-
-                               ext4_inode_set_deletion_time(child.inode,
-                                                            0xFFFFFFFF);
-                               ext4_inode_set_links_count(child.inode, 0);
-                               child.dirty = true;
-                               /*Turncate*/
-                               r = ext4_fs_truncate_inode(&child, 0);
-                               if (r != EOK) {
-                                       ext4_fs_put_inode_ref(&child);
-                                       break;
-                               }
-
-                               r = ext4_fs_free_inode(&child);
-                               if (r != EOK) {
-                                       ext4_fs_put_inode_ref(&child);
-                                       break;
-                               }
+       /*Load parent.*/
+       r = ext4_fs_get_inode_ref(&f.mp->fs, inode_up,
+                       &parent);
+       if (r != EOK)
+               goto Finish;
+       r = ext4_fs_get_inode_ref(&f.mp->fs, inode_current,
+                       &act);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&parent);
+               goto Finish;
+       }
+       r = ext4_has_children(&has_children, &act);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&parent);
+               ext4_fs_put_inode_ref(&act);
+               goto Finish;
+       }
+       if (has_children) {
+               r = ENOTEMPTY;
+               ext4_fs_put_inode_ref(&parent);
+               ext4_fs_put_inode_ref(&act);
+               goto Finish;
+       }
 
-                               r = ext4_fs_put_inode_ref(&child);
-                               if (r != EOK)
-                                       break;
-                       }
+       ext4_trans_start(mp);
 
-                       r = ext4_dir_iterator_next(&it);
+       if (ext4_inode_get_links_cnt(act.inode) == 2) {
+               /*Truncate*/
+               r = ext4_fs_truncate_inode(&act, 0);
+               if (r != EOK) {
+                       ext4_fs_put_inode_ref(&parent);
+                       ext4_fs_put_inode_ref(&act);
+                       goto Finish;
                }
+       }
 
-               if (dir_end) {
-                       /*Directory iterator reached last entry*/
-                       ext4_has_children(&has_children, &current);
-                       if (!has_children) {
-                               inode_current = inode_up;
-                               if (depth)
-                                       depth--;
-                       }
-                       /*Last unlink*/
-                       if (!depth) {
-                               /*Load parent.*/
-                               struct ext4_inode_ref parent;
-                               r = ext4_fs_get_inode_ref(&f.mp->fs, inode_up,
-                                                         &parent);
-                               if (r != EOK)
-                                       goto End;
-
-                               /* In this place all directories should be
-                                * unlinked.
-                                * Last unlink from root of current directory*/
-                               r = ext4_unlink(f.mp, &parent, &current,
-                                               (char *)path, len);
-                               if (r != EOK) {
-                                       ext4_fs_put_inode_ref(&parent);
-                                       goto End;
-                               }
+        /* Unlink from root of current directory*/
+       r = ext4_unlink(f.mp, &parent, &act,
+                       (char *)path, len);
+       if (r != EOK) {
+               ext4_fs_put_inode_ref(&parent);
+               ext4_fs_put_inode_ref(&act);
+               goto Finish;
+       }
 
-                               if (ext4_inode_get_links_count(current.inode) ==
-                                   2) {
-                                       ext4_inode_set_deletion_time(
-                                           current.inode, 0xFFFFFFFF);
-                                       ext4_inode_set_links_count(
-                                           current.inode, 0);
-                                       current.dirty = true;
-                                       /*Turncate*/
-                                       r = ext4_fs_truncate_inode(&current, 0);
-                                       if (r != EOK) {
-                                               ext4_fs_put_inode_ref(&parent);
-                                               goto End;
-                                       }
-
-                                       r = ext4_fs_free_inode(&current);
-                                       if (r != EOK) {
-                                               ext4_fs_put_inode_ref(&parent);
-                                               goto End;
-                                       }
-                               }
+       if (ext4_inode_get_links_cnt(act.inode) == 2) {
+               ext4_inode_set_del_time(act.inode, -1L);
+               ext4_inode_set_links_cnt(act.inode, 0);
+               act.dirty = true;
 
-                               r = ext4_fs_put_inode_ref(&parent);
-                               if (r != EOK)
-                                       goto End;
-                       }
+               r = ext4_fs_free_inode(&act);
+               if (r != EOK) {
+                       ext4_fs_put_inode_ref(&parent);
+                       ext4_fs_put_inode_ref(&act);
+                       goto Finish;
                }
+       }
 
-       End:
-               ext4_dir_iterator_fini(&it);
-               ext4_fs_put_inode_ref(&current);
-               dir_end = false;
-
-               /*When something goes wrong. End loop.*/
-               if (r != EOK)
-                       break;
+       r = ext4_fs_put_inode_ref(&parent);
+       if (r != EOK)
+               goto Finish;
 
-       } while (depth);
+       r = ext4_fs_put_inode_ref(&act);
+Finish:
+       if (r != EOK)
+               ext4_trans_abort(mp);
+       else
+               ext4_trans_stop(mp);
 
        ext4_block_cache_write_back(mp->fs.bdev, 0);
        EXT4_MP_UNLOCK(mp);
+
        return r;
 }
 
@@ -2161,11 +2612,13 @@ int ext4_dir_mk(const char *path)
                return ENOENT;
 
        EXT4_MP_LOCK(mp);
+       ext4_trans_start(mp);
 
        /*Check if exist.*/
        r = ext4_generic_open(&f, path, "r", false, 0, 0);
        if (r == EOK) {
                /*Directory already created*/
+               ext4_trans_stop(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
@@ -2173,10 +2626,12 @@ int ext4_dir_mk(const char *path)
        /*Create new dir*/
        r = ext4_generic_open(&f, path, "w", false, 0, 0);
        if (r != EOK) {
+               ext4_trans_abort(mp);
                EXT4_MP_UNLOCK(mp);
                return r;
        }
 
+       ext4_trans_stop(mp);
        EXT4_MP_UNLOCK(mp);
        return r;
 }
@@ -2208,7 +2663,7 @@ const ext4_direntry *ext4_dir_entry_next(ext4_dir *d)
        int r;
        ext4_direntry *de = 0;
        struct ext4_inode_ref dir;
-       struct ext4_directory_iterator it;
+       struct ext4_dir_iter it;
 
        EXT4_MP_LOCK(d->f.mp);
 
@@ -2228,13 +2683,12 @@ const ext4_direntry *ext4_dir_entry_next(ext4_dir *d)
                goto Finish;
        }
 
-       memcpy(&d->de, it.current, sizeof(ext4_direntry));
+       memcpy(&d->de, it.curr, sizeof(ext4_direntry));
        de = &d->de;
 
        ext4_dir_iterator_next(&it);
 
-       d->next_off =
-           it.current ? it.current_offset : EXT4_DIR_ENTRY_OFFSET_TERM;
+       d->next_off = it.curr ? it.curr_off : EXT4_DIR_ENTRY_OFFSET_TERM;
 
        ext4_dir_iterator_fini(&it);
        ext4_fs_put_inode_ref(&dir);
@@ -2246,7 +2700,106 @@ Finish:
 
 void ext4_dir_entry_rewind(ext4_dir *d)
 {
-    d->next_off = 0;
+       d->next_off = 0;
+}
+
+int ext4_test_journal(const char *mount_point)
+{
+       struct ext4_mountpoint *mp = ext4_get_mount(mount_point);
+       if (!mp)
+               return ENOENT;
+
+       int r = ENOTSUP;
+       EXT4_MP_LOCK(mp);
+       ext4_block_cache_write_back(mp->fs.bdev, 1);
+       if (ext4_sb_feature_com(&mp->fs.sb, EXT4_FCOM_HAS_JOURNAL)) {
+               struct jbd_fs *jbd_fs = calloc(1, sizeof(struct jbd_fs));
+               struct jbd_journal *journal;
+               if (!jbd_fs) {
+                        r = ENOMEM;
+                        goto Finish;
+               }
+               journal = calloc(1, sizeof(struct jbd_journal));
+               if (!journal) {
+                       free(jbd_fs);
+                       r = ENOMEM;
+                       goto Finish;
+               }
+
+               r = jbd_get_fs(&mp->fs, jbd_fs);
+               if (r != EOK) {
+                       free(jbd_fs);
+                       goto Finish;
+               }
+               r = jbd_journal_start(jbd_fs, journal);
+               if (r != EOK) {
+                       jbd_put_fs(jbd_fs);
+                       free(journal);
+                       free(jbd_fs);
+                       goto Finish;
+               }
+
+               int i;
+               for (i = 0;i < 50;i++) {
+                       ext4_fsblk_t rand_block = rand() % 4096;
+                       if (!rand_block)
+                               rand_block = 1;
+                       struct ext4_block block;
+                       r = ext4_block_get(mp->fs.bdev, &block, rand_block);
+                       if (r != EOK)
+                               goto out;
+
+                       struct jbd_trans *t = jbd_journal_new_trans(journal);
+                       if (!t) {
+                               ext4_block_set(mp->fs.bdev, &block);
+                               r = ENOMEM;
+                               goto out;
+                       }
+
+                       switch (rand() % 2) {
+                       case 0:
+                               r = jbd_trans_get_access(journal, t, &block);
+                               if (r != EOK) {
+                                       jbd_journal_free_trans(journal, t,
+                                                              true);
+                                       ext4_block_set(mp->fs.bdev, &block);
+                                       r = ENOMEM;
+                                       goto out;
+                               }
+                               r = jbd_trans_set_block_dirty(t, &block);
+                               if (r != EOK) {
+                                       jbd_journal_free_trans(journal, t,
+                                                              true);
+                                       ext4_block_set(mp->fs.bdev, &block);
+                                       r = ENOMEM;
+                                       goto out;
+                               }
+                               break;
+                       case 1:
+                               r = jbd_trans_try_revoke_block(t, rand_block);
+                               if (r != EOK) {
+                                       jbd_journal_free_trans(journal, t,
+                                                              true);
+                                       ext4_block_set(mp->fs.bdev, &block);
+                                       r = ENOMEM;
+                                       goto out;
+                               }
+                       }
+                       ext4_block_set(mp->fs.bdev, &block);
+                       jbd_journal_submit_trans(journal, t);
+                       jbd_journal_commit_one(journal);
+               }
+out:
+               jbd_journal_stop(journal);
+               jbd_put_fs(jbd_fs);
+               free(journal);
+               free(jbd_fs);
+       }
+
+Finish:
+       ext4_block_cache_write_back(mp->fs.bdev, 0);
+       EXT4_MP_UNLOCK(mp);
+       return r;
 }
 
 /**