> Namjae Jeon <namjae.jeon@xxxxxxxxxxx> writes: > > > This patch set provides support for doing fallocate operation > > on FAT filesystem. > > I'm still not reviewing deeply though, basically looks like good to > me. But I'm still not testing this one at all, sorry. > > If not tested yet, we would be able to start test, e.g. stress. Hi, OGAWA. Okay, I will share stress test result(per sector size) using fsx, fsstress, etc. And will fix the review points of 2/4, 3/4 patch also. Thanks for review! > > Thanks. > > > v6: > > - Remove usage of i_disksize > > - Align all fallocate code sequence to use i_blocks as > > this is more or less can be used to denote the on-disksize. > > Also, i_blocks is common in update path of cluster allocation via. > > fat_chain_add() both from normal write path and fallocate() operation > > > > v5: > > - change all routines to make ->i_disksize aligned by cluster size. > > - merge fat_bmap2() and fat_get_block_bmap(). > > > > v4: > > - ->i_disksize is aligned by block size in fill_inode. > > - use i_disksize when computing the number of cluster for fallocate. > > - make fat_get_block_bmap to avoid race issue. > > - use __fat_write_inode() with inode_needs_sync() in evict. > > > > v3: > > - fix i_disksize and mmu_private comments(uninit/initialized) > > - remove unneeded condition in fat_bmap. > > - use mmu_private instead of fclus in fat_fallocate. > > - make function mark buffer new for block in fallocated region. > > - use flags for bmap. > > - use mmu_private instead of i_size in direct_IO. > > - use flags and remove i_mutex in bmap. > > - update vfat documentation. > > > > v2: > > - remove a stary unlock(from Dan Carpenter) > > - check ->mmu_private and ->i_disksize to truncate uninitialized data. > > > > Namjae Jeon (4): > > fat: add fat_fallocate operation > > fat: skip cluster allocation on fallocated region > > fat: permit to return phy block number by fibmap in fallocated region > > Documentation/filesystems/vfat.txt: update the limitation for fat > > fallocate > > -- > OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> -- 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
