✗ CI.checkpatch: warning for Fix device page migration in low memory fallback
Patchwork <[email protected]> Wed, 05 Aug 2026 15:22:20 -0000
| Newsgroups | org.freedesktop.lists.intel-xe |
|---|---|
| Message-ID | <178594334087.21461.17359992907916666710@61270ab9df2a> |
== Series Details == Series: Fix device page migration in low memory fallback URL : https://patchwork.freedesktop.org/series/171651/ State : warning == Summary == + KERNEL=/kernel + git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt Cloning into 'mt'... warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/ + git -C mt rev-list -n1 origin/master 061140b9bc586ae7f40abc1249c97e1cc72d1b9d + cd /kernel + git config --global --add safe.directory /kernel + git log -n1 commit 736b7f802b4297f160069ec61b375c2eddb5d473 Author: Matthew Brost <[email protected]> Date: Wed Aug 5 04:33:38 2026 -0700 drm/pagemap: Add fault injection for higher-order RAM folio allocation Migrating a device-private THP back to system memory has two distinct paths in __migrate_device_pages(): the fast path where both source and destination carry MIGRATE_PFN_COMPOUND, and the fallback path where the destination could only be satisfied with order-0 folios and the source THP therefore has to be split via migrate_vma_split_unmapped_folio(). The fallback path only triggers under genuine memory pressure, which makes it both rare and awkward to reproduce, yet it is the path where the interesting refcounting happens (the CPU fault holds an extra reference on the device folio taken by do_huge_pmd_device_private()). Add a fault_attr, modelled on backup_fault_inject in ttm_pool.c, that forces the higher-order allocation in drm_pagemap_migrate_populate_ram_pfn() to fail so the existing order-0 fallback is taken deterministically. The attribute is exposed at /sys/kernel/debug/drm_pagemap_fault_inject and requires CONFIG_FAULT_INJECTION_DEBUG_FS. With CONFIG_FAULT_INJECTION disabled the helper compiles out to a constant false and the injection has no cost. Cc: Andrew Morton <[email protected]> Cc: David Hildenbrand <[email protected]> Cc: Lorenzo Stoakes <[email protected]> Cc: Zi Yan <[email protected]> Cc: Baolin Wang <[email protected]> Cc: Liam R. Howlett <[email protected]> Cc: Nico Pache <[email protected]> Cc: Ryan Roberts <[email protected]> Cc: Dev Jain <[email protected]> Cc: Barry Song <[email protected]> Cc: Lance Yang <[email protected]> Cc: Usama Arif <[email protected]> Cc: Joshua Hahn <[email protected]> Cc: Rakie Kim <[email protected]> Cc: Byungchul Park <[email protected]> Cc: Gregory Price <[email protected]> Cc: Ying Huang <[email protected]> Cc: Alistair Popple <[email protected]> Cc: Balbir Singh <[email protected]> Cc: Maarten Lankhorst <[email protected]> Cc: Maxime Ripard <[email protected]> Cc: Thomas Zimmermann <[email protected]> Cc: David Airlie <[email protected]> Cc: Simona Vetter <[email protected]> Cc: Thomas Hellström <[email protected]> Cc: Francois Dugast <[email protected]> Cc: [email protected] Cc: [email protected] Cc: [email protected] Cc: [email protected] Assisted-by: GitHub Copilot:claude-opus-5 Signed-off-by: Matthew Brost <[email protected]> + /mt/dim checkpatch b50d67c8afa30f742e87999b53f80d22f8be4ffc drm-intel b2b124c65ad7 mm/migrate_device: Fix THP splitting of a CPU faulted device private folio -:136: WARNING:BAD_SIGN_OFF: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format #136: Assisted-by: GitHub Copilot:claude-opus-5 total: 0 errors, 1 warnings, 0 checks, 133 lines checked 6757a8e3bf13 mm/migrate_device: Apply the fault reference to the correct folio -:63: WARNING:BAD_SIGN_OFF: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format #63: Assisted-by: GitHub Copilot:claude-opus-5 -:87: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV) #87: FILE: mm/migrate_device.c:1300: + struct page *src_page = migrate_pfn_to_page(src_pfns[i+j]); ^ total: 0 errors, 1 warnings, 1 checks, 37 lines checked 85ebfbd0a2db drm/pagemap: Fix folio allocation fallback and use-after-put -:75: WARNING:BAD_SIGN_OFF: Duplicate signature #75: Cc: [email protected] -:76: WARNING:BAD_SIGN_OFF: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format #76: Assisted-by: GitHub Copilot:claude-opus-5 total: 0 errors, 2 warnings, 0 checks, 168 lines checked 736b7f802b42 drm/pagemap: Add fault injection for higher-order RAM folio allocation -:61: WARNING:BAD_SIGN_OFF: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format #61: Assisted-by: GitHub Copilot:claude-opus-5 -:131: WARNING:MISSING_FIXES_TAG: The commit message has 'stable@', perhaps it also needs a 'Fixes:' tag? total: 0 errors, 2 warnings, 0 checks, 60 lines checked