Re: [f2fs-dev] [PATCH 1/2] f2fs: don't trigger recovery for normal umount image

2015-02-05 Thread Chao Yu
Hi Jaegeuk, -Original Message- From: Jaegeuk Kim [mailto:jaeg...@kernel.org] Sent: Tuesday, February 03, 2015 7:32 AM To: Chao Yu Cc: Changman Lee; linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org Subject: Re: [PATCH 1/2] f2fs: don't trigger recovery for normal

Re: [f2fs-dev] [PATCH 2/2] f2fs: don't discard next free dnode page for an umount checkpoint

2015-02-05 Thread Chao Yu
Hi Jaegeuk, -Original Message- From: Jaegeuk Kim [mailto:jaeg...@kernel.org] Sent: Tuesday, February 03, 2015 7:38 AM To: Chao Yu Cc: Changman Lee; linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org Subject: Re: [PATCH 2/2] f2fs: don't discard next free dnode

[f2fs-dev] [PATCH 6/9 v5] xfs/086, 087: remove specific testing options in output

2015-02-05 Thread Jaegeuk Kim
This patch removes the mkfs and mount options specified in output files in xfs/086 and xfs/087. Signed-off-by: Jaegeuk Kim jaeg...@kernel.org --- tests/xfs/086 | 2 +- tests/xfs/086.out | 20 tests/xfs/087 | 2 +- tests/xfs/087.out | 9 - 4 files changed,

Re: [f2fs-dev] [PATCH 2/2] f2fs: don't discard next free dnode page for an umount checkpoint

2015-02-05 Thread Jaegeuk Kim
Hi Chao, On Thu, Feb 05, 2015 at 04:16:04PM +0800, Chao Yu wrote: Hi Jaegeuk, -Original Message- From: Jaegeuk Kim [mailto:jaeg...@kernel.org] Sent: Tuesday, February 03, 2015 7:38 AM To: Chao Yu Cc: Changman Lee; linux-f2fs-devel@lists.sourceforge.net;

Re: [f2fs-dev] [PATCH 5/5 v4] f2fs: introduce a batched trim

2015-02-05 Thread Jaegeuk Kim
Hi Chao, Something like this? Change log from v3: o align to section size This patch introduces a batched trimming feature, which submits split discard commands. This is to avoid long latency due to huge trim commands. If fstrim was triggered ranging from 0 to the end of device, we should

[f2fs-dev] [PATCH 9/9 v5] tests/generic: relocate xfs's tests into tests/generic/

2015-02-05 Thread Jaegeuk Kim
This patch moves the generic testcases defined in xfs into tests/generic/. xfs/085 - generic/052 xfs/086 - generic/054 xfs/087 - generic/055 Signed-off-by: Jaegeuk Kim jaeg...@kernel.org --- tests/{xfs/085 = generic/052} | 6 +++--- tests/{xfs/085.out = generic/052.out} | 2 +-

[f2fs-dev] [PATCH 3/9 v5] common/rc: add _require_norecovery

2015-02-05 Thread Jaegeuk Kim
This patch adds checking code whether filesystem supports norecovery mount option or not. And let use this into the following xfs test. xfs/200 (recovery vs ro-block device) * Currently, norecovery mount option is used by xfs only. But some of log-based filesystems (e.g., f2fs) are able

[f2fs-dev] [PATCH 0/9 v5] make xfs/tests generic by introducing shutdown for filesystems

2015-02-05 Thread Jaegeuk Kim
Thanks to Dave. :) Change log from v4: o add mkfs in _require_scratch_shutdown o add _require_fiemap and _extent_hole_counts o add _require_logstate o modify xfs/086,087 output rules for the generic usage o add _get_log_configs to introduce consistent testing options o add _get_quota_option

[f2fs-dev] [PATCH 7/9 v5] common/log: add _get_log_configs for testing options

2015-02-05 Thread Jaegeuk Kim
This patch adds _get_log_configs for xfs and f2fs to test several mount options for: xfs/086 * xfs/087 In xfs/087, one more test was added, so 10 tests will be done in total. Signed-off-by: Jaegeuk Kim jaeg...@kernel.org --- common/log| 47

[f2fs-dev] [PATCH 1/9 v5] common/rc: add _require_scratch_shtudown

2015-02-05 Thread Jaegeuk Kim
This is to detect whether filesystem supports shutdown feature or not. And let use this into the following xfs tests. xfs/053 (data exposure) xfs/137 (data vs filesize) xfs/138 (data vs filesize vs truncate) xfs/139 (data vs filesize vs partial truncate)

Re: [f2fs-dev] [PATCH RESEND 1/2] mkfs.f2fs: support large sector size

2015-02-05 Thread Kinglong Mee
On Thu, Feb 5, 2015 at 5:36 PM, Chao Yu chao2...@samsung.com wrote: Since f2fs support large sector size in commit 55cf9cb63f0e f2fs: support large sector size, block device with sector size of 512/1024/2048/4096 bytes can be supported. But mkfs.f2fs still use default sector size: 512 bytes

[f2fs-dev] [PATCH v3 04/10] f2fs: introduce universal lookup/update interface for extent cache

2015-02-05 Thread Chao Yu
In this patch, we do these jobs: 1. rename {check,update}_extent_cache to {lookup,update}_extent_info; 2. introduce universal lookup/update interface of extent cache: f2fs_{lookup,update}_extent_cache including above two real functions, then export them to function callers. So after above

