✗ CI.checkpatch: warning for iommu/vt-d: Enable ATS before cache tag assignment (rev2)

Patchwork patchwork at emeril.freedesktop.org
Tue Jun 24 13:33:32 UTC 2025


== Series Details ==

Series: iommu/vt-d: Enable ATS before cache tag assignment (rev2)
URL   : https://patchwork.freedesktop.org/series/150664/
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
f8ff75ae1d2127635239b134695774ed4045d05b
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit e62871f3161a1a7546fa770d53afaec196cb71a9
Author: Lu Baolu <baolu.lu at linux.intel.com>
Date:   Tue Jun 24 14:54:12 2025 +0530

    iommu/vt-d: Enable ATS before cache tag assignment
    
    This is a core-for-CI patch for
    https://lore.kernel.org/linux-iommu/20250620060802.3036137-1-baolu.lu@linux.intel.com/
    
    Commit <4f1492efb495> ("iommu/vt-d: Revert ATS timing change to fix boot
    failure") placed the enabling of ATS in the probe_finalize callback. This
    occurs after the default domain attachment, which is when the ATS cache
    tag is assigned. Consequently, the device TLB cache tag is missed when the
    domain is attached, leading to the device TLB not being invalidated in the
    iommu_unmap paths.
    
    Fix it by moving the ATS enabling to the default domain attachment path,
    ensuring ATS is enabled before the cache tag assignment.
    
    Fixes: 4f1492efb495 ("iommu/vt-d: Revert ATS timing change to fix boot failure")
    Link: https://lore.kernel.org/linux-iommu/20250620060802.3036137-1-baolu.lu@linux.intel.com/
    References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14531
    Signed-off-by: Lu Baolu <baolu.lu at linux.intel.com>
    Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
+ /mt/dim checkpatch fb5dada21e3cfa26179ca58e1d7c26cdad217201 drm-intel
e62871f3161a iommu/vt-d: Enable ATS before cache tag assignment
-:7: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#7: 
https://lore.kernel.org/linux-iommu/20250620060802.3036137-1-baolu.lu@linux.intel.com/

-:21: WARNING:COMMIT_LOG_USE_LINK: Unknown link reference 'References:', use 'Link:' or 'Closes:' instead
#21: 
References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14531

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




More information about the Intel-xe mailing list