On Mon, Mar 22, 2021 at 12:22 PM Christian König
wrote:
>
> Don't print a warning when we fail to allocate a page for swapping things out.
>
> v2: only stop the warning
>
> Signed-off-by: Christian König
Reviewed-by: Daniel Vetter
It is kinda surprising that page allocator warns here even though we
explicitly asked for non-GFP_KERNEL (which is the only one where you
pretty much can assume you will get memory no matter what, since worst
case the OOM killer makes space for you).
But then with memalloc_no* and friends these failures could happen in
unexpected places, and I think the code that warns isn't aware of the
original gfp flags, so makes some sense from an implementation pov.
-Daniel
> ---
> drivers/gpu/drm/ttm/ttm_tt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c
> index 2f0833c98d2c..48b9a650630b 100644
> --- a/drivers/gpu/drm/ttm/ttm_tt.c
> +++ b/drivers/gpu/drm/ttm/ttm_tt.c
> @@ -369,7 +369,7 @@ static unsigned long ttm_tt_shrinker_scan(struct shrinker
> *shrink,
> };
> int ret;
>
> - ret = ttm_bo_swapout(, GFP_NOFS);
> + ret = ttm_bo_swapout(, GFP_NOFS | __GFP_NOWARN);
> return ret < 0 ? SHRINK_EMPTY : ret;
> }
>
> --
> 2.25.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
