[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5867e22f-43cc-ca21-bc60-ec67554e66@google.com>
Date: Sun, 6 Mar 2022 14:15:35 -0800 (PST)
From: David Rientjes <rientjes@...gle.com>
To: Eric Dumazet <eric.dumazet@...il.com>
cc: Andrew Morton <akpm@...ux-foundation.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
linux-mm <linux-mm@...ck.org>,
Eric Dumazet <edumazet@...gle.com>,
Mel Gorman <mgorman@...hsingularity.net>,
Vlastimil Babka <vbabka@...e.cz>,
Michal Hocko <mhocko@...nel.org>,
Shakeel Butt <shakeelb@...gle.com>,
Wei Xu <weixugc@...gle.com>, Greg Thelen <gthelen@...gle.com>,
Hugh Dickins <hughd@...gle.com>
Subject: Re: [PATCH v2] mm/page_alloc: call check_new_pages() while zone
spinlock is not held
On Fri, 4 Mar 2022, Eric Dumazet wrote:
> From: Eric Dumazet <edumazet@...gle.com>
>
> For high order pages not using pcp, rmqueue() is currently calling
> the costly check_new_pages() while zone spinlock is held,
> and hard irqs masked.
>
> This is not needed, we can release the spinlock sooner to reduce
> zone spinlock contention.
>
> Note that after this patch, we call __mod_zone_freepage_state()
> before deciding to leak the page because it is in bad state.
>
> v2: We need to keep interrupts disabled to call __mod_zone_freepage_state()
>
> Signed-off-by: Eric Dumazet <edumazet@...gle.com>
> Cc: Mel Gorman <mgorman@...hsingularity.net>
> Cc: Vlastimil Babka <vbabka@...e.cz>
> Cc: Michal Hocko <mhocko@...nel.org>
> Cc: Shakeel Butt <shakeelb@...gle.com>
> Cc: Wei Xu <weixugc@...gle.com>
> Cc: Greg Thelen <gthelen@...gle.com>
> Cc: Hugh Dickins <hughd@...gle.com>
> Cc: David Rientjes <rientjes@...gle.com>
Acked-by: David Rientjes <rientjes@...gle.com>
Powered by blists - more mailing lists