Re: [f2fs-dev] [PATCH 1/2] f2fs: replace __get_victim by dirty_segments in FG_GC

2017-02-23 Thread Chao Yu
On 2017/2/18 3:15, Jaegeuk Kim wrote: > Hi Yunlei, > > I also removed "segno = NULL_SEGNO;" in the beginning part near gc_more. > > Thanks, > > On 02/17, Yunlei He wrote: >> In FG_GC process, it will search victim section twice. This will >> cause some dirty section with less valid blocks skip

Re: [f2fs-dev] [PATCH 1/2] f2fs: replace __get_victim by dirty_segments in FG_GC

2017-02-17 Thread Jaegeuk Kim
Hi Yunlei, I also removed "segno = NULL_SEGNO;" in the beginning part near gc_more. Thanks, On 02/17, Yunlei He wrote: > In FG_GC process, it will search victim section twice. This will > cause some dirty section with less valid blocks skip garbage > collection. > > section # 26425 : valid

[f2fs-dev] [PATCH 1/2] f2fs: replace __get_victim by dirty_segments in FG_GC

2017-02-17 Thread Yunlei He
In FG_GC process, it will search victim section twice. This will cause some dirty section with less valid blocks skip garbage collection. section # 26425 : valid blocks # 3 142.037567: get_victim_by_default: victim 26425 : valid blocks # 3 142.037585: f2fs_get_victim: dev = (259,30), type = No