Re: [PATCH 2/2] ext4: Spot-check block group sub-table locations

[Theodore Ts'o <tytso@mit.edu>]

On Wed, Oct 09, 2013 at 03:08:24PM -0700, Darrick J. Wong wrote:
> > FWIW, I notice that in ext4_check_descriptors() we also check the
> > location of bitmap.  So maybe we could use ext4_is_sane_bgdata_location
> > in that function.
> 
> That could be done, but all we'd be doing is trading this:
> 
> 	block_bitmap = ext4_block_bitmap(sb, gdp);
> 	if (block_bitmap < first_block || block_bitmap > last_block) {
> 
> for this slightly slower code:
> 
> 	if (ext4_is_sane_bgdata_location(sb, gdp,
> 					 ext4_block_bitmap(sb, gdb))) {
> 
> I don't know if that's worth the code churn or not.  It's an easy change, but I
> don't think it adds much.  Ted -- if you want me to make the change I'll do it,
> but if you're content to leave the patch as it is, then so am I.

The question I have is if we are checking to make sure all of the
block and inode bitmaps are sane at mount time, how much is it worth
it to check to make sure that they are sane each time we need to load
the bitmap?  This is especially true since for ext4 file systems with
flex_bg, the only real check we can do is to is whether the block
number is out of range, which will almost certainly cause a device
error.

						- Ted