✓ CI.checkpatch: success for drm/xe/userptr: fix notifier vs folio deadlock

Patchwork patchwork at emeril.freedesktop.org
Mon Apr 14 14:08:00 UTC 2025


== Series Details ==

Series: drm/xe/userptr: fix notifier vs folio deadlock
URL   : https://patchwork.freedesktop.org/series/147678/
State : success

== 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
13a92ce9fd458ebd6064f23cec8c39c53d02ed26
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 2321ba3851883ed6e350942060ed29949024fb39
Author: Matthew Auld <matthew.auld at intel.com>
Date:   Mon Apr 14 14:25:40 2025 +0100

    drm/xe/userptr: fix notifier vs folio deadlock
    
    User is reporting what smells like notifier vs folio deadlock, where
    migrate_pages_batch() on core kernel side is holding folio lock(s) and
    then interacting with the mappings of it, however those mappings are
    tied to some userptr, which means calling into the notifier callback and
    grabbing the notifier lock. With perfect timing it looks possible that
    the pages we pulled from the hmm fault can get sniped by
    migrate_pages_batch() at the same time that we are holding the notifier
    lock to mark the pages as accessed/dirty, but at this point we also want
    to grab the folio locks(s) to mark them as dirty, but if they are
    contended from notifier/migrate_pages_batch side then we deadlock since
    folio lock won't be dropped until we drop the notifier lock.
    
    Fortunately the mark_page_accessed/dirty is not really needed in the
    first place it seems and should have already been done by hmm fault, so
    just remove it.
    
    Link: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/4765
    Fixes: 0a98219bcc96 ("drm/xe/hmm: Don't dereference struct page pointers without notifier lock")
    Signed-off-by: Matthew Auld <matthew.auld at intel.com>
    Cc: Thomas Hellström <thomas.hellstrom at intel.com>
    Cc: Matthew Brost <matthew.brost at intel.com>
    Cc: <stable at vger.kernel.org> # v6.10+
    Reviewed-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 24805f4111e68bf0f6f7a513d2145fe7debd7eea drm-intel
2321ba385188 drm/xe/userptr: fix notifier vs folio deadlock




More information about the Intel-xe mailing list