]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/xe: Fix stale and mismatched kernel-doc member tags in header files
authorShuicheng Lin <shuicheng.lin@intel.com>
Tue, 14 Apr 2026 22:54:30 +0000 (22:54 +0000)
committerShuicheng Lin <shuicheng.lin@intel.com>
Thu, 16 Apr 2026 15:31:59 +0000 (08:31 -0700)
Fix kernel-doc member tags that do not match the actual struct member
names, which would cause warnings with scripts/kernel-doc or produce
incorrect documentation:

- xe_bo_types.h: @atomic_access -> @attr.atomic_access
- xe_device_types.h: @mem.sys_mgr -> @mem.shrinker,
  @pinned.external -> @pinned.late.external,
  @vram_usefault.list -> @mem_access.vram_userfault.list,
  @usm.asid -> @usm.asid_to_vm,
  @usm.pagemap_shrinker -> @usm.dpagemap_shrinker,
  @vm.xe -> @vm.xa
- xe_exec_queue_types.h: @last_fence -> @tlb_inval.last_fence
- xe_gt_sriov_pf_types.h: Remove stale @migration doc for non-existent
  member
- xe_gt_types.h: @info->engine_mask -> @info.engine_mask,
  @multi_queue_engine_class_mask -> @info.multi_queue_engine_class_mask
- xe_guc_ct_types.h: @ctbs.send -> @ctbs.h2g,
  @ctbs.recv -> @ctbs.g2h
- xe_guc_submit_types.h: @valid -> @multi_queue.valid
- xe_hw_engine_types.h: @sched_props.set_job_timeout ->
  @sched_props.job_timeout_ms
- xe_oa_types.h: duplicate @format -> @bo
- xe_vm_types.h: @read_only -> @invalidate_on_bind,
  add missing @vma_flags doc

Assisted-by: GitHub Copilot:claude-sonnet-4.6
Reviewed-by: Brian Nguyen <brian3.nguyen@intel.com>
Link: https://patch.msgid.link/20260414225457.3687449-2-shuicheng.lin@intel.com
Signed-off-by: Shuicheng Lin <shuicheng.lin@intel.com>
drivers/gpu/drm/xe/xe_bo_types.h
drivers/gpu/drm/xe/xe_device_types.h
drivers/gpu/drm/xe/xe_exec_queue_types.h
drivers/gpu/drm/xe/xe_gt_sriov_pf_types.h
drivers/gpu/drm/xe/xe_gt_types.h
drivers/gpu/drm/xe/xe_guc_ct_types.h
drivers/gpu/drm/xe/xe_guc_submit_types.h
drivers/gpu/drm/xe/xe_hw_engine_types.h
drivers/gpu/drm/xe/xe_oa_types.h
drivers/gpu/drm/xe/xe_vm_types.h

