Hi,
Here are assorted kernel-doc fixes for 6.19 cycle. As the name implies, for the merging strategy, the patches can be taken by respective maintainers to appropriate fixes branches (targetting 6.19 of course) (e.g. for mm it will be mm-hotfixes).
Enjoy!
Bagas Sanjaya (14): genalloc: Describe @start_addr parameter in genpool_algo_t mm: Describe @flags parameter in memalloc_flags_save() textsearch: Describe @list member in ts_ops search mm: vmalloc: Fix up vrealloc_node_align() kernel-doc macro name mm, kfence: Describe @slab parameter in __kfence_obj_info() virtio: Describe @map and @vmap members in virtio_device struct fs: Describe @isnew parameter in ilookup5_nowait() VFS: fix __start_dirop() kernel-doc warnings drm/amd/display: Don't use kernel-doc comment in dc_register_software_state struct drm/amdgpu: Describe @AMD_IP_BLOCK_TYPE_RAS in amd_ip_block_type enum drm/gem/shmem: Describe @shmem and @size parameters drm/scheduler: Describe @result in drm_sched_job_done() drm/gpusvm: Fix drm_gpusvm_pages_valid_unlocked() kernel-doc comment net: bridge: Describe @tunnel_hash member in net_bridge_vlan_group struct
drivers/gpu/drm/amd/display/dc/dc.h | 2 +- drivers/gpu/drm/amd/include/amd_shared.h | 1 + drivers/gpu/drm/drm_gem_shmem_helper.c | 3 ++- drivers/gpu/drm/drm_gpusvm.c | 4 ++-- drivers/gpu/drm/scheduler/sched_main.c | 1 + fs/inode.c | 1 + fs/namei.c | 3 ++- include/linux/genalloc.h | 1 + include/linux/kfence.h | 1 + include/linux/sched/mm.h | 1 + include/linux/textsearch.h | 1 + include/linux/virtio.h | 2 ++ mm/vmalloc.c | 2 +- net/bridge/br_private.h | 1 + 14 files changed, 18 insertions(+), 6 deletions(-)
base-commit: 8f0b4cce4481fb22653697cced8d0d04027cb1e8
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/genalloc.h:52 function parameter 'start_addr' not described in 'genpool_algo_t'
Describe @start_addr to fix it.
Fixes: 52fbf1134d4792 ("lib/genalloc.c: fix allocation of aligned buffer from non-aligned chunk") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- include/linux/genalloc.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/genalloc.h b/include/linux/genalloc.h index 0bd581003cd5df..0ee23ddd0acd3a 100644 --- a/include/linux/genalloc.h +++ b/include/linux/genalloc.h @@ -44,6 +44,7 @@ struct gen_pool; * @nr: The number of zeroed bits we're looking for * @data: optional additional data used by the callback * @pool: the pool being allocated from + * @start_addr: chunk start address */ typedef unsigned long (*genpool_algo_t)(unsigned long *map, unsigned long size,
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/sched/mm.h:332 function parameter 'flags' not described in 'memalloc_flags_save'
Describe @flags to fix it.
Fixes: 3f6d5e6a468d02 ("mm: introduce memalloc_flags_{save,restore}") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- include/linux/sched/mm.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/sched/mm.h b/include/linux/sched/mm.h index 0e1d73955fa511..95d0040df58413 100644 --- a/include/linux/sched/mm.h +++ b/include/linux/sched/mm.h @@ -325,6 +325,7 @@ static inline void might_alloc(gfp_t gfp_mask)
/** * memalloc_flags_save - Add a PF_* flag to current->flags, save old value + * @flags: Flags to add. * * This allows PF_* flags to be conveniently added, irrespective of current * value, and then the old version restored with memalloc_flags_restore().
On 12/15/25 12:38, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/sched/mm.h:332 function parameter 'flags' not described in 'memalloc_flags_save'
Describe @flags to fix it.
Fixes: 3f6d5e6a468d02 ("mm: introduce memalloc_flags_{save,restore}") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
include/linux/sched/mm.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/sched/mm.h b/include/linux/sched/mm.h index 0e1d73955fa511..95d0040df58413 100644 --- a/include/linux/sched/mm.h +++ b/include/linux/sched/mm.h @@ -325,6 +325,7 @@ static inline void might_alloc(gfp_t gfp_mask) /**
- memalloc_flags_save - Add a PF_* flag to current->flags, save old value
- @flags: Flags to add.
- This allows PF_* flags to be conveniently added, irrespective of current
- value, and then the old version restored with memalloc_flags_restore().
Acked-by: David Hildenbrand (Red Hat) david@kernel.org
On Mon, Dec 15, 2025 at 06:38:50PM +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/sched/mm.h:332 function parameter 'flags' not described in 'memalloc_flags_save'
Describe @flags to fix it.
Fixes: 3f6d5e6a468d02 ("mm: introduce memalloc_flags_{save,restore}") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
Acked-by: Harry Yoo harry.yoo@oracle.com
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/textsearch.h:49 struct member 'list' not described in 'ts_ops'
Describe @list member to fix it.
Cc: Thomas Graf tgraf@suug.ch Cc: "David S. Miller" davem@davemloft.net Fixes: 2de4ff7bd658c9 ("[LIB]: Textsearch infrastructure.") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- include/linux/textsearch.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/textsearch.h b/include/linux/textsearch.h index 6673e4d4ac2e1b..4933777404d618 100644 --- a/include/linux/textsearch.h +++ b/include/linux/textsearch.h @@ -35,6 +35,7 @@ struct ts_state * @get_pattern: return head of pattern * @get_pattern_len: return length of pattern * @owner: module reference to algorithm + * @list: list to search */ struct ts_ops {
Sphinx reports kernel-doc warning:
WARNING: ./mm/vmalloc.c:4284 expecting prototype for vrealloc_node_align_noprof(). Prototype was for vrealloc_node_align() instead
Fix the macro name in vrealloc_node_align_noprof() kernel-doc comment.
Fixes: 4c5d3365882dbb ("mm/vmalloc: allow to set node and align in vrealloc") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- mm/vmalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c index ecbac900c35f9c..2c3db9fefeb7ab 100644 --- a/mm/vmalloc.c +++ b/mm/vmalloc.c @@ -4248,7 +4248,7 @@ void *vzalloc_node_noprof(unsigned long size, int node) EXPORT_SYMBOL(vzalloc_node_noprof);
/** - * vrealloc_node_align_noprof - reallocate virtually contiguous memory; contents + * vrealloc_node_align - reallocate virtually contiguous memory; contents * remain unchanged * @p: object to reallocate memory for * @size: the size to reallocate
On Mon, Dec 15, 2025 at 06:38:52PM +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./mm/vmalloc.c:4284 expecting prototype for vrealloc_node_align_noprof(). Prototype was for vrealloc_node_align() instead
Fix the macro name in vrealloc_node_align_noprof() kernel-doc comment.
Fixes: 4c5d3365882dbb ("mm/vmalloc: allow to set node and align in vrealloc") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
LGTM. Reviewed-by: Vishal Moola (Oracle) vishal.moola@gmail.com
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/kfence.h:220 function parameter 'slab' not described in '__kfence_obj_info'
Fix it by describing @slab parameter.
Fixes: 2dfe63e61cc31e ("mm, kfence: support kmem_dump_obj() for KFENCE objects") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- include/linux/kfence.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/kfence.h b/include/linux/kfence.h index 0ad1ddbb8b996a..e5822f6e7f2794 100644 --- a/include/linux/kfence.h +++ b/include/linux/kfence.h @@ -211,6 +211,7 @@ struct kmem_obj_info; * __kfence_obj_info() - fill kmem_obj_info struct * @kpp: kmem_obj_info to be filled * @object: the object + * @slab: the slab * * Return: * * false - not a KFENCE object
On Mon, 15 Dec 2025 at 12:39, Bagas Sanjaya bagasdotme@gmail.com wrote:
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/kfence.h:220 function parameter 'slab' not described in '__kfence_obj_info'
Fix it by describing @slab parameter.
Fixes: 2dfe63e61cc31e ("mm, kfence: support kmem_dump_obj() for KFENCE objects") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
Acked-by: Marco Elver elver@google.com
Thanks!
include/linux/kfence.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/kfence.h b/include/linux/kfence.h index 0ad1ddbb8b996a..e5822f6e7f2794 100644 --- a/include/linux/kfence.h +++ b/include/linux/kfence.h @@ -211,6 +211,7 @@ struct kmem_obj_info;
- __kfence_obj_info() - fill kmem_obj_info struct
- @kpp: kmem_obj_info to be filled
- @object: the object
- @slab: the slab
- Return:
- false - not a KFENCE object
-- An old man doll... just what I always wanted! - Clara
On 12/15/25 12:38, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/kfence.h:220 function parameter 'slab' not described in '__kfence_obj_info'
Fix it by describing @slab parameter.
Fixes: 2dfe63e61cc31e ("mm, kfence: support kmem_dump_obj() for KFENCE objects") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
include/linux/kfence.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/linux/kfence.h b/include/linux/kfence.h index 0ad1ddbb8b996a..e5822f6e7f2794 100644 --- a/include/linux/kfence.h +++ b/include/linux/kfence.h @@ -211,6 +211,7 @@ struct kmem_obj_info;
- __kfence_obj_info() - fill kmem_obj_info struct
- @kpp: kmem_obj_info to be filled
- @object: the object
- @slab: the slab
- Return:
- false - not a KFENCE object
Acked-by: David Hildenbrand (Red Hat) david@kernel.org
On Mon, Dec 15, 2025 at 06:38:53PM +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./include/linux/kfence.h:220 function parameter 'slab' not described in '__kfence_obj_info'
Fix it by describing @slab parameter.
Fixes: 2dfe63e61cc31e ("mm, kfence: support kmem_dump_obj() for KFENCE objects") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
Acked-by: Harry Yoo harry.yoo@oracle.com
Sphinx reports kernel-doc warnings:
WARNING: ./include/linux/virtio.h:181 struct member 'map' not described in 'virtio_device' WARNING: ./include/linux/virtio.h:181 struct member 'vmap' not described in 'virtio_device'
Describe these members.
Fixes: bee8c7c24b7373 ("virtio: introduce map ops in virtio core") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- include/linux/virtio.h | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/include/linux/virtio.h b/include/linux/virtio.h index 132a474e59140a..68ead8fda9c921 100644 --- a/include/linux/virtio.h +++ b/include/linux/virtio.h @@ -150,11 +150,13 @@ struct virtio_admin_cmd { * @id: the device type identification (used to match it with a driver). * @config: the configuration ops for this device. * @vringh_config: configuration ops for host vrings. + * @map: configuration ops for device's mapping buffer * @vqs: the list of virtqueues for this device. * @features: the 64 lower features supported by both driver and device. * @features_array: the full features space supported by both driver and * device. * @priv: private pointer for the driver's use. + * @vmap: device virtual map * @debugfs_dir: debugfs directory entry. * @debugfs_filter_features: features to be filtered set by debugfs. */
On Mon, Dec 15, 2025 at 06:38:54PM +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warnings:
WARNING: ./include/linux/virtio.h:181 struct member 'map' not described in 'virtio_device' WARNING: ./include/linux/virtio.h:181 struct member 'vmap' not described in 'virtio_device'
Describe these members.
Fixes: bee8c7c24b7373 ("virtio: introduce map ops in virtio core") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
Acked-by: Michael S. Tsirkin mst@redhat.com
include/linux/virtio.h | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/include/linux/virtio.h b/include/linux/virtio.h index 132a474e59140a..68ead8fda9c921 100644 --- a/include/linux/virtio.h +++ b/include/linux/virtio.h @@ -150,11 +150,13 @@ struct virtio_admin_cmd {
- @id: the device type identification (used to match it with a driver).
- @config: the configuration ops for this device.
- @vringh_config: configuration ops for host vrings.
- @map: configuration ops for device's mapping buffer
- @vqs: the list of virtqueues for this device.
- @features: the 64 lower features supported by both driver and device.
- @features_array: the full features space supported by both driver and
device.- @priv: private pointer for the driver's use.
*/
- @vmap: device virtual map
- @debugfs_dir: debugfs directory entry.
- @debugfs_filter_features: features to be filtered set by debugfs.
-- An old man doll... just what I always wanted! - Clara
On Mon, Dec 15, 2025 at 7:39 PM Bagas Sanjaya bagasdotme@gmail.com wrote:
Sphinx reports kernel-doc warnings:
WARNING: ./include/linux/virtio.h:181 struct member 'map' not described in 'virtio_device' WARNING: ./include/linux/virtio.h:181 struct member 'vmap' not described in 'virtio_device'
Describe these members.
Fixes: bee8c7c24b7373 ("virtio: introduce map ops in virtio core") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
Acked-by: Jason Wang jasowang@redhat.com
Thanks
Sphinx reports kernel-doc warning:
WARNING: ./fs/inode.c:1607 function parameter 'isnew' not described in 'ilookup5_nowait'
Describe the parameter.
Fixes: a27628f4363435 ("fs: rework I_NEW handling to operate without fences") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- fs/inode.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/fs/inode.c b/fs/inode.c index 521383223d8a45..2f4beda7bb8841 100644 --- a/fs/inode.c +++ b/fs/inode.c @@ -1593,6 +1593,7 @@ EXPORT_SYMBOL(igrab); * @hashval: hash value (usually inode number) to search for * @test: callback used for comparisons between inodes * @data: opaque data pointer to pass to @test + * @isnew: whether the inode is new or not * * Search for the inode specified by @hashval and @data in the inode cache. * If the inode is in the cache, the inode is returned with an incremented
On Mon 15-12-25 18:38:55, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./fs/inode.c:1607 function parameter 'isnew' not described in 'ilookup5_nowait'
Describe the parameter.
Fixes: a27628f4363435 ("fs: rework I_NEW handling to operate without fences") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
...
@@ -1593,6 +1593,7 @@ EXPORT_SYMBOL(igrab);
- @hashval: hash value (usually inode number) to search for
- @test: callback used for comparisons between inodes
- @data: opaque data pointer to pass to @test
- @isnew: whether the inode is new or not
I'm sorry but this is true but misleading at the same time. I'd write there something like:
* @isnew: return argument telling whether I_NEW was set when the inode was found in hash (the caller needs to wait for I_NEW to clear).
Honza
Sphinx report kernel-doc warnings:
WARNING: ./fs/namei.c:2853 function parameter 'state' not described in '__start_dirop' WARNING: ./fs/namei.c:2853 expecting prototype for start_dirop(). Prototype was for __start_dirop() instead
Fix them up.
Fixes: ff7c4ea11a05c8 ("VFS: add start_creating_killable() and start_removing_killable()") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- fs/namei.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/fs/namei.c b/fs/namei.c index bf0f66f0e9b92c..91fd3a786704e2 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -2836,10 +2836,11 @@ static int filename_parentat(int dfd, struct filename *name, }
/** - * start_dirop - begin a create or remove dirop, performing locking and lookup + * __start_dirop - begin a create or remove dirop, performing locking and lookup * @parent: the dentry of the parent in which the operation will occur * @name: a qstr holding the name within that parent * @lookup_flags: intent and other lookup flags. + * @state: task state bitmask * * The lookup is performed and necessary locks are taken so that, on success, * the returned dentry can be operated on safely.
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/amd/display/dc/dc.h:2796 This comment starts with '/**', but isn't a kernel-doc comment. Refer to Documentation/doc-guide/kernel-doc.rst * Software state variables used to program register fields across the display pipeline
Don't use kernel-doc comment syntax to fix it.
Fixes: b0ff344fe70cd2 ("drm/amd/display: Add interface to capture expected HW state from SW state") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- drivers/gpu/drm/amd/display/dc/dc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h index 29edfa51ea2cc0..0a9758a042586f 100644 --- a/drivers/gpu/drm/amd/display/dc/dc.h +++ b/drivers/gpu/drm/amd/display/dc/dc.h @@ -2793,7 +2793,7 @@ void dc_get_underflow_debug_data_for_otg(struct dc *dc, int primary_otg_inst, st
void dc_get_power_feature_status(struct dc *dc, int primary_otg_inst, struct power_features *out_data);
-/** +/* * Software state variables used to program register fields across the display pipeline */ struct dc_register_software_state {
Applied. Thanks!
On Mon, Dec 15, 2025 at 6:41 AM Bagas Sanjaya bagasdotme@gmail.com wrote:
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/amd/display/dc/dc.h:2796 This comment starts with '/**', but isn't a kernel-doc comment. Refer to Documentation/doc-guide/kernel-doc.rst
- Software state variables used to program register fields across the display pipeline
Don't use kernel-doc comment syntax to fix it.
Fixes: b0ff344fe70cd2 ("drm/amd/display: Add interface to capture expected HW state from SW state") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
drivers/gpu/drm/amd/display/dc/dc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h index 29edfa51ea2cc0..0a9758a042586f 100644 --- a/drivers/gpu/drm/amd/display/dc/dc.h +++ b/drivers/gpu/drm/amd/display/dc/dc.h @@ -2793,7 +2793,7 @@ void dc_get_underflow_debug_data_for_otg(struct dc *dc, int primary_otg_inst, st
void dc_get_power_feature_status(struct dc *dc, int primary_otg_inst, struct power_features *out_data);
-/** +/*
- Software state variables used to program register fields across the display pipeline
*/ struct dc_register_software_state { -- An old man doll... just what I always wanted! - Clara
Linaro-mm-sig mailing list -- linaro-mm-sig@lists.linaro.org To unsubscribe send an email to linaro-mm-sig-leave@lists.linaro.org
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/amd/include/amd_shared.h:113 Enum value 'AMD_IP_BLOCK_TYPE_RAS' not described in enum 'amd_ip_block_type'
Describe the value to fix it.
Fixes: 7169e706c82d7b ("drm/amdgpu: Add ras module ip block to amdgpu discovery") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- drivers/gpu/drm/amd/include/amd_shared.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h index 17945094a13834..d8ed3799649172 100644 --- a/drivers/gpu/drm/amd/include/amd_shared.h +++ b/drivers/gpu/drm/amd/include/amd_shared.h @@ -89,6 +89,7 @@ enum amd_apu_flags { * @AMD_IP_BLOCK_TYPE_VPE: Video Processing Engine * @AMD_IP_BLOCK_TYPE_UMSCH_MM: User Mode Scheduler for Multimedia * @AMD_IP_BLOCK_TYPE_ISP: Image Signal Processor +* @AMD_IP_BLOCK_TYPE_RAS: RAS * @AMD_IP_BLOCK_TYPE_NUM: Total number of IP block types */ enum amd_ip_block_type {
On Mon, Dec 15, 2025 at 6:48 AM Bagas Sanjaya bagasdotme@gmail.com wrote:
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/amd/include/amd_shared.h:113 Enum value 'AMD_IP_BLOCK_TYPE_RAS' not described in enum 'amd_ip_block_type'
Describe the value to fix it.
Fixes: 7169e706c82d7b ("drm/amdgpu: Add ras module ip block to amdgpu discovery") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
drivers/gpu/drm/amd/include/amd_shared.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h index 17945094a13834..d8ed3799649172 100644 --- a/drivers/gpu/drm/amd/include/amd_shared.h +++ b/drivers/gpu/drm/amd/include/amd_shared.h @@ -89,6 +89,7 @@ enum amd_apu_flags {
- @AMD_IP_BLOCK_TYPE_VPE: Video Processing Engine
- @AMD_IP_BLOCK_TYPE_UMSCH_MM: User Mode Scheduler for Multimedia
- @AMD_IP_BLOCK_TYPE_ISP: Image Signal Processor
+* @AMD_IP_BLOCK_TYPE_RAS: RAS
Reliability, Availability, Serviceability. I've fixed this up locally and applied the patch.
Thanks,
Alex
- @AMD_IP_BLOCK_TYPE_NUM: Total number of IP block types
*/ enum amd_ip_block_type { -- An old man doll... just what I always wanted! - Clara
Sphinx reports kernel-doc warnings:
WARNING: ./drivers/gpu/drm/drm_gem_shmem_helper.c:104 function parameter 'shmem' not described in 'drm_gem_shmem_init' WARNING: ./drivers/gpu/drm/drm_gem_shmem_helper.c:104 function parameter 'size' not described in 'drm_gem_shmem_init'
Describe the parameters.
Fixes: e3f4bdaf2c5bfe ("drm/gem/shmem: Extract drm_gem_shmem_init() from drm_gem_shmem_create()") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- drivers/gpu/drm/drm_gem_shmem_helper.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_gem_shmem_helper.c b/drivers/gpu/drm/drm_gem_shmem_helper.c index 93b9cff89080f9..7f73900abcbb9d 100644 --- a/drivers/gpu/drm/drm_gem_shmem_helper.c +++ b/drivers/gpu/drm/drm_gem_shmem_helper.c @@ -96,7 +96,8 @@ static int __drm_gem_shmem_init(struct drm_device *dev, struct drm_gem_shmem_obj /** * drm_gem_shmem_init - Initialize an allocated object. * @dev: DRM device - * @obj: The allocated shmem GEM object. + * @shmem: The allocated shmem GEM object. + * @size: shmem GEM object size * * Returns: * 0 on success, or a negative error code on failure.
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/scheduler/sched_main.c:367 function parameter 'result' not described in 'drm_sched_job_done'
Describe @result parameter to fix it.
Fixes: 539f9ee4b52a8b ("drm/scheduler: properly forward fence errors") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- drivers/gpu/drm/scheduler/sched_main.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/scheduler/sched_main.c b/drivers/gpu/drm/scheduler/sched_main.c index 1d4f1b822e7b76..4f844087fd48eb 100644 --- a/drivers/gpu/drm/scheduler/sched_main.c +++ b/drivers/gpu/drm/scheduler/sched_main.c @@ -361,6 +361,7 @@ static void drm_sched_run_free_queue(struct drm_gpu_scheduler *sched) /** * drm_sched_job_done - complete a job * @s_job: pointer to the job which is done + * @result: job result * * Finish the job's fence and resubmit the work items. */
nit about commit title: We use "drm/sched:" as prefix nowadays
On Mon, 2025-12-15 at 18:39 +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./drivers/gpu/drm/scheduler/sched_main.c:367 function parameter 'result' not described in 'drm_sched_job_done'
Describe @result parameter to fix it
Thx for fixing this!
.
Fixes: 539f9ee4b52a8b ("drm/scheduler: properly forward fence errors") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
drivers/gpu/drm/scheduler/sched_main.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/scheduler/sched_main.c b/drivers/gpu/drm/scheduler/sched_main.c index 1d4f1b822e7b76..4f844087fd48eb 100644 --- a/drivers/gpu/drm/scheduler/sched_main.c +++ b/drivers/gpu/drm/scheduler/sched_main.c @@ -361,6 +361,7 @@ static void drm_sched_run_free_queue(struct drm_gpu_scheduler *sched) /** * drm_sched_job_done - complete a job * @s_job: pointer to the job which is done
- @result: job result
"error code for the job's finished-fence" would be a bit better and more verbose.
With that:
Reviewed-by: Philipp Stanner phasta@kernel.org
* * Finish the job's fence and resubmit the work items. */
Commit 6364afd532bcab ("drm/gpusvm: refactor core API to use pages struct") renames drm_gpusvm_range_pages_valid_unlocked() to drm_gpusvm_pages_valid_unlocked(), but its kernel-doc comment gets stale, hence kernel-doc complains:
WARNING: ./drivers/gpu/drm/drm_gpusvm.c:1229 function parameter 'svm_pages' not described in 'drm_gpusvm_pages_valid_unlocked' WARNING: ./drivers/gpu/drm/drm_gpusvm.c:1229 expecting prototype for drm_gpusvm_range_pages_valid_unlocked(). Prototype was for drm_gpusvm_pages_valid_unlocked() instead
Fix them up.
Fixes: 6364afd532bcab ("drm/gpusvm: refactor core API to use pages struct") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- drivers/gpu/drm/drm_gpusvm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_gpusvm.c b/drivers/gpu/drm/drm_gpusvm.c index 73e550c8ff8c98..fcfbe8c062bf6d 100644 --- a/drivers/gpu/drm/drm_gpusvm.c +++ b/drivers/gpu/drm/drm_gpusvm.c @@ -1216,9 +1216,9 @@ bool drm_gpusvm_range_pages_valid(struct drm_gpusvm *gpusvm, EXPORT_SYMBOL_GPL(drm_gpusvm_range_pages_valid);
/** - * drm_gpusvm_range_pages_valid_unlocked() - GPU SVM range pages valid unlocked + * drm_gpusvm_pages_valid_unlocked() - GPU SVM range pages valid unlocked * @gpusvm: Pointer to the GPU SVM structure - * @range: Pointer to the GPU SVM range structure + * @svm_pages: Pointer to the GPU SVM pages * * This function determines if a GPU SVM range pages are valid. Expected be * called without holding gpusvm->notifier_lock.
Sphinx reports kernel-doc warning:
WARNING: ./net/bridge/br_private.h:267 struct member 'tunnel_hash' not described in 'net_bridge_vlan_group'
Fix it by describing @tunnel_hash member.
Fixes: efa5356b0d9753 ("bridge: per vlan dst_metadata netlink support") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com --- net/bridge/br_private.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h index 7280c4e9305f36..bf441ac1c4d38a 100644 --- a/net/bridge/br_private.h +++ b/net/bridge/br_private.h @@ -247,6 +247,7 @@ struct net_bridge_vlan { * struct net_bridge_vlan_group * * @vlan_hash: VLAN entry rhashtable + * @tunnel_hash: tunnel rhashtable * @vlan_list: sorted VLAN entry list * @num_vlans: number of total VLAN entries * @pvid: PVID VLAN id
On Mon, Dec 15, 2025 at 06:39:02PM +0700, Bagas Sanjaya wrote:
Sphinx reports kernel-doc warning:
WARNING: ./net/bridge/br_private.h:267 struct member 'tunnel_hash' not described in 'net_bridge_vlan_group'
Fix it by describing @tunnel_hash member.
Fixes: efa5356b0d9753 ("bridge: per vlan dst_metadata netlink support") Signed-off-by: Bagas Sanjaya bagasdotme@gmail.com
net/bridge/br_private.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h index 7280c4e9305f36..bf441ac1c4d38a 100644 --- a/net/bridge/br_private.h +++ b/net/bridge/br_private.h @@ -247,6 +247,7 @@ struct net_bridge_vlan {
- struct net_bridge_vlan_group
- @vlan_hash: VLAN entry rhashtable
- @tunnel_hash: tunnel rhashtable
While you are at it, I suggest making the comment a bit more useful. Something like:
@tunnel_hash: Hash table to map from tunnel key ID (e.g., VXLAN VNI) to VLAN
- @vlan_list: sorted VLAN entry list
- @num_vlans: number of total VLAN entries
- @pvid: PVID VLAN id
-- An old man doll... just what I always wanted! - Clara
On Mon, 15 Dec 2025 18:38:48 +0700 Bagas Sanjaya wrote:
Here are assorted kernel-doc fixes for 6.19 cycle. As the name implies, for the merging strategy, the patches can be taken by respective maintainers to appropriate fixes branches (targetting 6.19 of course) (e.g. for mm it will be mm-hotfixes).
Please submit just the relevant changes directly to respective subsystems. Maintainers don't have time to sort patches for you. You should know better.
On Tue, Dec 16, 2025 at 02:08:57PM -0800, Jakub Kicinski wrote:
On Mon, 15 Dec 2025 18:38:48 +0700 Bagas Sanjaya wrote:
Here are assorted kernel-doc fixes for 6.19 cycle. As the name implies, for the merging strategy, the patches can be taken by respective maintainers to appropriate fixes branches (targetting 6.19 of course) (e.g. for mm it will be mm-hotfixes).
Please submit just the relevant changes directly to respective subsystems. Maintainers don't have time to sort patches for you. You should know better.
OK, thanks!
linaro-mm-sig@lists.linaro.org
-
Alex Deucher -
Bagas Sanjaya -
David Hildenbrand (Red Hat) -
Harry Yoo -
Ido Schimmel -
Jakub Kicinski -
Jan Kara -
Jason Wang -
Marco Elver -
Michael S. Tsirkin -
Philipp Stanner -
Vishal Moola (Oracle)