From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out-177.mta0.migadu.com (out-177.mta0.migadu.com [91.218.175.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0F502352023 for ; Tue, 19 May 2026 23:25:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779233155; cv=none; b=DLRWKnMzubMMSyGS7gU9ryejrOKBynhRDbxTpJS+i+hbbTsPJB+sEwcf0ZpmlcmFQgtbqdqhGyuSsV43wM7q4vnus363hiREuW+32DNGM+l7A2i/Ui8SowNh5le1lxqAlfInvRMV/QbaaEoKU49LAYdwXxZaTYqXdT7mLa5imnM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779233155; c=relaxed/simple; bh=YWdEIl8inOc/olBW5cpjBBKqiYoslrOtnKDOy1cCdsg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=q6lLstMJNy9obvsUSkViAopDomC3rhQIR3vKF5V242Yeawx0fyo6g3japTECpAhaptdZSuOlQ3lg+dZWDu7Jb5rh9FlEPd8V61Gx9STVSgTGRPeO09LZsof9FZOwfY087wamcYJoKgqXP2Vvfbuns53JkuMtBfmNGJqYACY3Ax0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=oBp3J+Q2; arc=none smtp.client-ip=91.218.175.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="oBp3J+Q2" Message-ID: <8e3586e2-1683-4db2-802d-2478293423aa@linux.dev> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1779233151; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=YlOgClpXkeAqGbaQuJOr4v8l9JGbtUu2G0NG2GKshGY=; b=oBp3J+Q2ZfWVrq2k5fOTHKRvAaWeFwxYjYHpbDdf3UOzFRx7vfm5Xrp59Jg6kCMYB90AO1 Ueweaqyq51L29fNjCxCwAsFitMd37N5vGHEOndsp9OcMDRclRfQcGuJvjW7+TEw9FgVHTu MbB+hnQ8hwa8+Vhth4FPTKBGxq12cK8= Date: Tue, 19 May 2026 16:25:45 -0700 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Subject: Re: [PATCH] mm/page_alloc: skip high atomic reservation at or below costly order To: Andrew Morton Cc: vbabka@kernel.org, surenb@google.com, mhocko@suse.com, jackmanb@google.com, hannes@cmpxchg.org, ziy@nvidia.com, linux-mm@kvack.org, usama.arif@linux.dev, kirill@shutemov.name, willy@infradead.org, linux-kernel@vger.kernel.org, kernel-team@meta.com References: <20260519012532.272770-1-jp.kobryn@linux.dev> <20260519122711.1afe69455bcaf22a0962dce8@linux-foundation.org> Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: "JP Kobryn (Meta)" In-Reply-To: <20260519122711.1afe69455bcaf22a0962dce8@linux-foundation.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Migadu-Flow: FLOW_OUT On 5/19/26 12:27 PM, Andrew Morton wrote: > On Mon, 18 May 2026 18:25:32 -0700 "JP Kobryn (Meta)" wrote: > >> We're seeing a pattern in production where 2MB THP order-9 allocations are >> failing due to fragmentation and triggering reclaim on systems with plenty >> of free memory. Over time, the success rate of these THP allocations do not >> increase at all. >> >> Inspecting zone->vm_stat[NR_FREE_PAGES] via kprobe on compaction_suitable() >> indicated the given zone had sufficient free pages for order-9 allocations, >> yet they were going unused. Drilling down into the zone and inspecting >> /proc/pagetypeinfo revealed why. Order-9 blocks were accumulating in the >> zone's HighAtomic bucket (while zero were present in Movable). THP is >> unable to draw blocks from HighAtomic since that bucket is not in the >> fallback list. >> >> The heuristic for reserving pageblocks in HighAtomic is that any atomic >> allocation greater than order-0 will result in the full pageblock being >> captured. This means that an order-1 atomic allocation will over-reserve by >> 256x, a full 512 pageblock. >> >> Gate the reservation on order. Skip for allocations at or below >> PAGE_ALLOC_COSTLY_ORDER. This prevents smaller atomic allocations from >> reserving entire pageblocks, and significantly helps when THP is in use on >> a fragmented but otherwise healthy system. >> >> Testing was performed using an A/B instagram workload receiving prod >> traffic. Each side had ~60 hosts with 64G memory. The patch resulted in >> several gains: >> >> Unpatched >> HighAtomic pageblocks per host: 309-312 (1% of zone or 620MB), >> ...all order-9 blocks in HighAtomic >> THP success rate: 1-6% >> Compaction success rate: 0-2% >> pgscan_kswapd (total across ~60 hosts, per minute): ~70.2M >> Atomic order-4+ allocations: 0 >> >> Patched >> HighAtomic pageblocks per host: 1 >> THP success rate: 44-78% >> Compaction success rate: 24-47% >> pgscan_kswapd (total across ~60 hosts, per minute): ~29.9M >> Atomic order-4+ allocations: 0 >> >> Note that for this workload all atomic allocations were order 0-3 >> originating from the network stack, btrfs, and scheduler. >> >> ... >> >> --- a/mm/page_alloc.c >> +++ b/mm/page_alloc.c >> @@ -3446,6 +3446,13 @@ static void reserve_highatomic_pageblock(struct page *page, int order, >> int mt; >> unsigned long max_managed; >> >> + /* >> + * Don't reserve a pageblock for lower orders. >> + * Order 1-3 allocs should not capture a huge page size block. >> + */ >> + if (order <= PAGE_ALLOC_COSTLY_ORDER) >> + return; >> + >> /* >> * The number reserved as: minimum is 1 pageblock, maximum is >> * roughly 1% of a zone. But if 1% of a zone falls below a > > Sashiko asked > > : Does skipping the HighAtomic reservation for orders 1-3 break the > : anti-fragmentation guarantees for these atomic allocations? The data included in the changelog supports the claim that the reserve does not provide a benefit at these orders. Even on fragmented systems, orders 1-3 have plenty of pages available. > : > : The MIGRATE_HIGHATOMIC reserve protects high-order atomic allocations > : from failing under fragmentation by taking ownership of the entire > : pageblock. In the experiments, there were no failures for orders 1-3 despite a fragmented system and no reserved pageblocks for these orders. > : > : If order-1 through order-3 atomic allocations fall back to stealing > : pages, but the pageblock remains in its original migratetype, won't > : order-0 non-atomic allocations consume the remaining contiguous space? With the patch, these pageblocks stay movable. So if fallback is needed, moveable pages can still be taken. But the patch actually improves compaction so contiguous space is increased overall. > : > : Under memory pressure, this could leave no contiguous blocks for atomic > : allocations to steal. Because these atomic allocations cannot trigger > : direct reclaim or compaction, they might fail, potentially leading to > : dropped packets or I/O errors in subsystems like the network stack or > : BTRFS. Reserved HighAtomic pageblocks are not currently treated as a precious resource. Under real memory pressure, the kernel already gives them up - the unreserve mechanism kicks in and converts the HighAtomic pageblocks back to their original migrate type. > : > : Could background compaction or khugepaged be used to unreserve > : HighAtomic blocks dynamically instead of disabling the reserve for > : these orders? This would call for extra scanning/overhead/stats. The patch reduces reclaim, for example. A new scanner feels like going in the opposite direction.