Here ([1] and below) is the 'git format-patch ...' with an added test case.
>From 59279e3c7437e24c4488c312823a471890037e2d Mon Sep 17 00:00:00 2001 From: Debarshi Ray <[email protected]> Date: Tue, 6 Jan 2009 01:02:38 +0530 Subject: [PATCH] Differentiate between Ext4 and Ext3 file systems. Fixes http://parted.alioth.debian.org/cgi-bin/trac.cgi/ticket/188 * libparted/fs/ext2/ext2_fs.h (EXT4_FEATURE_INCOMPAT_EXTENTS, EXT4_FEATURE_INCOMPAT_64BIT): New constants. * libparted/fs/ext2/interface.c (_ext2_generic_probe): Ext4 file systems will have EXT4_FEATURE_INCOMPAT_EXTENTS, while Ext3 will not. (_ext2_probe, _ext3_probe): Pass 2 and 3 respectively, instead of 0 and 1, to _ext2_generic_probe. (_ext4_probe): New function. (_ext4_ops, _ext4_type): New structures. (ped_file_system_ext2_init): Register Ext4 file system type. (ped_file_system_ext2_done): Likewise. * tests/Makefile.am (TESTS): Add t1700-ext-probe.sh. * tests/t1700-ext-probe.sh: New file. --- libparted/fs/ext2/ext2_fs.h | 2 + libparted/fs/ext2/interface.c | 61 +++++++++++++++++++++++++++++++++++----- tests/Makefile.am | 1 + tests/t1700-ext-probe.sh | 54 ++++++++++++++++++++++++++++++++++++ 4 files changed, 110 insertions(+), 8 deletions(-) create mode 100755 tests/t1700-ext-probe.sh diff --git a/libparted/fs/ext2/ext2_fs.h b/libparted/fs/ext2/ext2_fs.h index 713cc21..08d0c38 100644 --- a/libparted/fs/ext2/ext2_fs.h +++ b/libparted/fs/ext2/ext2_fs.h @@ -59,6 +59,8 @@ #define EXT2_FEATURE_INCOMPAT_FILETYPE 0x0002 #define EXT3_FEATURE_INCOMPAT_RECOVER 0x0004 +#define EXT4_FEATURE_INCOMPAT_EXTENTS 0x0040 +#define EXT4_FEATURE_INCOMPAT_64BIT 0x0080 /* * Special inodes numbers diff --git a/libparted/fs/ext2/interface.c b/libparted/fs/ext2/interface.c index 8935840..80ae9bd 100644 --- a/libparted/fs/ext2/interface.c +++ b/libparted/fs/ext2/interface.c @@ -32,7 +32,7 @@ static PedFileSystemType _ext3_type; struct ext2_dev_handle* ext2_make_dev_handle_from_parted_geometry(PedGeometry* geom); static PedGeometry* -_ext2_generic_probe (PedGeometry* geom, int expect_ext3) +_ext2_generic_probe (PedGeometry* geom, int expect_ext_ver) { struct ext2_super_block sb; @@ -46,10 +46,25 @@ _ext2_generic_probe (PedGeometry* geom, int expect_ext3) PedSector group_nr = EXT2_SUPER_BLOCK_GROUP_NR(sb); PedSector first_data_block = EXT2_SUPER_FIRST_DATA_BLOCK(sb); int version = EXT2_SUPER_REV_LEVEL(sb); - int is_ext3 = (EXT2_SUPER_FEATURE_COMPAT(sb) - & EXT3_FEATURE_COMPAT_HAS_JOURNAL) != 0; - - if (expect_ext3 != is_ext3) + int is_ext3 = 0; + int is_ext4 = 0; + + is_ext3 = (EXT2_SUPER_FEATURE_COMPAT (sb) + & EXT3_FEATURE_COMPAT_HAS_JOURNAL) != 0; + if (is_ext3) { + is_ext4 + = is_ext3 + && ((EXT2_SUPER_FEATURE_INCOMPAT (sb) + & EXT4_FEATURE_INCOMPAT_EXTENTS) != 0); + if (is_ext4) + is_ext3 = 0; + } + + if (expect_ext_ver == 2 && (is_ext3 || is_ext4)) + return NULL; + if (expect_ext_ver == 3 && !is_ext3) + return NULL; + else if (expect_ext_ver == 4 && !is_ext4) return NULL; if (version > 0 && group_nr > 0) { @@ -64,7 +79,8 @@ _ext2_generic_probe (PedGeometry* geom, int expect_ext3) return NULL; ped_geometry_init (&probe_geom, geom->dev, start, block_count * block_size); - return _ext2_generic_probe (&probe_geom, expect_ext3); + return _ext2_generic_probe (&probe_geom, + expect_ext_ver); } else { return ped_geometry_new (geom->dev, geom->start, block_count * block_size); @@ -76,13 +92,19 @@ _ext2_generic_probe (PedGeometry* geom, int expect_ext3) static PedGeometry* _ext2_probe (PedGeometry* geom) { - return _ext2_generic_probe (geom, 0); + return _ext2_generic_probe (geom, 2); } static PedGeometry* _ext3_probe (PedGeometry* geom) { - return _ext2_generic_probe (geom, 1); + return _ext2_generic_probe (geom, 3); +} + +static PedGeometry* +_ext4_probe (PedGeometry* geom) +{ + return _ext2_generic_probe (geom, 4); } #ifndef DISCOVER_ONLY @@ -323,6 +345,20 @@ static PedFileSystemOps _ext3_ops = { #endif /* !DISCOVER_ONLY */ }; +static PedFileSystemOps _ext4_ops = { + probe: _ext4_probe, + clobber: NULL, + open: NULL, + create: NULL, + close: NULL, + check: NULL, + resize: NULL, + copy: NULL, + get_create_constraint: NULL, + get_copy_constraint: NULL, + get_resize_constraint: NULL +}; + #define EXT23_BLOCK_SIZES ((int[6]){512, 1024, 2048, 4096, 8192, 0}) static PedFileSystemType _ext2_type = { @@ -339,14 +375,23 @@ static PedFileSystemType _ext3_type = { block_sizes: EXT23_BLOCK_SIZES }; +static PedFileSystemType _ext4_type = { + next: NULL, + ops: &_ext4_ops, + name: "ext4", + block_sizes: EXT23_BLOCK_SIZES +}; + void ped_file_system_ext2_init () { ped_file_system_type_register (&_ext2_type); ped_file_system_type_register (&_ext3_type); + ped_file_system_type_register (&_ext4_type); } void ped_file_system_ext2_done () { ped_file_system_type_unregister (&_ext2_type); ped_file_system_type_unregister (&_ext3_type); + ped_file_system_type_unregister (&_ext4_type); } diff --git a/tests/Makefile.am b/tests/Makefile.am index ab3b7cb..1214f9c 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -4,6 +4,7 @@ TESTS = \ t1000-mkpartfs.sh \ t1100-busy-label.sh \ t1500-small-ext2.sh \ + t1700-ext-probe.sh \ t2000-mkfs.sh \ t2100-mkswap.sh \ t2200-dos-label-recog.sh \ diff --git a/tests/t1700-ext-probe.sh b/tests/t1700-ext-probe.sh new file mode 100755 index 0000000..035753c --- /dev/null +++ b/tests/t1700-ext-probe.sh @@ -0,0 +1,54 @@ +#!/bin/sh + +# Copyright (C) 2008 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. + +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. + +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. + +test_description='Probe Ext2, Ext3 and Ext4 file systems.' + +: ${srcdir=.} +. $srcdir/test-lib.sh + +dev=loop-file +test_expect_success \ + 'setup' ' + dd if=/dev/zero of=$dev bs=1024 count=4096> /dev/null && + mkfs -F -t ext2 $dev >/dev/null' + +test_expect_success \ + 'probe the ext2 file system' ' + parted -s $dev print >out 2>&1 + grep -w "ext2" out' + +test_expect_success \ + 'setup' ' + dd if=/dev/zero of=$dev bs=1024 count=4096> /dev/null && + mkfs -F -t ext3 $dev >/dev/null' + +test_expect_success \ + 'probe the ext3 file system' ' + parted -s $dev print >out 2>&1 + grep -w "ext3" out' + +test_expect_success \ + 'setup' ' + dd if=/dev/zero of=$dev bs=1024 count=4096> /dev/null && + mkfs -F -t ext4 $dev >/dev/null' + +test_expect_success \ + 'probe the ext4 file system' ' + parted -s $dev print >out 2>&1 + grep -w "ext4" out' + +test_done -- 1.6.0.6 Comments? Happy hacking, Debarshi [1] http://rishi.fedorapeople.org/gnu/0001-Differentiate-between-Ext4-and-Ext3-file-systems.-Fi.patch _______________________________________________ parted-devel mailing list [email protected] http://lists.alioth.debian.org/mailman/listinfo/parted-devel
