diff mbox

ext4: Update EXT4_USE_FOR_EXT2 description

Message ID 20150924123822.1a0917bf@endymion.delvare
State Accepted, archived
Headers show

Commit Message

Jean Delvare Sept. 24, 2015, 10:38 a.m. UTC
Configuration option EXT4_USE_FOR_EXT2 has no effect on ext3 support.
Support for ext3 is always included now.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Fixes: c290ea01ab ("fs: Remove ext3 filesystem driver")
Cc: Jan Kara <jack@suse.cz>
Cc: Theodore Ts'o <tytso@mit.edu>
Cc: Andreas Dilger <adilger.kernel@dilger.ca>
---
 fs/ext4/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jan Kara Sept. 24, 2015, 11:28 a.m. UTC | #1
On Thu 24-09-15 12:38:22, Jean Delvare wrote:
> Configuration option EXT4_USE_FOR_EXT2 has no effect on ext3 support.
> Support for ext3 is always included now.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Fixes: c290ea01ab ("fs: Remove ext3 filesystem driver")
> Cc: Jan Kara <jack@suse.cz>
> Cc: Theodore Ts'o <tytso@mit.edu>
> Cc: Andreas Dilger <adilger.kernel@dilger.ca>

Thanks. I've merged the patch to my tree.

								Honza

> ---
>  fs/ext4/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-4.3-rc2.orig/fs/ext4/Kconfig	2015-09-20 23:32:34.000000000 +0200
> +++ linux-4.3-rc2/fs/ext4/Kconfig	2015-09-24 11:45:21.911961421 +0200
> @@ -63,7 +63,7 @@ config EXT4_FS
>  	  If unsure, say N.
>  
>  config EXT4_USE_FOR_EXT2
> -	bool "Use ext4 for ext2/ext3 file systems"
> +	bool "Use ext4 for ext2 file systems"
>  	depends on EXT4_FS
>  	depends on EXT2_FS=n
>  	default y
> 
> 
> -- 
> Jean Delvare
> SUSE L3 Support
Jean Delvare Sept. 24, 2015, 1:14 p.m. UTC | #2
Le Thursday 24 September 2015 à 13:28 +0200, Jan Kara a écrit :
> On Thu 24-09-15 12:38:22, Jean Delvare wrote:
> > Configuration option EXT4_USE_FOR_EXT2 has no effect on ext3 support.
> > Support for ext3 is always included now.
> > 
> > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > Fixes: c290ea01ab ("fs: Remove ext3 filesystem driver")
> > Cc: Jan Kara <jack@suse.cz>
> > Cc: Theodore Ts'o <tytso@mit.edu>
> > Cc: Andreas Dilger <adilger.kernel@dilger.ca>
> 
> Thanks. I've merged the patch to my tree.
> 
> 								Honza
> 
> > ---
> >  fs/ext4/Kconfig |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > --- linux-4.3-rc2.orig/fs/ext4/Kconfig	2015-09-20 23:32:34.000000000 +0200
> > +++ linux-4.3-rc2/fs/ext4/Kconfig	2015-09-24 11:45:21.911961421 +0200
> > @@ -63,7 +63,7 @@ config EXT4_FS
> >  	  If unsure, say N.
> >  
> >  config EXT4_USE_FOR_EXT2
> > -	bool "Use ext4 for ext2/ext3 file systems"
> > +	bool "Use ext4 for ext2 file systems"

On second thought I'm not sure if the use of plural is still desired.
You decide ;-)

> >  	depends on EXT4_FS
> >  	depends on EXT2_FS=n
> >  	default y
> > 

Thanks,
diff mbox

Patch

--- linux-4.3-rc2.orig/fs/ext4/Kconfig	2015-09-20 23:32:34.000000000 +0200
+++ linux-4.3-rc2/fs/ext4/Kconfig	2015-09-24 11:45:21.911961421 +0200
@@ -63,7 +63,7 @@  config EXT4_FS
 	  If unsure, say N.
 
 config EXT4_USE_FOR_EXT2
-	bool "Use ext4 for ext2/ext3 file systems"
+	bool "Use ext4 for ext2 file systems"
 	depends on EXT4_FS
 	depends on EXT2_FS=n
 	default y