✗ CI.checkpatch: warning for drm/xe/debugfs: Make sysfs gt force reset synchronous

Patchwork patchwork at emeril.freedesktop.org
Thu Jan 4 08:46:09 UTC 2024


== Series Details ==

Series: drm/xe/debugfs: Make sysfs gt force reset synchronous
URL   : https://patchwork.freedesktop.org/series/128033/
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
6030b24c1386b00de8187b5fb987e283a57b372a
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit ab00ab86b666d95f0dd9ed4692d4a51c136bc3e8
Author: Karthik Poosa <karthik.poosa at intel.com>
Date:   Wed Dec 27 14:02:04 2023 +0530

    drm/xe/debugfs: Make sysfs gt force reset synchronous
    
    Wait for gt reset to complete before returning from force_reset
    sysfs call. Without this igt test freq_reset_multiple fails
    sporadically in case xe_guc_pc is not started.
    
    v2:
    - Changed wait for completion to interruptible (Anshuman).
    - Moved timeout to xe_gt.h (Anshuman).
    - Created a debugfs for updating timeout (Rodrigo).
    
    Testcase: igt at xe_guc_pc@freq_reset_multiple
    Signed-off-by: Karthik Poosa <karthik.poosa at intel.com>
+ /mt/dim checkpatch 7b3b98d034784b125bad7aca46f9e7a3cfcde45a drm-intel
ab00ab86b drm/xe/debugfs: Make sysfs gt force reset synchronous
-:51: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#51: FILE: drivers/gpu/drm/xe/xe_gt_debugfs.c:66:
+	ret = wait_for_completion_interruptible_timeout(&gt->reset_done,
+						msecs_to_jiffies(gt->reset_timeout_ms));

-:66: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#66: FILE: drivers/gpu/drm/xe/xe_gt_debugfs.c:239:
+	debugfs_create_u32("gt_reset_timeout_ms", 0600, root,
+					&gt->reset_timeout_ms);

total: 0 errors, 0 warnings, 2 checks, 52 lines checked




More information about the Intel-xe mailing list