[Intel-xe] ✗ CI.checkpatch: warning for drm/xe/guc: Fix h2g_write usage of GUC_CTB_MSG_MAX_LEN
Patchwork
patchwork at emeril.freedesktop.org
Fri Jun 2 20:27:18 UTC 2023
== Series Details ==
Series: drm/xe/guc: Fix h2g_write usage of GUC_CTB_MSG_MAX_LEN
URL : https://patchwork.freedesktop.org/series/118799/
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
c7d32770e3cd31d9fc134ce41f329b10aa33ee15
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 622d33c2a95936c55a52554ee2446f849e21baf1
Author: Alan Previn <alan.previn.teres.alexis at intel.com>
Date: Fri Jun 2 11:52:13 2023 -0700
drm/xe/guc: Fix h2g_write usage of GUC_CTB_MSG_MAX_LEN
In the ABI header, GUC_CTB_MSG_MIN_LEN is '1' because
GUC_CTB_HDR_LEN is 1. This aligns with H2G/G2H CTB specification
where all command formats are defined in units of dwords so that '1'
is a dword. Accordingly, GUC_CTB_MSG_MAX_LEN is 256-1 (i.e. 255
dwords). However, h2g_write was incorrectly assuming that
GUC_CTB_MSG_MAX_LEN was in bytes. Fix this.
Signed-off-by: Alan Previn <alan.previn.teres.alexis at intel.com>
+ /mt/dim checkpatch ef153c77d25ffd86b074190b77f9f561b9a5a909 drm-intel
622d33c2a drm/xe/guc: Fix h2g_write usage of GUC_CTB_MSG_MAX_LEN
-:33: WARNING:AVOID_BUG: Do not crash the kernel unless it is absolutely unavoidable--use WARN_ON_ONCE() plus recovery code (if feasible) instead of BUG() or variants
#33: FILE: drivers/gpu/drm/xe/xe_guc_ct.c:412:
+ XE_BUG_ON(len > GUC_CTB_MSG_MAX_LEN - 1);
total: 0 errors, 1 warnings, 0 checks, 16 lines checked
More information about the Intel-xe
mailing list