diff options
| author | Qi Zheng <zhengqi.arch@bytedance.com> | 2022-11-18 13:00:11 +0300 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2022-11-23 05:50:44 +0300 |
| commit | ea4452de2ae987342fadbdd2c044034e6480daad (patch) | |
| tree | 5789122b9ff301d81d2cc73e48300f01124bfe5a /lib/packing.c | |
| parent | de1ccfb648243a031cfbdc2d5571dfdaf5023106 (diff) | |
| download | linux-ea4452de2ae987342fadbdd2c044034e6480daad.tar.xz | |
mm: fix unexpected changes to {failslab|fail_page_alloc}.attr
When we specify __GFP_NOWARN, we only expect that no warnings will be
issued for current caller. But in the __should_failslab() and
__should_fail_alloc_page(), the local GFP flags alter the global
{failslab|fail_page_alloc}.attr, which is persistent and shared by all
tasks. This is not what we expected, let's fix it.
[akpm@linux-foundation.org: unexport should_fail_ex()]
Link: https://lkml.kernel.org/r/20221118100011.2634-1-zhengqi.arch@bytedance.com
Fixes: 3f913fc5f974 ("mm: fix missing handler for __GFP_NOWARN")
Signed-off-by: Qi Zheng <zhengqi.arch@bytedance.com>
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Reviewed-by: Akinobu Mita <akinobu.mita@gmail.com>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Cc: Akinobu Mita <akinobu.mita@gmail.com>
Cc: Matthew Wilcox <willy@infradead.org>
Cc: <stable@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'lib/packing.c')
0 files changed, 0 insertions, 0 deletions