On 06/27/2013 07:21 AM, Jaegeuk Kim wrote: > Hi Gu, > > Could you check your email environment? > I can't merge your patch, since all the indentations consist of white > spaces. > And also, please check 80 characters limit per line. > Thanks,
OK, thanks for your reminder. Regards, Gu > > 2013-06-26 (수), 17:58 +0800, Gu Zheng: >> Destroy_fsync_dnodes() is a simple list-cleanup func, so delete the unused >> and unrelated f2fs_sb_info argument of it. >> >> >> Signed-off-by: Gu Zheng <[email protected]> >> --- >> fs/f2fs/recovery.c | 5 ++--- >> 1 files changed, 2 insertions(+), 3 deletions(-) >> >> diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c >> index 9db8239..d56d951 100644 >> --- a/fs/f2fs/recovery.c >> +++ b/fs/f2fs/recovery.c >> @@ -192,8 +192,7 @@ out: >> return err; >> } >> >> -static void destroy_fsync_dnodes(struct f2fs_sb_info *sbi, >> - struct list_head *head) >> +static void destroy_fsync_dnodes(struct list_head *head) >> { >> struct fsync_inode_entry *entry, *tmp; >> >> @@ -438,7 +437,7 @@ int recover_fsync_data(struct f2fs_sb_info *sbi) >> err = recover_data(sbi, &inode_list, CURSEG_WARM_NODE); >> BUG_ON(!list_empty(&inode_list)); >> out: >> - destroy_fsync_dnodes(sbi, &inode_list); >> + destroy_fsync_dnodes(&inode_list); >> kmem_cache_destroy(fsync_entry_slab); >> sbi->por_doing = 0; >> if (!err) > ------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
