Re: [PATCH] fs: quota: Replace GFP_ATOMIC with GFP_KERNEL in dquot_init

2018-04-09 Thread Jan Kara
On Mon 09-04-18 22:31:19, Jia-Ju Bai wrote: > dquot_init() is never called in atomic context. > This function is only set as a parameter of fs_initcall(). > > Despite never getting called from atomic context, > dquot_init() calls __get_free_pages() with GFP_ATOMIC, > which waits busily for

Re: [PATCH] fs: quota: Replace GFP_ATOMIC with GFP_KERNEL in dquot_init

2018-04-09 Thread Jan Kara
On Mon 09-04-18 22:31:19, Jia-Ju Bai wrote: > dquot_init() is never called in atomic context. > This function is only set as a parameter of fs_initcall(). > > Despite never getting called from atomic context, > dquot_init() calls __get_free_pages() with GFP_ATOMIC, > which waits busily for

[PATCH] fs: quota: Replace GFP_ATOMIC with GFP_KERNEL in dquot_init

2018-04-09 Thread Jia-Ju Bai
dquot_init() is never called in atomic context. This function is only set as a parameter of fs_initcall(). Despite never getting called from atomic context, dquot_init() calls __get_free_pages() with GFP_ATOMIC, which waits busily for allocation. GFP_ATOMIC is not necessary and can be replaced

[PATCH] fs: quota: Replace GFP_ATOMIC with GFP_KERNEL in dquot_init

2018-04-09 Thread Jia-Ju Bai
dquot_init() is never called in atomic context. This function is only set as a parameter of fs_initcall(). Despite never getting called from atomic context, dquot_init() calls __get_free_pages() with GFP_ATOMIC, which waits busily for allocation. GFP_ATOMIC is not necessary and can be replaced