]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ext4: fix EXT4_IOC_GROUP_ADD ioctl
authorruippan (潘睿) <ruippan@tencent.com>
Tue, 4 Dec 2018 06:04:12 +0000 (01:04 -0500)
committerTheodore Ts'o <tytso@mit.edu>
Tue, 4 Dec 2018 06:04:12 +0000 (01:04 -0500)
Commit e2b911c53584 ("ext4: clean up feature test macros with
predicate functions") broke the EXT4_IOC_GROUP_ADD ioctl.  This was
not noticed since only very old versions of resize2fs (before
e2fsprogs 1.42) use this ioctl.  However, using a new kernel with an
enterprise Linux userspace will cause attempts to use online resize to
fail with "No reserved GDT blocks".

Fixes: e2b911c53584 ("ext4: clean up feature test macros with predicate...")
Cc: stable@kernel.org # v4.4
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: ruippan (潘睿) <ruippan@tencent.com>
fs/ext4/resize.c

index 87350b6426818afad1f70cc2a649005c74016881..bc8ee0c498cc7627af0a3d4cd0accd5f21c008cb 100644 (file)
@@ -1631,7 +1631,7 @@ int ext4_group_add(struct super_block *sb, struct ext4_new_group_data *input)
        }
 
        if (reserved_gdb || gdb_off == 0) {
-               if (ext4_has_feature_resize_inode(sb) ||
+               if (!ext4_has_feature_resize_inode(sb) ||
                    !le16_to_cpu(es->s_reserved_gdt_blocks)) {
                        ext4_warning(sb,
                                     "No reserved GDT blocks, can't resize");