[Intel-xe] ✓ CI.checkpatch: success for drm/xe: Fix pagefault and access counter worker functions (rev3)

Patchwork patchwork at emeril.freedesktop.org
Mon Nov 6 17:20:11 UTC 2023


== Series Details ==

Series: drm/xe: Fix pagefault and access counter worker functions (rev3)
URL   : https://patchwork.freedesktop.org/series/125936/
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
63c2b6b160bca2df6efc7bc4cea6f442097d7854
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit d23bb932b5cc660e6c339e84bbcd591316e8829e
Author: Brian Welty <brian.welty at intel.com>
Date:   Fri Nov 3 12:46:38 2023 -0700

    drm/xe: Fix pagefault and access counter worker functions
    
    When processing G2H messages for pagefault or access counters, we queue a
    work item and call queue_work(). This fails if the worker thread is already
    queued to run.
    The expectation is that the worker function will do more than process a
    single item and return. It needs to either process all pending items or
    requeue itself if items are pending. But requeuing will add latency and
    potential context switch can occur.
    
    We don't want to add unnecessary latency and so the worker should process
    as many faults as it can within a reasonable duration of time.
    We also do not want to hog the cpu core, so here we execute in a loop
    and requeue if still running after more than 20 ms.
    This seems reasonable framework and easy to tune this futher if needed.
    
    This resolves issues seen with several igt at xe_exec_fault_mode subtests
    where the GPU will hang when KMD ignores a pending pagefault.
    
    v2: requeue the worker instead of having an internal processing loop.
    v3: implement hybrid model of v1 and v2
        now, run for 20 msec before we will requeue if still running
    v4: only requeue in worker if queue is non-empty (Matt B)
    
    Signed-off-by: Brian Welty <brian.welty at intel.com>
    Reviewed-by: Matthew Brost <matthew.brost at intel.com>
    Reviewed-by: Stuart Summers <stuart.summers at intel.com>
+ /mt/dim checkpatch fc457bcc18109467961edc92c80b238009f6a9fb drm-intel
d23bb932b drm/xe: Fix pagefault and access counter worker functions




More information about the Intel-xe mailing list