On Fri, Feb 01, 2019 at 02:59:46PM +0800, Qing Xia wrote:

In the first loop, gfp_flags will be modified to high_order_gfp_flags, and there will be no chance to change back to low_order_gfp_flags.

Fixes: e7f63771 ("ION: Sys_heap: Add cached pool to spead up cached buffer alloc")

Huh... Presumably you found this bug just by reading the code. I wonder how it affects runtime?

regards, dan carpenter