✗ CI.checkpatch: warning for drm/xe/uapi: Use hint for guc to set GT frequency

[Patchwork]

== Series Details ==

Series: drm/xe/uapi: Use hint for guc to set GT frequency
URL   : https://patchwork.freedesktop.org/series/143793/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 91c61b01d641c4e4d8d3821c0392500e4c3e5c33
Author: Tejas Upadhyay <tejas.upadhyay at intel.com>
Date:   Tue Jan 21 18:46:05 2025 +0530

    drm/xe/uapi: Use hint for guc to set GT frequency
    
    Allow user to provide a low latency hint. When set,
    KMD sends a hint to GuC which results in special handling for that process.
    SLPC will ramp the GT frequency aggressively every time
    it switches to this process.
    
    We need to enable the use of SLPC Compute strategy during init, but
    it will apply only to processes that set this bit during process
    creation.
    
    Improvement with this approach as below:
    
    Before,
    
    :~$ NEOReadDebugKeys=1 EnableDirectSubmission=0 clpeak --kernel-latency
    Platform: Intel(R) OpenCL Graphics
      Device: Intel(R) Graphics [0xe20b]
        Driver version  : 24.52.0 (Linux x64)
        Compute units   : 160
        Clock frequency : 2850 MHz
        Kernel launch latency : 283.16 us
    
    After,
    
    :~$ NEOReadDebugKeys=1 EnableDirectSubmission=0 clpeak --kernel-latency
    Platform: Intel(R) OpenCL Graphics
      Device: Intel(R) Graphics [0xe20b]
        Driver version  : 24.52.0 (Linux x64)
        Compute units   : 160
        Clock frequency : 2850 MHz
    
        Kernel launch latency : 63.38 us
    
    UMD will indicate low latency hint with flag as mentioned below,
    
    *     struct drm_xe_exec_queue_create exec_queue_create = {
    *          .flags = DRM_XE_EXEC_QUEUE_LOW_LATENCY_HINT or 0
    *          .extensions = 0,
    *          .vm_id = vm,
    *          .num_bb_per_exec = 1,
    *          .num_eng_per_bb = 1,
    *          .instances = to_user_pointer(&instance),
    *     };
    *     ioctl(fd, DRM_IOCTL_XE_EXEC_QUEUE_CREATE, &exec_queue_create);
    
    Link to UMD PR : https://github.com/intel/compute-runtime/pull/794
    
    V4:
      - To make it clear, dont use exec queue word (Vinay)
      - Correct typo in description of flag (Jose/Vinay)
      - rename set_strategy api and replace ctx with exec queue(Vinay)
      - Start with 0th bit to indentify user flags (Jose)
    V3:
      - Conver user flag to kernel internal flag and use (Oak)
      - Support query config for use to check kernel support (Jose)
      - Dont need to take runtime pm (Vinay)
    V2:
      - DRM_XE_EXEC_QUEUE_LOW_LATENCY_HINT 1 is already planned for other hint(Szymon)
      - Add motivation to description (Lucas)
    
    Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
+ /mt/dim checkpatch d471f521da387883809f72b139a2013c0652bee1 drm-intel
91c61b01d641 drm/xe/uapi: Use hint for guc to set GT frequency
-:62: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#62: 
  - DRM_XE_EXEC_QUEUE_LOW_LATENCY_HINT 1 is already planned for other hint(Szymon)

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