On 2019/7/16 17:27, Chao Yu wrote:
> Hi Xiang,
>
> On 2019/7/16 17:12, Gao Xiang wrote:
>> Hi Chao,
>>
>> On 2019/7/16 16:52, Chao Yu wrote:
>>> Use __GFP_NOFAIL to avoid opened loop codes in z_erofs_vle_unzip().
>>>
>>> Signed-off-by: Chao Yu <yuch...@huawei.com>
>>> ---
>>> drivers/staging/erofs/unzip_vle.c | 17 ++++++++---------
>>> 1 file changed, 8 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/drivers/staging/erofs/unzip_vle.c
>>> b/drivers/staging/erofs/unzip_vle.c
>>> index f0dab81ff816..3a0dbcb8cc9f 100644
>>> --- a/drivers/staging/erofs/unzip_vle.c
>>> +++ b/drivers/staging/erofs/unzip_vle.c
>>> @@ -921,18 +921,17 @@ static int z_erofs_vle_unzip(struct super_block *sb,
>>> mutex_trylock(&z_pagemap_global_lock))
>>> pages = z_pagemap_global;
>>> else {
>>> -repeat:
>>> - pages = kvmalloc_array(nr_pages, sizeof(struct page *),
>>> - GFP_KERNEL);
>>> + gfp_t flags = GFP_KERNEL;
>>> +
>>> + if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES)
>>> + flags |= __GFP_NOFAIL;
>>> +
>>> + pages = kvmalloc_array(nr_pages, sizeof(struct page *), flags);
>>
>> How about omitting variable `flags' since it's only used once, or
>> rename it since `flags' is too general? some thoughts?
>
> That's minor thing, if you do care about this, I guess 'gfp_flags' maybe?
Be better, I prefer to omitting this variable, gfp_flags looks good as well.
>
>>
>> BTW, This piece of code has been changed in
>> "[PATCH v2 00/24] erofs: promote erofs from staging", I will sync the code
>> after some guys takes a look at v2....
>
> We have enough time to merge modified staging codes to generic fs one till
> next
> merge window, so don't worry, you can do the merge in batch.
Yes, you are right. Actually I am not too worry about the staging merges but
how to catch Viro's eye (sigh...)...
Thanks,
Gao Xiang
>
> Thanks,
>
>>
>> Thanks,
>> Gao Xiang
>>
>>>
>>> /* fallback to global pagemap for the lowmem scenario */
>>> if (unlikely(!pages)) {
>>> - if (nr_pages > Z_EROFS_VLE_VMAP_GLOBAL_PAGES)
>>> - goto repeat;
>>> - else {
>>> - mutex_lock(&z_pagemap_global_lock);
>>> - pages = z_pagemap_global;
>>> - }
>>> + mutex_lock(&z_pagemap_global_lock);
>>> + pages = z_pagemap_global;
>>> }
>>> }
>>>
>>>
>> .
>>
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
