✗ CI.checkpatch: warning for Only timeout jobs if they run longer than timeout period
Patchwork
patchwork at emeril.freedesktop.org
Mon Jun 10 14:23:53 UTC 2024
== Series Details ==
Series: Only timeout jobs if they run longer than timeout period
URL : https://patchwork.freedesktop.org/series/134679/
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 20b60b023b1b61eac219d63a5aa63f8c128c7656
Author: Matthew Brost <matthew.brost at intel.com>
Date: Mon Jun 10 07:18:23 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 ef92810a097c2ec72756c58e64cfc3096e6a2386 drm-intel
d9365922f989 drm/xe: Add LRC ctx timestamp support functions
d763e8cc01bf drm/xe: Add MI_COPY_MEM_MEM GPU instruction definitions
42896ce2c84c drm/xe: Emit ctx timestamp copy in ring ops
85f893719d2f drm/xe: Add ctx timestamp to LRC snapshot
5ec438403d55 drm/xe: Add xe_gt_clock_interval_to_ms helper
c19f08e4c71f drm/xe: Improve unexpected state error messages
5175b9184a80 drm/xe: Add GuC state asserts to deregister_exec_queue
cbdf6ab25d88 drm/xe: Add pending disable assert to handle_sched_done
077548c091e9 drm/xe: Add killed, banned, or wedged as stick bit during GuC reset
20b60b023b1b 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