✗ 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