[RFC PATCH 0/4] kho: Support preserving unsplit high-order pages
Pranjal Shrivastava <[email protected]>
| Newsgroups | org.infradead.lists.kexec,org.kernel.vger.linux-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
This series is required for the ongoing effort to preserve DMA allocations across KHO [1]. It addresses a fundamental mismatch between the current KHO restoration logic and adds support for high-order buddy allocations. The Problem =========== The current KHO restore implementation treats all multi-page blocks as split pages during restoration, i.e. kho_restore_pages() initializes every 4KB page with a refcount of 1. However, many kernel subsystems, most notably the DMA allocator (via dma_alloc_coherent), frequently return high-order non-compound pages. In this unsplit state, only the head page carries a refcount of 1, while all tail pages have a reference count of 0. Consequently, when these contiguous but unsplit blocks are restored by KHO in the new kernel, the forced refcount of 1 on tail pages causes some trouble with the buddy allocator. Downstream of the eventual free path the __free_pages_prepare() [2] ends up calling page_expected_state() [3] when is_check_pages_enabled() returns true (only when CONFIG_DEBUG_VM or debug_pagealloc=on). This detects the non-zero refcounts on tail pages [4] and incorrectly taints the kernel while leaking the pages in question. Proposed Solution ================= This series introduces a "Page Type" field to the KHO ABI to track the refcount pattern of the preserved pages. 1. KHO detects the physical state (CONTIG vs SPLIT) during preservation by peeking at the refcount of the second page in each buddy block. 2. The type bit is preserved in the high bits of the KHO radix tree key (Bit 63) and stashed in page->private metadata during boot. 3. kho_restore_page() applies the correct refcount pattern based on the preserved metadata. 4. A new helper, kho_split_preserved_pages(), is provided for subsystems that may need to split memory after it has already been preserved. Considerations ============== 1. A primary goal of this approach is to prevent driver/subsystem code from peeking into MM internals. Drivers should not need to understand the distinction between head/tail pages or compound metadata. The KHO core handles this internally. 2. To handle rare cases where a caller might wish to split a high-order block after preservation, we provide kho_split_preserved_pages(). 3. The callers must ensure that the split_page() doesn't race with kho_preserve_pages for consistency. 4. Folios are always implicitly considered of the CONTIG type Thanks, Praan [1] https://lore.kernel.org/all/[email protected]/ [2] https://elixir.bootlin.com/linux/v7.1.1/source/mm/page_alloc.c#L1370 [3] https://elixir.bootlin.com/linux/v7.1.1/source/mm/page_alloc.c#L1027 [4] https://elixir.bootlin.com/linux/v7.1.1/source/mm/page_alloc.c#L1034 Pranjal Shrivastava (4): kho: Introduce infrastructure to track preserved page types kho: Detect preserved page types kho: Implement page-aware refcount restoration kho: Introduce kho_split_preserved_pages() helper include/linux/kexec_handover.h | 7 ++ include/linux/kho_radix_tree.h | 17 +++- kernel/liveupdate/kexec_handover.c | 144 +++++++++++++++++++++-------- 3 files changed, 124 insertions(+), 44 deletions(-) base-commit: 87320be9f0d24fce67631b7eef919f0b79c3e45c -- 2.55.0.rc0.799.gd6f94ed593-goog