Re: [f2fs-dev] [PATCH v2 1/2] f2fs: introduce checkpoint=merge mount option

2021-01-15 Thread Chao Yu
On 2021/1/15 22:00, Daeho Jeong wrote: ktime_get() returns time based ns unit, in extreme scenario, average time cp cost will overflow 32-bit variable, I doubt. sum_diff is already turned into msec using ktime_ms_delta() above. Yup, I missed ktime_ms_delta(). On 2021/1/15 22:23, Daeho

Re: [f2fs-dev] [PATCH v2 1/2] f2fs: introduce checkpoint=merge mount option

2021-01-15 Thread Daeho Jeong
2021년 1월 15일 (금) 오후 11:00, Daeho Jeong 님이 작성: > > 2021년 1월 15일 (금) 오후 6:22, Chao Yu 님이 작성: > > > > On 2021/1/14 14:23, Daeho Jeong wrote: > > > From: Daeho Jeong > > > > > > We've added a new mount option "checkpoint=merge", which creates a > > > kernel daemon and makes it to merge concurrent

Re: [f2fs-dev] [PATCH v2 1/2] f2fs: introduce checkpoint=merge mount option

2021-01-15 Thread Daeho Jeong
2021년 1월 15일 (금) 오후 6:22, Chao Yu 님이 작성: > > On 2021/1/14 14:23, Daeho Jeong wrote: > > From: Daeho Jeong > > > > We've added a new mount option "checkpoint=merge", which creates a > > kernel daemon and makes it to merge concurrent checkpoint requests as > > much as possible to eliminate

Re: [f2fs-dev] [PATCH v2 1/2] f2fs: introduce checkpoint=merge mount option

2021-01-15 Thread Chao Yu
On 2021/1/14 14:23, Daeho Jeong wrote: From: Daeho Jeong We've added a new mount option "checkpoint=merge", which creates a kernel daemon and makes it to merge concurrent checkpoint requests as much as possible to eliminate redundant checkpoint issues. Plus, we can eliminate the sluggish issue

[PATCH v2 1/2] f2fs: introduce checkpoint=merge mount option

2021-01-13 Thread Daeho Jeong
From: Daeho Jeong We've added a new mount option "checkpoint=merge", which creates a kernel daemon and makes it to merge concurrent checkpoint requests as much as possible to eliminate redundant checkpoint issues. Plus, we can eliminate the sluggish issue caused by slow checkpoint operation when