✗ CI.checkpatch: warning for Only timeout jobs if they run longer than timeout period

Patchwork patchwork at emeril.freedesktop.org
Mon Jun 10 13:54:41 UTC 2024


== Series Details ==

Series: Only timeout jobs if they run longer than timeout period
URL   : https://patchwork.freedesktop.org/series/134677/
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
51ce9f6cd981d42d7467409d7dbc559a450abc1e
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit ec39258838418f072f5bd72919282e074cd36aea
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Mon Jun 10 06:50:11 2024 -0700

    drm/xe: Sample ctx timestamp to determine if jobs have timed out
    
    In GuC TDR sample ctx timestamp to determine if jobs have timed out. The
    scheduling enable needs to be toggled to properly sample the timestamp.
    If a job has not been running for longer than the timeout period,
    re-enable scheduling and restart the TDR.
    
    v2:
     - Use GT clock to msec helper (Umesh, off list)
     - s/ctx_timestamp_job/ctx_job_timestamp
    v3:
     - Fix state machine for TDR, mainly decouple sched disable and
       deregister (testing)
     - Rebase (CI)
    v4:
     - Fix checkpatch && newline issue (CI)
     - Do not deregister on wedged or unregistered (CI)
     - Fix refcounting bugs (CI)
     - Move devcoredump above VM / kernel job check (John H)
     - Add comment for check_timeout state usage (John H)
     - Assert pending disable not inflight when enabling scheduling (John H)
     - Use enable_scheduling in other scheduling enable code (John H)
     - Add comments on a few steps in TDR (John H)
     - Add assert for timestamp overflow protection (John H)
    
    Signed-off-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 183cef86631f1ea60930365f10582bb963a790f1 drm-intel
66a84e18ab11 drm/xe: Add ctx timestamp to LRC snapshot
7e7d4d60f5d6 drm/xe: Add xe_gt_clock_interval_to_ms helper
de7afa65bac3 drm/xe: Improve unexpected state error messages
53624491cb1e drm/xe: Add GuC state asserts to deregister_exec_queue
137b80c80e9b drm/xe: Add pending disable assert to handle_sched_done
2d43a417f2b9 drm/xe: Add killed, banned, or wedged as stick bit during GuC reset
ec3925883841 drm/xe: Sample ctx timestamp to determine if jobs have timed out
-:89: CHECK:SPACING: No space is necessary after a cast
#89: FILE: drivers/gpu/drm/xe/xe_guc_submit.c:951:
+#define ADJUST_FIVE_PERCENT(__t)	(((__t) * 105) / 100)

-:89: ERROR:SPACING: space prohibited after that '*' (ctx:WxW)
#89: FILE: drivers/gpu/drm/xe/xe_guc_submit.c:951:
+#define ADJUST_FIVE_PERCENT(__t)	(((__t) * 105) / 100)
                                 	        ^

total: 1 errors, 0 warnings, 1 checks, 420 lines checked




More information about the Intel-xe mailing list