Changeset 2175178 in mainline for uspace/lib/ext4/src/superblock.c

Timestamp:

2018-10-03T08:34:52Z (6 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

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

Children:

b209135

Parents:

174156fd

git-author:

Jiri Svoboda <jiri@…> (2018-10-02 21:31:25)

git-committer:

Jiri Svoboda <jiri@…> (2018-10-03 08:34:52)

Message:

Allow creating ext2 dynamic revision. Allow choosing ext filesystem version by mkext4 argument. Fix support for filesystems without filetype feature.

File:

• uspace/lib/ext4/src/superblock.c (modified) (3 diffs)

uspace/lib/ext4/src/superblock.c

@@ -45 +45 @@
 #include <stdlib.h>
 #include <time.h>
+#include "ext4/cfg.h"
 #include "ext4/superblock.h"
 
@@ -1454 +1455 @@
  * @param dev_bsize Device block size
  * @param dev_bcnt Device number of blocks
+ * @param ver Filesystem version
  * @param rsb Place to store pointer to newly allocated superblock
  * @return EOK on success or error code
  */
 errno_t ext4_superblock_create(size_t dev_bsize, uint64_t dev_bcnt,
-    ext4_superblock_t **rsb)
+    ext4_cfg_ver_t ver, ext4_superblock_t **rsb)
 {
         ext4_superblock_t *sb;
@@ -1557 +1559 @@
         ext4_superblock_set_check_interval(sb, 0);
         ext4_superblock_set_creator_os(sb, EXT4_SUPERBLOCK_OS_LINUX);
-        ext4_superblock_set_rev_level(sb, EXT4_GOOD_OLD_REV);
+        if (ver >= extver_ext2)
+                ext4_superblock_set_rev_level(sb, EXT4_DYNAMIC_REV);
+        else
+                ext4_superblock_set_rev_level(sb, EXT4_GOOD_OLD_REV);
         ext4_superblock_set_def_resuid(sb, 0);
         ext4_superblock_set_def_resgid(sb, 0);
+
+        if (ver >= extver_ext2) {
+                /* Dynamic rev */
+                ext4_superblock_set_first_inode(sb, EXT4_REV0_FIRST_INO);
+                ext4_superblock_set_inode_size(sb, EXT4_REV0_INODE_SIZE);
+                ext4_superblock_set_block_group_index(sb, 0); // XXX
+                ext4_superblock_set_features_compatible(sb, 0);
+                ext4_superblock_set_features_incompatible(sb, 0);
+                ext4_superblock_set_features_read_only(sb, 0);
+
+                ext4_superblock_set_uuid(sb, &uuid);
+                /* 16-byte Latin-1 string padded with null characters */
+                ext4_superblock_set_volume_name(sb, "HelenOS-Ext4\0\0\0\0");
+                /* 64-byte Latin-1 string padded with null characters */
+                ext4_superblock_set_last_mounted(sb,
+                    "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"
+                    "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0");
+                sb->algorithm_usage_bitmap = 0;
+        }
 #if 0
-        /* Dynamic rev */
-        ext4_superblock_set_first_inode(sb, EXT4_REV0_FIRST_INO);
-        ext4_superblock_set_inode_size(sb, EXT4_REV0_INODE_SIZE);
-        ext4_superblock_set_block_group_index(sb, 0); // XXX
-        ext4_superblock_set_features_compatible(sb, 0);
-        ext4_superblock_set_features_incompatible(sb, 0);
-        ext4_superblock_set_features_read_only(sb, 0);
-
-        ext4_superblock_set_uuid(sb, &uuid);
-        /* 16-byte Latin-1 string padded with null characters */
-        ext4_superblock_set_volume_name(sb, "HelenOS-Ext4\0\0\0\0");
-        /* 64-byte Latin-1 string padded with null characters */
-        ext4_superblock_set_last_mounted(sb,
-            "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"
-            "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0");
-        sb->algorithm_usage_bitmap = 0;
-
         /* Journalling */
         ext4_superblock_set_desc_size(sb, EXT4_MAX_BLOCK_GROUP_DESCRIPTOR_SIZE);