aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChao Yu <yuchao0@huawei.com>2016-08-20 03:12:01 -0400
committerJaegeuk Kim <jaegeuk@kernel.org>2016-08-29 21:31:15 -0400
commitdfd02e4de1c5f40c268984254045d388ab0c3e74 (patch)
tree50a2895579364ef2748ccfa4eef770c824561839
parent6a7a3aedd562838fd402cccb5ad07b8063a3582e (diff)
f2fs: fix to preallocate block only aligned to 4K
In write_begin(), we skip checking dnode block for preallocating block when whole block needs to be updated since we preallocated its block in f2fs_preallocate_blocks, for partial updated block, we will still try to lock its node and do preallocation in write_begin(), so in f2fs_preallocate_blocks we should not preallocate its block. But previously, the calculation of preallocating block number is incorrect, fix it. Signed-off-by: Chao Yu <yuchao0@huawei.com> [Jaegeuk Kim: fix a bug] Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
-rw-r--r--fs/f2fs/data.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index ced6beb5debc..7c8e219f93dc 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -626,7 +626,12 @@ ssize_t f2fs_preallocate_blocks(struct kiocb *iocb, struct iov_iter *from)
626 ssize_t ret = 0; 626 ssize_t ret = 0;
627 627
628 map.m_lblk = F2FS_BLK_ALIGN(iocb->ki_pos); 628 map.m_lblk = F2FS_BLK_ALIGN(iocb->ki_pos);
629 map.m_len = F2FS_BYTES_TO_BLK(iov_iter_count(from)); 629 map.m_len = F2FS_BYTES_TO_BLK(iocb->ki_pos + iov_iter_count(from));
630 if (map.m_len > map.m_lblk)
631 map.m_len -= map.m_lblk;
632 else
633 map.m_len = 0;
634
630 map.m_next_pgofs = NULL; 635 map.m_next_pgofs = NULL;
631 636
632 if (f2fs_encrypted_inode(inode)) 637 if (f2fs_encrypted_inode(inode))
@@ -672,6 +677,9 @@ int f2fs_map_blocks(struct inode *inode, struct f2fs_map_blocks *map,
672 bool allocated = false; 677 bool allocated = false;
673 block_t blkaddr; 678 block_t blkaddr;
674 679
680 if (!maxblocks)
681 return 0;
682
675 map->m_len = 0; 683 map->m_len = 0;
676 map->m_flags = 0; 684 map->m_flags = 0;
677 685