Re: [PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-18 Thread Chao Yu
Hi Yunlong, I think you have changed original implication of the function, IMO, it would be more accurate to use user_free_segment_blocks instead of free_user_blocks. Thanks, On 2017/8/18 18:02, Yunlong Song wrote: > ping... > > On 2017/8/15 15:14, Yunlong Song wrote: >> The part

Re: [PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-18 Thread Chao Yu
Hi Yunlong, I think you have changed original implication of the function, IMO, it would be more accurate to use user_free_segment_blocks instead of free_user_blocks. Thanks, On 2017/8/18 18:02, Yunlong Song wrote: > ping... > > On 2017/8/15 15:14, Yunlong Song wrote: >> The part

Re: [PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-18 Thread Yunlong Song
ping... On 2017/8/15 15:14, Yunlong Song wrote: The part (overprovision_segments - reserved_segments) can still be used for LFS in some case, e.g., there are lots of invalid block from dirty segments, then the part (overprovision_segments - reserved_segments) can be safely used. So

Re: [PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-18 Thread Yunlong Song
ping... On 2017/8/15 15:14, Yunlong Song wrote: The part (overprovision_segments - reserved_segments) can still be used for LFS in some case, e.g., there are lots of invalid block from dirty segments, then the part (overprovision_segments - reserved_segments) can be safely used. So

[PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-15 Thread Yunlong Song
The part (overprovision_segments - reserved_segments) can still be used for LFS in some case, e.g., there are lots of invalid block from dirty segments, then the part (overprovision_segments - reserved_segments) can be safely used. So free_use_blocks should use reserved_segments instead, rather

[PATCH v2] f2fs: free_user_blocks should use reserved_segments instead

2017-08-15 Thread Yunlong Song
The part (overprovision_segments - reserved_segments) can still be used for LFS in some case, e.g., there are lots of invalid block from dirty segments, then the part (overprovision_segments - reserved_segments) can be safely used. So free_use_blocks should use reserved_segments instead, rather