On 2010-11-26, at 01:01, Lukas Czerner wrote: > On Thu, 25 Nov 2010, Andreas Dilger wrote: >> One question I have is why a change like this is being done in ext3 instead of only in ext4? There are a lot of ext4 features that _could_ be included in ext3 (basically all of them), but the request from the rest of the kernel developers was to leave ext3 with minimal changes (for continued stability), and put all of the major changes into ext4. > > You're right that ext3 should be "closed" for major (intrusive) changes, > however this is not a major change at all. It interacts with ext3 code > just very little and it is very well separated. Basically, it is dead > code until FITRIM ioctl is done. I don't want to pick on this patch, per se, but if someone is installing a new filesystem on an SSD with a new kernel, they could just as easily use ext4 for that instead of ext3, and they likely should use ext4 because of performance. Either they are using ext3 because they don't want any changes (in which case they also don't want this one), or if it is a new filesystem on a new device they can as easily use ext4. Since this code is only going to end up in new kernels, this isn't about what users are using for legacy systems where only ext3 is available. Cheers, Andreas -- 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