]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
RDMA/uverbs: fix some kernel-doc warnings
authorRandy Dunlap <rdunlap@infradead.org>
Mon, 20 Oct 2025 03:43:20 +0000 (20:43 -0700)
committerLeon Romanovsky <leon@kernel.org>
Mon, 20 Oct 2025 15:45:46 +0000 (11:45 -0400)
Fix 49 kernel-doc warnings in ib_verbs.h:

- Add struct short description for rdma_stat_desc, rdma_hw_stats.
- Fix kernel-doc format for struct members (use ':' instead of '-') for
  several structs.
- Don't use "/**" kernel-doc notation for struct members in ib_device_ops
  (most members are not documented and most of the kernel-doc was
  not formatted correctly).
- Spell function parameters correctly in ib_dma_map_sgtable_attrs(),
  ib_device_try_get(), rdma_roce_rescan_device().
- Add kernel-doc for the function parameter in
  rdma_flow_label_to_udp_sport().

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Link: https://patch.msgid.link/20251020034320.3011094-1-rdunlap@infradead.org
Signed-off-by: Leon Romanovsky <leon@kernel.org>
include/rdma/ib_verbs.h

index 6139223e92e4c2f26686595279836af20a261da5..0a85af610b6b72db33ddd90b30163e18f7038e7d 100644 (file)
@@ -586,10 +586,10 @@ enum ib_stat_flag {
 };
 
 /**
- * struct rdma_stat_desc
- * @name - The name of the counter
- * @flags - Flags of the counter; For example, IB_STAT_FLAG_OPTIONAL
- * @priv - Driver private information; Core code should not use
+ * struct rdma_stat_desc - description of one rdma stat/counter
+ * @name: The name of the counter
+ * @flags: Flags of the counter; For example, IB_STAT_FLAG_OPTIONAL
+ * @priv: Driver private information; Core code should not use
  */
 struct rdma_stat_desc {
        const char *name;
@@ -598,24 +598,24 @@ struct rdma_stat_desc {
 };
 
 /**
- * struct rdma_hw_stats
- * @lock - Mutex to protect parallel write access to lifespan and values
+ * struct rdma_hw_stats - collection of hardware stats and their management
+ * @lock: Mutex to protect parallel write access to lifespan and values
  *    of counters, which are 64bits and not guaranteed to be written
  *    atomicaly on 32bits systems.
- * @timestamp - Used by the core code to track when the last update was
- * @lifespan - Used by the core code to determine how old the counters
+ * @timestamp: Used by the core code to track when the last update was
+ * @lifespan: Used by the core code to determine how old the counters
  *   should be before being updated again.  Stored in jiffies, defaults
  *   to 10 milliseconds, drivers can override the default be specifying
  *   their own value during their allocation routine.
- * @descs - Array of pointers to static descriptors used for the counters
+ * @descs: Array of pointers to static descriptors used for the counters
  *   in directory.
- * @is_disabled - A bitmap to indicate each counter is currently disabled
+ * @is_disabled: A bitmap to indicate each counter is currently disabled
  *   or not.
- * @num_counters - How many hardware counters there are.  If name is
+ * @num_counters: How many hardware counters there are.  If name is
  *   shorter than this number, a kernel oops will result.  Driver authors
  *   are encouraged to leave BUILD_BUG_ON(ARRAY_SIZE(@name) < num_counters)
  *   in their code to prevent this.
- * @value - Array of u64 counters that are accessed by the sysfs code and
+ * @value: Array of u64 counters that are accessed by the sysfs code and
  *   filled in by the drivers get_stats routine
  */
 struct rdma_hw_stats {
@@ -2405,7 +2405,7 @@ struct ib_device_ops {
        int (*modify_port)(struct ib_device *device, u32 port_num,
                           int port_modify_mask,
                           struct ib_port_modify *port_modify);
-       /**
+       /*
         * The following mandatory functions are used only at device
         * registration.  Keep functions such as these at the end of this
         * structure to avoid cache line misses when accessing struct ib_device
@@ -2415,7 +2415,7 @@ struct ib_device_ops {
                                  struct ib_port_immutable *immutable);
        enum rdma_link_layer (*get_link_layer)(struct ib_device *device,
                                               u32 port_num);
-       /**
+       /*
         * When calling get_netdev, the HW vendor's driver should return the
         * net device of device @device at port @port_num or NULL if such
         * a net device doesn't exist. The vendor driver should call dev_hold
@@ -2425,7 +2425,7 @@ struct ib_device_ops {
         */
        struct net_device *(*get_netdev)(struct ib_device *device,
                                         u32 port_num);
-       /**
+       /*
         * rdma netdev operation
         *
         * Driver implementing alloc_rdma_netdev or rdma_netdev_get_params
@@ -2439,14 +2439,14 @@ struct ib_device_ops {
        int (*rdma_netdev_get_params)(struct ib_device *device, u32 port_num,
                                      enum rdma_netdev_t type,
                                      struct rdma_netdev_alloc_params *params);
-       /**
+       /*
         * query_gid should be return GID value for @device, when @port_num
         * link layer is either IB or iWarp. It is no-op if @port_num port
         * is RoCE link layer.
         */
        int (*query_gid)(struct ib_device *device, u32 port_num, int index,
                         union ib_gid *gid);
-       /**
+       /*
         * When calling add_gid, the HW vendor's driver should add the gid
         * of device of port at gid index available at @attr. Meta-info of
         * that gid (for example, the network device related to this gid) is
@@ -2460,7 +2460,7 @@ struct ib_device_ops {
         * roce_gid_table is used.
         */
        int (*add_gid)(const struct ib_gid_attr *attr, void **context);
-       /**
+       /*
         * When calling del_gid, the HW vendor's driver should delete the
         * gid of device @device at gid index gid_index of port port_num
         * available in @attr.
@@ -2475,7 +2475,7 @@ struct ib_device_ops {
                              struct ib_udata *udata);
        void (*dealloc_ucontext)(struct ib_ucontext *context);
        int (*mmap)(struct ib_ucontext *context, struct vm_area_struct *vma);
-       /**
+       /*
         * This will be called once refcount of an entry in mmap_xa reaches
         * zero. The type of the memory that was mapped may differ between
         * entries and is opaque to the rdma_user_mmap interface.
@@ -2516,12 +2516,12 @@ struct ib_device_ops {
        int (*modify_cq)(struct ib_cq *cq, u16 cq_count, u16 cq_period);
        int (*destroy_cq)(struct ib_cq *cq, struct ib_udata *udata);
        int (*resize_cq)(struct ib_cq *cq, int cqe, struct ib_udata *udata);
-       /**
+       /*
         * pre_destroy_cq - Prevent a cq from generating any new work
         * completions, but not free any kernel resources
         */
        int (*pre_destroy_cq)(struct ib_cq *cq);
-       /**
+       /*
         * post_destroy_cq - Free all kernel resources
         */
        void (*post_destroy_cq)(struct ib_cq *cq);
@@ -2615,7 +2615,7 @@ struct ib_device_ops {
                            struct scatterlist *meta_sg, int meta_sg_nents,
                            unsigned int *meta_sg_offset);
 
-       /**
+       /*
         * alloc_hw_[device,port]_stats - Allocate a struct rdma_hw_stats and
         *   fill in the driver initialized data.  The struct is kfree()'ed by
         *   the sysfs core when the device is removed.  A lifespan of -1 in the
@@ -2624,7 +2624,7 @@ struct ib_device_ops {
        struct rdma_hw_stats *(*alloc_hw_device_stats)(struct ib_device *device);
        struct rdma_hw_stats *(*alloc_hw_port_stats)(struct ib_device *device,
                                                     u32 port_num);
-       /**
+       /*
         * get_hw_stats - Fill in the counter value(s) in the stats struct.
         * @index - The index in the value array we wish to have updated, or
         *   num_counters if we want all stats updated
@@ -2639,14 +2639,14 @@ struct ib_device_ops {
        int (*get_hw_stats)(struct ib_device *device,
                            struct rdma_hw_stats *stats, u32 port, int index);
 
-       /**
+       /*
         * modify_hw_stat - Modify the counter configuration
         * @enable: true/false when enable/disable a counter
         * Return codes - 0 on success or error code otherwise.
         */
        int (*modify_hw_stat)(struct ib_device *device, u32 port,
                              unsigned int counter_index, bool enable);
-       /**
+       /*
         * Allows rdma drivers to add their own restrack attributes.
         */
        int (*fill_res_mr_entry)(struct sk_buff *msg, struct ib_mr *ibmr);
@@ -2682,39 +2682,39 @@ struct ib_device_ops {
                         u8 pdata_len);
        int (*iw_create_listen)(struct iw_cm_id *cm_id, int backlog);
        int (*iw_destroy_listen)(struct iw_cm_id *cm_id);
-       /**
+       /*
         * counter_bind_qp - Bind a QP to a counter.
         * @counter - The counter to be bound. If counter->id is zero then
         *   the driver needs to allocate a new counter and set counter->id
         */
        int (*counter_bind_qp)(struct rdma_counter *counter, struct ib_qp *qp,
                               u32 port);
-       /**
+       /*
         * counter_unbind_qp - Unbind the qp from the dynamically-allocated
         *   counter and bind it onto the default one
         */
        int (*counter_unbind_qp)(struct ib_qp *qp, u32 port);
-       /**
+       /*
         * counter_dealloc -De-allocate the hw counter
         */
        int (*counter_dealloc)(struct rdma_counter *counter);
-       /**
+       /*
         * counter_alloc_stats - Allocate a struct rdma_hw_stats and fill in
         * the driver initialized data.
         */
        struct rdma_hw_stats *(*counter_alloc_stats)(
                struct rdma_counter *counter);
-       /**
+       /*
         * counter_update_stats - Query the stats value of this counter
         */
        int (*counter_update_stats)(struct rdma_counter *counter);
 
-       /**
+       /*
         * counter_init - Initialize the driver specific rdma counter struct.
         */
        void (*counter_init)(struct rdma_counter *counter);
 
-       /**
+       /*
         * Allows rdma drivers to add their own restrack attributes
         * dumped via 'rdma stat' iproute2 command.
         */
@@ -2730,25 +2730,25 @@ struct ib_device_ops {
         */
        int (*get_numa_node)(struct ib_device *dev);
 
-       /**
+       /*
         * add_sub_dev - Add a sub IB device
         */
        struct ib_device *(*add_sub_dev)(struct ib_device *parent,
                                         enum rdma_nl_dev_type type,
                                         const char *name);
 
-       /**
+       /*
         * del_sub_dev - Delete a sub IB device
         */
        void (*del_sub_dev)(struct ib_device *sub_dev);
 
-       /**
+       /*
         * ufile_cleanup - Attempt to cleanup ubojects HW resources inside
         * the ufile.
         */
        void (*ufile_hw_cleanup)(struct ib_uverbs_file *ufile);
 
-       /**
+       /*
         * report_port_event - Drivers need to implement this if they have
         * some private stuff to handle when link status changes.
         */
@@ -3157,8 +3157,8 @@ static inline u32 rdma_start_port(const struct ib_device *device)
 
 /**
  * rdma_for_each_port - Iterate over all valid port numbers of the IB device
- * @device - The struct ib_device * to iterate over
- * @iter - The unsigned int to store the port number
+ * @device: The struct ib_device * to iterate over
+ * @iter: The unsigned int to store the port number
  */
 #define rdma_for_each_port(device, iter)                                       \
        for (iter = rdma_start_port(device +                                   \
@@ -3524,7 +3524,7 @@ static inline bool rdma_core_cap_opa_port(struct ib_device *device,
 /**
  * rdma_mtu_enum_to_int - Return the mtu of the port as an integer value.
  * @device: Device
- * @port_num: Port number
+ * @port: Port number
  * @mtu: enum value of MTU
  *
  * Return the MTU size supported by the port as an integer value. Will return
@@ -3542,7 +3542,7 @@ static inline int rdma_mtu_enum_to_int(struct ib_device *device, u32 port,
 /**
  * rdma_mtu_from_attr - Return the mtu of the port from the port attribute.
  * @device: Device
- * @port_num: Port number
+ * @port: Port number
  * @attr: port attribute
  *
  * Return the MTU size supported by the port as an integer value.
@@ -3919,7 +3919,7 @@ static inline int ib_destroy_qp(struct ib_qp *qp)
 
 /**
  * ib_open_qp - Obtain a reference to an existing sharable QP.
- * @xrcd - XRC domain
+ * @xrcd: XRC domain
  * @qp_open_attr: Attributes identifying the QP to open.
  *
  * Returns a reference to a sharable QP.
@@ -4273,9 +4273,9 @@ static inline void ib_dma_unmap_sg_attrs(struct ib_device *dev,
 /**
  * ib_dma_map_sgtable_attrs - Map a scatter/gather table to DMA addresses
  * @dev: The device for which the DMA addresses are to be created
- * @sg: The sg_table object describing the buffer
+ * @sgt: The sg_table object describing the buffer
  * @direction: The direction of the DMA
- * @attrs: Optional DMA attributes for the map operation
+ * @dma_attrs: Optional DMA attributes for the map operation
  */
 static inline int ib_dma_map_sgtable_attrs(struct ib_device *dev,
                                           struct sg_table *sgt,
@@ -4419,8 +4419,8 @@ struct ib_mr *ib_alloc_mr_integrity(struct ib_pd *pd,
 /**
  * ib_update_fast_reg_key - updates the key portion of the fast_reg MR
  *   R_Key and L_Key.
- * @mr - struct ib_mr pointer to be updated.
- * @newkey - new key to be used.
+ * @mr: struct ib_mr pointer to be updated.
+ * @newkey: new key to be used.
  */
 static inline void ib_update_fast_reg_key(struct ib_mr *mr, u8 newkey)
 {
@@ -4431,7 +4431,7 @@ static inline void ib_update_fast_reg_key(struct ib_mr *mr, u8 newkey)
 /**
  * ib_inc_rkey - increments the key portion of the given rkey. Can be used
  * for calculating a new rkey for type 2 memory windows.
- * @rkey - the rkey to increment.
+ * @rkey: the rkey to increment.
  */
 static inline u32 ib_inc_rkey(u32 rkey)
 {
@@ -4525,7 +4525,7 @@ int ib_check_mr_status(struct ib_mr *mr, u32 check_mask,
 
 /**
  * ib_device_try_get: Hold a registration lock
- * device: The device to lock
+ * @dev: The device to lock
  *
  * A device under an active registration lock cannot become unregistered. It
  * is only possible to obtain a registration lock on a device that is fully
@@ -4832,7 +4832,7 @@ ib_get_vector_affinity(struct ib_device *device, int comp_vector)
  * rdma_roce_rescan_device - Rescan all of the network devices in the system
  * and add their gids, as needed, to the relevant RoCE devices.
  *
- * @device:         the rdma device
+ * @ibdev:         the rdma device
  */
 void rdma_roce_rescan_device(struct ib_device *ibdev);
 void rdma_roce_rescan_port(struct ib_device *ib_dev, u32 port);
@@ -4885,7 +4885,7 @@ static inline struct ib_device *rdma_device_to_ibdev(struct device *device)
 
 /**
  * ibdev_to_node - return the NUMA node for a given ib_device
- * @dev:       device to get the NUMA node for.
+ * @ibdev:     device to get the NUMA node for.
  */
 static inline int ibdev_to_node(struct ib_device *ibdev)
 {
@@ -4923,6 +4923,7 @@ static inline struct net *rdma_dev_net(struct ib_device *device)
 /**
  * rdma_flow_label_to_udp_sport - generate a RoCE v2 UDP src port value based
  *                               on the flow_label
+ * @fl: flow_label value
  *
  * This function will convert the 20 bit flow_label input to a valid RoCE v2
  * UDP src port 14 bit value. All RoCE V2 drivers should use this same