[f2fs-dev] [PATCH v3 10/10] f2fs: add trace for rb-tree extent cache ops

2015-02-05 Thread Chao Yu
This patch adds trace for lookup/update/shrink/destroy ops in rb-tree extent cache. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/data.c | 16 +- include/trace/events/f2fs.h | 134 2 files changed, 148 insertions(+), 2

[f2fs-dev] [PATCH v3 08/10] f2fs: enable rb-tree extent cache

2015-02-05 Thread Chao Yu
This patch enables rb-tree based extent cache in f2fs. When we mount with -o extent_cache, f2fs will try to add recently accessed page-block mappings into rb-tree based extent cache as much as possible, instead of original one extent info cache. By this way, f2fs can support more effective cache

[f2fs-dev] [PATCH v3 09/10] f2fs: show extent tree, node stat info in debugfs

2015-02-05 Thread Chao Yu
This patch add and show stat info of total memory footprint for extent tree,node in debugfs. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/debug.c | 7 +++ fs/f2fs/f2fs.h | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c index

[f2fs-dev] [PATCH 3/3] f2fs: use extent cache for dir

2015-02-05 Thread Chao Yu
We update extent cache for all user inode of f2fs including dir inode, so this patch gives another chance to try to get physical address of page from extent cache for dir inode. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/data.c | 14 ++ 1 file changed, 14 insertions(+)

Re: [f2fs-dev] [PATCH 5/5 v3] f2fs: introduce a batched trim

2015-02-05 Thread Chao Yu
Hi Jaegeuk, Changman, -Original Message- From: Jaegeuk Kim [mailto:jaeg...@kernel.org] Sent: Wednesday, February 04, 2015 4:11 AM To: Changman Lee Cc: Chao Yu; linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net Subject: Re:

Re: [f2fs-dev] [PATCH 1/2] mkfs.f2fs: support large sector size

2015-02-05 Thread Chao Yu
Hi Kinglong, -Original Message- From: Kinglong Mee [mailto:kinglong...@gmail.com] Sent: Wednesday, February 04, 2015 9:43 PM To: Chao Yu Cc: Jaegeuk Kim; Changman Lee; linux-ker...@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net Subject: Re: [f2fs-dev] [PATCH 1/2]

[f2fs-dev] [PATCH v3 01/10] f2fs: move ext_lock out of struct extent_info

2015-02-05 Thread Chao Yu
Move ext_lock out of struct extent_info, then in the following patches we can use variables with struct extent_info type as a parameter to pass pure data. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/data.c | 12 ++-- fs/f2fs/f2fs.h | 6 +- fs/f2fs/inode.c | 7 +++

[f2fs-dev] [PATCH v3 02/10] f2fs: simplfy a field name in struct f2fs_extent, extent_info

2015-02-05 Thread Chao Yu
Rename a filed name from 'blk_addr' to 'blk' in struct {f2fs_extent,extent_info} as annotation of this field descripts its meaning well to us. By this way, we can avoid long statement in code of following patches. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/data.c | 13

[f2fs-dev] [PATCH RESEND 1/2] mkfs.f2fs: support large sector size

2015-02-05 Thread Chao Yu
Since f2fs support large sector size in commit 55cf9cb63f0e f2fs: support large sector size, block device with sector size of 512/1024/2048/4096 bytes can be supported. But mkfs.f2fs still use default sector size: 512 bytes as sector size, let's fix this issue in this patch. v2: o remove

[f2fs-dev] [PATCH RESEND 2/2] fsck.f2fs: support large sector size

2015-02-05 Thread Chao Yu
Since f2fs support large sector size in commit 55cf9cb63f0e f2fs: support large sector size, block device with sector size of 512/1024/2048/4096 bytes can be supported. But fsck.f2fs still use default F2FS_LOG_SECTOR_SIZE/F2FS_LOG_SECTORS_PER_BLOCK to verify related data in f2fs image, it's

[f2fs-dev] [PATCH v3 03/10] f2fs: introduce f2fs_map_bh to clean codes of check_extent_cache

2015-02-05 Thread Chao Yu
This patch introduces f2fs_map_bh to clean codes of check_extent_cache. v2: o cleanup f2fs_map_bh pointed out by Jaegeuk Kim. Signed-off-by: Chao Yu chao2...@samsung.com --- fs/f2fs/data.c | 35 +-- 1 file changed, 21 insertions(+), 14 deletions(-) diff --git

[f2fs-dev] [PATCH 1/3] f2fs: support fast lookup in extent cache

2015-02-05 Thread Chao Yu
This patch adds a fast lookup path for rb-tree extent cache. In this patch we add a recently accessed extent node pointer 'cached_en' in extent tree. In lookup path of extent cache, we will firstly lookup the last accessed extent node which cached_en points, if we do not hit in this node, we will

[f2fs-dev] [PATCH v3 06/10] f2fs: add core functions for rb-tree extent cache

2015-02-05 Thread Chao Yu
This patch adds core functions including slab cache init function and init/lookup/update/shrink/destroy function for rb-tree based extent cache. Thank Jaegeuk Kim and Changman Lee as they gave much suggestion about detail design and implementation of extent cache. Todo: * register rb-based