On Tue 15-05-12 00:33:52, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 15.05.2012 00:19, Jan Kara wrote: > > > On Sun 13-05-12 15:48:55, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > >> minixfs file size check is buggy and it doesn't allow creating a block which > >> can't be fully filled > > Umm, I'm not really minix expert but who'd set s_max_size to something > > which is not a multiple of block size? This looks rather artifical problem > > to me... > > > > The usual and natural limit comes from interpreting 32-bit size field as > signed or unsigned. So it's either 2G - 1 or 4G - 1. Neither of which is > a multiple of block size. Oh, right. Then your patch should be OK, just it's enough to cast one of the arguments to u64. And BTW looking at minix, it should also set s_maxbytes to s_max_size. Otherwise it will be always limited by MAX_NON_LFS which is 2^31-1. Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- 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
