It's fine unless create_discard_cmd_control is used in remount flow in the future.
On 2017/7/6 21:16, Chao Yu wrote: > Hi Yunlong, > > It looks there is no way to create discard thread redundantly, > so here we don't need to check this? > > Thanks, > > On 2017/7/6 19:05, Yunlong Song wrote: >> Signed-off-by: Yunlong Song <[email protected]> >> --- >> fs/f2fs/segment.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c >> index 4c246e3..b48d004 100644 >> --- a/fs/f2fs/segment.c >> +++ b/fs/f2fs/segment.c >> @@ -1417,6 +1417,8 @@ static int create_discard_cmd_control(struct >> f2fs_sb_info *sbi) >> >> if (SM_I(sbi)->dcc_info) { >> dcc = SM_I(sbi)->dcc_info; >> + if (dcc->f2fs_issue_discard) >> + return err; >> goto init_thread; >> } >> >> > . > -- Thanks, Yunlong Song ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
