On Mon, 15 Aug 2011, Jan Kara wrote: > On Mon 15-08-11 17:50:53, Lukas Czerner wrote: > > On Mon, 15 Aug 2011, Jan Kara wrote: > > > On Mon 15-08-11 16:21:27, Lukas Czerner wrote: > > > > On Thu, 11 Aug 2011, Andreas Dilger wrote: > > > > > On 2011-08-11, at 8:58 AM, Lukas Czerner wrote: > > > > > > On Tue, 7 Jun 2011, Lukas Czerner wrote: > > > > > >> For a long time now orlov is the default block allocator in the ext4. It > > > > > >> performs better than the old one and no one seems to claim otherwise so > > > > > >> we can safely drop it and make oldalloc and orlov mount option > > > > > >> deprecated. > > > > > >> > > > > > >> This is a part of the effort to reduce number of ext4 options hence the > > > > > >> test matrix. > > > > > >> > > > > > >> Signed-off-by: Lukas Czerner <lczerner@xxxxxxxxxx> > > > > > > > > > > > > ping > > > > > > > > > > I'm OK with removing this, I don't think anyone uses it, and it has almost > > > > > no meaning with flex_bg anyway. > > > > > > > > > > That said, "orlov" is also mostly meaningless with flex_bg as well, since > > > > > there is very little real benefit/affinity from inodes being "close" to > > > > > their data blocks. We gain far more benefit from keeping the inodes > > > > > together than spreading them out and keeping them close to the data blocks. > > > > > > > > What about removing it for ext3 as well ? I can prepare a patch. > > > OK, let's start warning the option is deprecated and will be removed from > > > ext3. We can remove it after 2-3 releases... > > > > Is that really necessary ? It is not like we are removing a feature which > > would not work anymore. > Well, we are removing a mount option so if someone is using it e.g. in > /etc/fstab, his machine will fail to mount the filesystem. Or did I > misunderstood your intention? It will not fail to boot, see the patch at the beginning of the thread. It will just print KERN_WARNING that we are ignoring this option. Since we are not removing a feature that will be missing, we can do that. Also note that this is the same thing what we have done to nobh option, and there are probably even other examples. Thanks! -Lukas > > Honza > -- 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
