Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-13 Thread Daeho Jeong
> Actually, we didn't change priority of discard command, so that it is still > synchronous IO for I/O scheduler, hence I/O interference will still exist if > we > try to issue discard without IO aware ability. > Of course we can change the priority of discard command to lower, but > potential

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-13 Thread Chao Yu
On 2017/9/12 12:34, Daeho Jeong wrote: >> Yeah, that's exactly like what I made a mistake before. >> I should have mentioned that earlier. :) > > Or I think the previous code which used "iter++" might be right. > You might just want to check the fixed number of small discards, > DISCARD_ISSUE_RAT

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-12 Thread Jaegeuk Kim
On 09/12, Daeho Jeong wrote: > > Yeah, that's exactly like what I made a mistake before. > > I should have mentioned that earlier. :) > > Or I think the previous code which used "iter++" might be right. > You might just want to check the fixed number of small discards, > DISCARD_ISSUE_RATE, > whe

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-11 Thread Daeho Jeong
> Yeah, that's exactly like what I made a mistake before. > I should have mentioned that earlier. :) Or I think the previous code which used "iter++" might be right. You might just want to check the fixed number of small discards, DISCARD_ISSUE_RATE, when issue_cond is "true". Anyways, I have an

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-11 Thread Jaegeuk Kim
On 09/12, Chao Yu wrote: > On 2017/9/12 9:53, Chao Yu wrote: > > On 2017/9/11 11:38, Jaegeuk Kim wrote: > >> If issue_cond is true, it does double count for # of issued commands. > >> > >> Signed-off-by: Jaegeuk Kim > > > > Reviewed-by: Chao Yu > > As Daeho Jeong mentioned, the change makes 'it

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-11 Thread Chao Yu
On 2017/9/12 9:53, Chao Yu wrote: > On 2017/9/11 11:38, Jaegeuk Kim wrote: >> If issue_cond is true, it does double count for # of issued commands. >> >> Signed-off-by: Jaegeuk Kim > > Reviewed-by: Chao Yu As Daeho Jeong mentioned, the change makes 'iter > DISCARD_ISSUE_RATE' dead code, I just

Re: [f2fs-dev] [PATCH] f2fs: fix double count on issued discard commands

2017-09-11 Thread Chao Yu
On 2017/9/11 11:38, Jaegeuk Kim wrote: > If issue_cond is true, it does double count for # of issued commands. > > Signed-off-by: Jaegeuk Kim Reviewed-by: Chao Yu > --- > fs/f2fs/segment.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/f2fs/segment.c b/fs/f2fs/se