Re: [PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-09 Thread Chao Yu
On 2017/11/10 1:51, Jaegeuk Kim wrote: > Hi Chao, > > On 11/07, Chao Yu wrote: >> We will keep __add_ino_entry success all the time, for ENOMEM failure >> case, we have already handled it by using __GFP_NOFAIL flag, so we >> don't have to use additional opened loop codes here, remove them. >>

Re: [PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-09 Thread Chao Yu
On 2017/11/10 1:51, Jaegeuk Kim wrote: > Hi Chao, > > On 11/07, Chao Yu wrote: >> We will keep __add_ino_entry success all the time, for ENOMEM failure >> case, we have already handled it by using __GFP_NOFAIL flag, so we >> don't have to use additional opened loop codes here, remove them. >>

Re: [PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-09 Thread Jaegeuk Kim
Hi Chao, On 11/07, Chao Yu wrote: > We will keep __add_ino_entry success all the time, for ENOMEM failure > case, we have already handled it by using __GFP_NOFAIL flag, so we > don't have to use additional opened loop codes here, remove them. > > Signed-off-by: Chao Yu >

Re: [PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-09 Thread Jaegeuk Kim
Hi Chao, On 11/07, Chao Yu wrote: > We will keep __add_ino_entry success all the time, for ENOMEM failure > case, we have already handled it by using __GFP_NOFAIL flag, so we > don't have to use additional opened loop codes here, remove them. > > Signed-off-by: Chao Yu > --- > v2: > As Michal

[PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-06 Thread Chao Yu
We will keep __add_ino_entry success all the time, for ENOMEM failure case, we have already handled it by using __GFP_NOFAIL flag, so we don't have to use additional opened loop codes here, remove them. Signed-off-by: Chao Yu --- v2: As Michal Hocko suggested, with

[PATCH v2 1/3] f2fs: avoid opened loop codes in __add_ino_entry

2017-11-06 Thread Chao Yu
We will keep __add_ino_entry success all the time, for ENOMEM failure case, we have already handled it by using __GFP_NOFAIL flag, so we don't have to use additional opened loop codes here, remove them. Signed-off-by: Chao Yu --- v2: As Michal Hocko suggested, with __GFP_NOFAIL, MM will do all