✓ CI.checkpatch: success for drm/xe/pxp: Don't kill queues while holding PXP locks (rev2)

[Patchwork]

== Series Details ==

Series: drm/xe/pxp: Don't kill queues while holding PXP locks (rev2)
URL   : https://patchwork.freedesktop.org/series/145134/
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
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 3e2c4e6870d1afd07390236d2bbc9f93c8b70d9c
Author: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
Date:   Tue Feb 25 15:53:28 2025 -0800

    drm/xe/pxp: Don't kill queues while holding PXP locks
    
    xe_exec_queue_kill can sleep, so we can't call it from under a spinlock.
    We can instead move the queues to a separate list and then kill them all
    after we release the spinlock.
    
    Furthermore, xe_exec_queue_kill can take the VM lock so we can't call it
    while holding the PXP mutex because the mutex is taken under VM lock at
    queue creation time. Note that while it is safe to call the kill without
    holding the mutex, we must call it after the PXP state has been updated,
    otherwise an app might be able to create a queue between the
    invalidation and the state update, which would break the state machine.
    
    Since being in the list is used to track whether RPM cleanup is needed,
    we can no longer defer that to queue_destroy, so we perform it
    immediately instead.
    
    v2: also avoid calling kill() under pxp->mutex.
    
    Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
    Closes: https://lore.kernel.org/intel-xe/34aaced9-4a9d-4e8c-900a-b8f73452e35c@stanley.mountain/
    Fixes: f8caa80154c4 ("drm/xe/pxp: Add PXP queue tracking and session start")
    Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
    Cc: John Harrison <John.C.Harrison at Intel.com>
    Cc: Matthew Brost <matthew.brost at intel.com>
    Reviewed-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 9527883f549b856628618a497d153e515752b457 drm-intel
3e2c4e6870d1 drm/xe/pxp: Don't kill queues while holding PXP locks