✓ CI.checkpatch: success for series starting with [01/12] drm/xe: Removed unused xe_ggtt_printk

Patchwork patchwork at emeril.freedesktop.org
Sat Aug 17 10:42:25 UTC 2024


== Series Details ==

Series: series starting with [01/12] drm/xe: Removed unused xe_ggtt_printk
URL   : https://patchwork.freedesktop.org/series/137409/
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
9fe5037901cabbcdf27a6fe0dfb047ca1474d363
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 2c0ccb2c3eff579196da9992a40d7998a5557611
Author: Rodrigo Vivi <rodrigo.vivi at intel.com>
Date:   Sat Aug 17 06:35:55 2024 -0400

    drm/xe: Fix missing runtime outer protection for ggtt_remove_node
    
    Defer the ggtt node removal to a thread if runtime_pm is not active.
    
    The ggtt node removal can be called from multiple places, including
    places where we cannot protect with outer callers and places we are
    within other locks. So, try to grab the runtime reference if the
    device is already active, otherwise defer the removal to a separate
    thread from where we are sure we can wake the device up.
    
    v2: - use xe wq instead of system wq (Matt and CI)
        - Avoid GFP_KERNEL to be future proof since this removal can
        be called from outside our drivers and we don't want to block
        if atomic is needed. (Brost)
    v3: amend forgot chunk declaring xe_device.
    v4: Use a xe_ggtt_region to encapsulate the node and remova info,
        wihtout the need for any memory allocation at runtime.
    v5: Actually fill the delayed_removal.invalidate (Brost)
    v6: - Ensure that ggtt_region is not freed before work finishes (Auld)
        - Own wq to ensures that the queued works are flushed before
          ggtt_fini (Brost)
    v7: also free ggtt_region on early !bound return (Auld)
    v8: Address the null deref (CI)
    v9: Based on the new xe_ggtt_node for the proper care of the lifetime
        of the object.
    v10: Redo the lost v5 change. (Brost)
    v11: Simplify the invalidate_on_remove (Lucas)
    
    Cc: Matthew Auld <matthew.auld at intel.com>
    Cc: Paulo Zanoni <paulo.r.zanoni at intel.com>
    Cc: Francois Dugast <francois.dugast at intel.com>
    Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
    Cc: Matthew Brost <matthew.brost at intel.com>
    Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>
    Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
+ /mt/dim checkpatch d6dac3db19935f5939cbb033eea30c90bdf3888c drm-intel
db1683a62d4b drm/xe: Removed unused xe_ggtt_printk
706ddaf71a29 drm/xe: Introduce GGTT documentation
d93be4585e5e drm/xe: Remove unnecessary drm_mm.h includes
8d42c3ab9b5c drm/{i915, xe}: Avoid direct inspection of dpt_vma from outside dpt
2625fcd5a854 drm/xe: Encapsulate drm_mm_node inside xe_ggtt_node
f3de4d3b96e5 drm/xe: Rename xe_ggtt_node related functions
5b533b9ea5e5 drm/xe: Limit drm_mm_node_allocated access to xe_ggtt_node
24b65ce6810f drm/xe: Introduce xe_ggtt_largest_hole
e2cc12e08e11 drm/xe: Introduce xe_ggtt_print_holes
7c9d6645b452 drm/xe: Refactor xe_ggtt balloon functions to make the node clear
85843d1777de drm/xe: Make xe_ggtt_node struct independent
2c0ccb2c3eff drm/xe: Fix missing runtime outer protection for ggtt_remove_node




More information about the Intel-xe mailing list