index 9d19940b8fc035935b109de88a9e337416157ae0..9c199badd9b27e0db25d14f20915838d8a304a42 100644 (file)
@@ -67,7 +67,7 @@ struct xe_bo {
        /** @attr: User controlled attributes for bo */
        struct {
                /**
-                * @atomic_access: type of atomic access bo needs
+                * @attr.atomic_access: type of atomic access bo needs
                 * protected by bo dma-resv lock
                 */
                u32 atomic_access;
index 150c76b2acaf1af0d30c4474929acf1d2f1a83a1..2ebe33a0dc074b38a1813a17f16d99a9506aaf23 100644 (file)
@@ -276,7 +276,7 @@ struct xe_device {
                struct xe_vram_region *vram;
                /** @mem.sys_mgr: system TTM manager */
                struct ttm_resource_manager sys_mgr;
-               /** @mem.sys_mgr: system memory shrinker. */
+               /** @mem.shrinker: system memory shrinker. */
                struct xe_shrinker *shrinker;
        } mem;
 
@@ -298,7 +298,7 @@ struct xe_device {
 
        /** @usm: unified memory state */
        struct {
-               /** @usm.asid: convert a ASID to VM */
+               /** @usm.asid_to_vm: convert an ASID to VM */
                struct xarray asid_to_vm;
                /** @usm.next_asid: next ASID, used to cyclical alloc asids */
                u32 next_asid;
@@ -315,7 +315,7 @@ struct xe_device {
                /** @usm.pf_queue: Page fault queues */
                struct xe_pagefault_queue pf_queue[XE_PAGEFAULT_QUEUE_COUNT];
 #if IS_ENABLED(CONFIG_DRM_XE_PAGEMAP)
-               /** @usm.pagemap_shrinker: Shrinker for unused pagemaps */
+               /** @usm.dpagemap_shrinker: Shrinker for unused pagemaps */
                struct drm_pagemap_shrinker *dpagemap_shrinker;
 #endif
        } usm;
@@ -337,7 +337,7 @@ struct xe_device {
                        struct list_head kernel_bo_present;
                        /** @pinned.late.evicted: pinned BO that have been evicted */
                        struct list_head evicted;
-                       /** @pinned.external: pinned external and dma-buf. */
+                       /** @pinned.late.external: pinned external and dma-buf. */
                        struct list_head external;
                } late;
        } pinned;
@@ -372,7 +372,7 @@ struct xe_device {
                struct {
                        /**
                         * @mem_access.vram_userfault.lock: Protects access to
-                        * @vram_usefault.list Using mutex instead of spinlock
+                        * @mem_access.vram_userfault.list Using mutex instead of spinlock
                         * as lock is applied to entire list operation which
                         * may sleep
                         */
@@ -584,7 +584,7 @@ struct xe_file {
 
        /** @vm: VM state for file */
        struct {
-               /** @vm.xe: xarray to store VMs */
+               /** @vm.xa: xarray to store VMs */
                struct xarray xa;
                /**
                 * @vm.lock: Protects VM lookup + reference and removal from
index 8ce78e0b1d502b50e8608975e4d6eb53f6c95236..2f5ccf294675afb17ad7f404ba98d8cfff8aef4d 100644 (file)
@@ -214,7 +214,7 @@ struct xe_exec_queue {
                 */
                struct xe_dep_scheduler *dep_scheduler;
                /**
-                * @last_fence: last fence for tlb invalidation, protected by
+                * @tlb_inval.last_fence: last fence for tlb invalidation, protected by
                 * vm->lock in write mode
                 */
                struct dma_fence *last_fence;
index 667b8310478d44c708d5db4b07db9be6f4df430a..6296483c44ec3825510d4f52bb1309325e935902 100644 (file)
@@ -49,7 +49,6 @@ struct xe_gt_sriov_pf_workers {
  * @service: service data.
  * @control: control data.
  * @policy: policy data.
- * @migration: migration data.
  * @spare: PF-only provisioning configuration.
  * @vfs: metadata for all VFs.
  */
index 8b55cf25a75fa14a38872dabb535d211f5366fd7..7351aadd238ed9666e93aaedf6e854350de0d9b9 100644 (file)
@@ -136,7 +136,7 @@ struct xe_gt {
                /** @info.gmdid: raw GMD_ID value from hardware */
                u32 gmdid;
                /**
-                * @multi_queue_engine_class_mask: Bitmask of engine classes with
+                * @info.multi_queue_engine_class_mask: Bitmask of engine classes with
                 * multi queue support enabled.
                 */
                u16 multi_queue_engine_class_mask;
@@ -355,7 +355,7 @@ struct xe_gt {
        /** @user_engines: engines present in GT and available to userspace */
        struct {
                /**
-                * @user_engines.mask: like @info->engine_mask, but take in
+                * @user_engines.mask: like @info.engine_mask, but take in
                 * consideration only engines available to userspace
                 */
                u64 mask;
index 5da1ce5dc3724398258d46ca9983c84df1197333..c81996f8915acc3c9d25bef70c32185dcf2a5d58 100644 (file)
@@ -134,9 +134,9 @@ struct xe_guc_ct {
        spinlock_t fast_lock;
        /** @ctbs: buffers for sending and receiving commands */
        struct {
-               /** @ctbs.send: Host to GuC (H2G, send) channel */
+               /** @ctbs.h2g: Host to GuC (H2G, send) channel */
                struct guc_ctb h2g;
-               /** @ctbs.recv: GuC to Host (G2H, receive) channel */
+               /** @ctbs.g2h: GuC to Host (G2H, receive) channel */
                struct guc_ctb g2h;
        } ctbs;
        /** @g2h_outstanding: number of outstanding G2H */
index 5ccc5f959bb3a6b844fccd4c85a97c94cebb5031..7824f61b12906b6e13bcabca5ff7e78c8313278c 100644 (file)
@@ -138,7 +138,7 @@ struct xe_guc_submit_exec_queue_snapshot {
                u32 primary;
                /** @multi_queue.pos: Position of the exec queue within the multi queue group */
                u8 pos;
-               /** @valid: The exec queue is part of a multi queue group */
+               /** @multi_queue.valid: The exec queue is part of a multi queue group */
                bool valid;
        } multi_queue;
 };
index e4191a7a2c31808c0828c6368fb823c2b4fb4a17..0f87128c6529090081e70ec0644a2ce2ad26d713 100644 (file)
@@ -79,7 +79,7 @@ struct xe_hw_engine_class_intf {
         * @defaults: default scheduling properties
         */
        struct {
-               /** @sched_props.set_job_timeout: Set job timeout in ms for engine */
+               /** @sched_props.job_timeout_ms: Set job timeout in ms for engine */
                u32 job_timeout_ms;
                /** @sched_props.job_timeout_min: Min job timeout in ms for engine */
                u32 job_timeout_min;
index b03ffd513483449adc6b2937c0707e5bfc619a5b..acd78bc1482252e001e8b4eb242888089300fd7d 100644 (file)
@@ -159,7 +159,7 @@ struct xe_oa_buffer {
        /** @format: data format */
        const struct xe_oa_format *format;
 
-       /** @format: xe_bo backing the OA buffer */
+       /** @bo: xe_bo backing the OA buffer */
        struct xe_bo *bo;
 
        /** @vaddr: mapped vaddr of the OA buffer */
index a94827d7fbeca6200ecb3fe4797610e69504c714..635ed29b9a69b0bfa678e6defead3885d490e0b7 100644 (file)
@@ -412,10 +412,11 @@ struct xe_vm {
 struct xe_vma_op_map {
        /** @vma: VMA to map */
        struct xe_vma *vma;
+       /** @vma_flags: VMA flags for this operation */
        unsigned int vma_flags;
        /** @immediate: Immediate bind */
        bool immediate;
-       /** @read_only: Read only */
+       /** @invalidate_on_bind: Invalidate on bind */
        bool invalidate_on_bind;
        /** @request_decompress: schedule decompression for GPU map */
        bool request_decompress;