On Sat, Apr 19, 2008 at 11:28:42PM +0200, Marcin Slusarz wrote: > > +static void set_block_shift(struct omfs_sb_info *sbi) > > +{ > > + unsigned int scale = sbi->s_blocksize / sbi->s_sys_blocksize; > > + sbi->s_block_shift = 0; > > + for (scale >>= 1; scale; scale >>= 1) > > + sbi->s_block_shift++; > > +} > isn't it get_bitmask_order(scale - 1)? I mean get_bitmask_order(scale >> 1) ;) Marcin -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html