Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752652AbbGOPFo (ORCPT ); Wed, 15 Jul 2015 11:05:44 -0400 Received: from casper.infradead.org ([85.118.1.10]:37775 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751918AbbGOPFm (ORCPT ); Wed, 15 Jul 2015 11:05:42 -0400 Message-ID: <55A676BF.8020903@infradead.org> Date: Wed, 15 Jul 2015 08:05:35 -0700 From: Randy Dunlap User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: "Theodore Ts'o" , Jan Kara , linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org, LKML , Andrew Morton , Andreas Dilger , Jens Axboe Subject: Re: [PATCH 0/3] Remove ext3 filesystem driver References: <1436955987-7305-1-git-send-email-jack@suse.com> <20150715141836.GA18400@thunk.org> In-Reply-To: <20150715141836.GA18400@thunk.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1437 Lines: 39 On 07/15/15 07:18, Theodore Ts'o 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? :-) > > 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 Yes, if someone is going to force us to use ext4, some automation would be helpful. Thanks, -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/