[PATCH] ext4/super.c: Fix error in ext4_fill_super: ext3_fs_type undeclared (first use in this function)

[PATCH] ext4/super.c: Fix error in ext4_fill_super: ext3_fs_type undeclared (first use in this function)

[Sedat Dilek]

[-- Attachment #1: Type: text/plain, Size: 122 bytes --]

Not sure if this is really fixing the problem in [1].

--
Sedat

[1] http://marc.info/?l=linux-ext4&m=126942870916446&w=2

[-- Attachment #2: 0001-ext4-super.c-Fix-error-in-ext4_fill_super-ext3_fs_ty.patch --]
[-- Type: text/x-diff, Size: 922 bytes --]

From 722b3ac46df769e5e759b6f824982d34e14037b4 Mon Sep 17 00:00:00 2001
From: Sedat Dilek <sedat.dilek@gmail.com>
Date: Wed, 24 Mar 2010 12:22:34 +0100
Subject: [PATCH] ext4/super.c: Fix error in ext4_fill_super: ext3_fs_type undeclared (first use in this function)

---
 fs/ext4/super.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/fs/ext4/super.c b/fs/ext4/super.c
index 1dbb37a..2dacc82 100644
--- a/fs/ext4/super.c
+++ b/fs/ext4/super.c
@@ -2550,7 +2550,9 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent)
 	 * enable delayed allocation by default
 	 * Use -o nodelalloc to turn it off
 	 */
+#if !defined(CONFIG_EXT3_FS) && !defined(CONFIG_EXT3_FS_MODULE) && defined(CONFIG_EXT4_USE_FOR_EXT23)	
 	if (sb->s_bdev->bd_holder != &ext3_fs_type)
+#endif
 		set_opt(sbi->s_mount_opt, DELALLOC);
 
 	if (!parse_options((char *) data, sb, &journal_devnum,
-- 
1.7.0.3

Re: [PATCH] ext4/super.c: Fix error in ext4_fill_super: ext3_fs_type undeclared (first use in this function)

[tytso]

On Wed, Mar 24, 2010 at 12:45:27PM +0100, Sedat Dilek wrote:
> Not sure if this is really fixing the problem in [1].

Yeah, that should fix it, but see the patch I sent just a moment
ago... (sorry, I didn't see your e-mail until I checked in my fix...)

       	       	 	    	 	- Ted

Re: [PATCH] ext4/super.c: Fix error in ext4_fill_super: ext3_fs_type undeclared (first use in this function)

[Sedat Dilek]

Hehe.
Yes, I am telling people so often to look first into the ML(s) before
asking on IRC :-).
Thanks, anyway.

--
Sedat

On Wed, Mar 24, 2010 at 5:13 PM,  <tytso@mit.edu> wrote:
> On Wed, Mar 24, 2010 at 12:45:27PM +0100, Sedat Dilek wrote:
>> Not sure if this is really fixing the problem in [1].
>
> Yeah, that should fix it, but see the patch I sent just a moment
> ago... (sorry, I didn't see your e-mail until I checked in my fix...)
>
>                                        - Ted
>
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html