✗ CI.checkpatch: warning for Only timeout jobs if they run longer than timeout period
Patchwork
patchwork at emeril.freedesktop.org
Tue Jun 11 15:42:23 UTC 2024
== Series Details ==
Series: Only timeout jobs if they run longer than timeout period
URL : https://patchwork.freedesktop.org/series/134724/
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 6f782be9b30a16233e9424615c9cd197f2f31ade
Author: Matthew Brost <matthew.brost at intel.com>
Date: Tue Jun 11 07:40:53 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)
v6:
- Use mul_u64_u32_div (CI, checkpath)
- Change check time to dbg level (Paulo)
- Add immediate mode to sched disable (inspection)
- Use xe_gt_* messages (John H)
- Fix typo in comment (John H)
- Check timeout before clearing pending disable (Paulo)
Signed-off-by: Matthew Brost <matthew.brost at intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
+ /mt/dim checkpatch 950ce28fe0b31a24c5bddcc337aba9554a41cbb3 drm-intel
c1b0c2fc00ae drm/xe: Add LRC ctx timestamp support functions
ac6396d24f9b drm/xe: Add MI_COPY_MEM_MEM GPU instruction definitions
9a936b4770e7 drm/xe: Emit ctx timestamp copy in ring ops
1d2489ac72db drm/xe: Add ctx timestamp to LRC snapshot
bbadb45fbac3 drm/xe: Add xe_gt_clock_interval_to_ms helper
eaeda6e8ec2e drm/xe: Improve unexpected state error messages
d89857d4b955 drm/xe: Assert runnable state in handle_sched_done
a223c956174b drm/xe: Add GuC state asserts to deregister_exec_queue
6b7e6b1e228d drm/xe: Add pending disable assert to handle_sched_done
93749ae0414b drm/xe: Add killed, banned, or wedged as stick bit during GuC reset
6f782be9b30a drm/xe: Sample ctx timestamp to determine if jobs have timed out
-:105: CHECK:SPACING: No space is necessary after a cast
#105: FILE: drivers/gpu/drm/xe/xe_guc_submit.c:952:
+#define ADJUST_FIVE_PERCENT(__t) mul_u64_u32_div((__t), 105, 100)
total: 0 errors, 0 warnings, 1 checks, 437 lines checked
More information about the Intel-xe
mailing list