On Wed 15-07-15 10:18:36, Ted Tso wrote: > On Wed, Jul 15, 2015 at 12:26:24PM +0200, Jan Kara wrote: > > > > so I have created this patch set which removes ext3 driver (and some related support > > code) from the kernel. See changelog of patch 2/3 for more details. If noone objects, > > I will queue the series in my tree for the next merge window. > > > > Jens, are you OK with me merging patch 3/3 or do you want me to send it to you once > > the first two patches get merged? > > > > Ted, this also touches ext4 driver so that it always handles ext3 filesystems so ack > > from you would be nice. > > Got tired of maintaining ext3, eh? :-) Not really. Maintaining ext3 has been pretty low-effort in last years. But the releasepage trouble has made me reconsider whether it's worth keeping ext3 codebase and I just came to conclusion that it may be the right time to drop it. > As a minor nit, I wonder if it's worth while to put in some hackery in > fs/Kconfig so that if someone only has CONFIG_FS_EXT3 defined, but not > CONFIG_FS_EXT4 defined, that it will automatically enable > CONFIG_FS_EXT4 instead? Otherwise someone upgrading their kernel > might accidentally end up getting a non-functional kernel. > > i.e., something like this? (not tested!) > > config FS_EXT3 > def_bool n > select FS_EXT4 Good point. Will add something like this, although the config will have to have some prompt for this to work... > In any case, since the 2/3 patch didn't hit the list, here's my: > > Acked-by: Theodore Ts'o <tytso@xxxxxxx> Thanks. Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
