✗ CI.checkpatch: warning for series starting with [CI,1/3] drm/xe: poor man's exhaustive eviction

Patchwork patchwork at emeril.freedesktop.org
Mon Jun 10 15:28:41 UTC 2024


== Series Details ==

Series: series starting with [CI,1/3] drm/xe: poor man's exhaustive eviction
URL   : https://patchwork.freedesktop.org/series/134682/
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 1b16672f2be9014953608a0aa57d8e1fedcc413e
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date:   Mon Jun 10 17:20:17 2024 +0200

    drm/xe/xe-for-ci: Check whether oom was due to ww mutex error injection
    
    When CONFIG_DEBUG_WW_MUTEX_SLOWPATH is enabled, which it is in CI, but
    not in production kernels, an injected -EDEADLK error will, due to
    limitations in TTM, cause false OOM notifications.
    
    Check whether the OOM was likely caused by an -EDEADLK injection and in
    that case, rerun the validation.
    
    Signed-off-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 3d66fb77a4da62a4282f60ddaab2bc5fabfe2fec drm-intel
2232a8ec62c2 drm/xe: poor man's exhaustive eviction
-:10: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:99: CHECK:SPACING: No space is necessary after a cast
#99: FILE: drivers/gpu/drm/xe/xe_gt_pagefault.c:157:
+	(void) xe_exec_init(&exec, 0, 0, vm->xe, false);

-:117: CHECK:SPACING: No space is necessary after a cast
#117: FILE: drivers/gpu/drm/xe/xe_gt_pagefault.c:526:
+	(void) xe_exec_init(&exec, 0, 0, vm->xe, false);

-:131: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#131: 
new file mode 100644

-:136: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#136: FILE: drivers/gpu/drm/xe/xe_val_lock.c:1:
+#include <drm/drm_exec.h>

-:194: ERROR:TRAILING_WHITESPACE: trailing whitespace
#194: FILE: drivers/gpu/drm/xe/xe_val_lock.c:59:
+^I$

-:207: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#207: FILE: drivers/gpu/drm/xe/xe_val_lock.h:1:
+#ifndef _XE_VAL_LOCK_H_

-:223: CHECK:LINE_SPACING: Please don't use multiple blank lines
#223: FILE: drivers/gpu/drm/xe/xe_val_lock.h:17:
+
+

-:293: CHECK:SPACING: No space is necessary after a cast
#293: FILE: drivers/gpu/drm/xe/xe_vm.c:1056:
+	(void) xe_exec_init(&exec, 0, 0, xe, false);

total: 1 errors, 4 warnings, 4 checks, 287 lines checked
fe96e95e92a4 drm/xe: Take the validation rwsem in exclusive mode on OOM
-:70: CHECK:SPACING: No space is necessary after a cast
#70: FILE: drivers/gpu/drm/xe/xe_gt_pagefault.c:157:
+	(void) xe_exec_init(&exec, 0, 0, vm->xe, exclusive);

total: 0 errors, 0 warnings, 1 checks, 149 lines checked
1b16672f2be9 drm/xe/xe-for-ci: Check whether oom was due to ww mutex error injection
-:36: WARNING:MACRO_ARG_UNUSED: Argument '_a' is not used in function-like macro
#36: FILE: drivers/gpu/drm/xe/xe_vm.c:349:
+#define xe_exec_contention_injected(_a) (false)

-:51: ERROR:TRAILING_WHITESPACE: trailing whitespace
#51: FILE: drivers/gpu/drm/xe/xe_vm.c:375:
+^Iif (err != -ENOMEM || *exclusive)^I  $

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




More information about the Intel-xe mailing list