Re: [f2fs-dev] [PATCH RESEND v3] f2fs: fix concurrent problem for updating free bitmap

2017-11-23 Thread Chao Yu
On 2017/11/22 16:07, LiFan wrote: > alloc_nid_failed and scan_nat_page can be called at the same time, > and we haven't protected add_free_nid and update_free_nid_bitmap > with the same nid_list_lock. That could lead to > > Thread A Thread B > - __build_free_nids > -

Re: [f2fs-dev] [PATCH RESEND v3] f2fs: fix concurrent problem for updating free bitmap

2017-11-23 Thread Chao Yu
On 2017/11/22 16:07, LiFan wrote: > alloc_nid_failed and scan_nat_page can be called at the same time, > and we haven't protected add_free_nid and update_free_nid_bitmap > with the same nid_list_lock. That could lead to > > Thread A Thread B > - __build_free_nids > -

[f2fs-dev] [PATCH RESEND v3] f2fs: fix concurrent problem for updating free bitmap

2017-11-22 Thread LiFan
alloc_nid_failed and scan_nat_page can be called at the same time, and we haven't protected add_free_nid and update_free_nid_bitmap with the same nid_list_lock. That could lead to Thread AThread B - __build_free_nids - scan_nat_page

[f2fs-dev] [PATCH RESEND v3] f2fs: fix concurrent problem for updating free bitmap

2017-11-22 Thread LiFan
alloc_nid_failed and scan_nat_page can be called at the same time, and we haven't protected add_free_nid and update_free_nid_bitmap with the same nid_list_lock. That could lead to Thread AThread B - __build_free_nids - scan_nat_page