RDMA: Fix kernel-doc warnings about wrong comment
authorLeon Romanovsky <leonro@nvidia.com>
Wed, 16 Jun 2021 06:09:47 +0000 (09:09 +0300)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 21 Jun 2021 23:32:50 +0000 (20:32 -0300)
Compilation with W=1 produces warnings similar to the below.

  drivers/infiniband/ulp/ipoib/ipoib_main.c:320: warning: This comment
starts with '/**', but isn't a kernel-doc comment. Refer
Documentation/doc-guide/kernel-doc.rst

All such occurrences were found with the following one line
 git grep -A 1 "\/\*\*" drivers/infiniband/

Link: https://lore.kernel.org/r/e57d5f4ddd08b7a19934635b44d6d632841b9ba7.1623823612.git.leonro@nvidia.com
Reviewed-by: Jack Wang <jinpu.wang@ionos.com> #rtrs
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>
Signed-off-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
16 files changed:
drivers/infiniband/core/iwpm_util.h
drivers/infiniband/core/roce_gid_mgmt.c
drivers/infiniband/hw/hfi1/chip.c
drivers/infiniband/hw/hfi1/file_ops.c
drivers/infiniband/hw/hfi1/hfi.h
drivers/infiniband/hw/hfi1/init.c
drivers/infiniband/hw/hfi1/pio.c
drivers/infiniband/hw/hns/hns_roce_hem.c
drivers/infiniband/sw/rdmavt/mr.c
drivers/infiniband/sw/rdmavt/qp.c
drivers/infiniband/sw/rdmavt/vt.c
drivers/infiniband/ulp/ipoib/ipoib_main.c
drivers/infiniband/ulp/iser/iser_verbs.c
drivers/infiniband/ulp/isert/ib_isert.c
drivers/infiniband/ulp/rtrs/rtrs-clt.c
drivers/infiniband/ulp/rtrs/rtrs-srv.c

index 5002ac6dfbc3ed3037a3c577749ac4bd852ffd9e..e201835de7332c629e9907b1b511082e5709414d 100644 (file)
@@ -183,7 +183,7 @@ u32 iwpm_check_registration(u8 nl_client, u32 reg);
 void iwpm_set_registration(u8 nl_client, u32 reg);
 
 /**
- * iwpm_get_registration
+ * iwpm_get_registration - Get the client registration
  * @nl_client: The index of the netlink client
  *
  * Returns the client registration type
index 7b638d91a4ec2aecc05ac9f384180f4d04df4753..68197e57643339bf7e394d92d675ebcfd7cba38c 100644 (file)
@@ -186,12 +186,13 @@ is_eth_port_inactive_slave_filter(struct ib_device *ib_dev, u32 port,
        return res;
 }
 
-/** is_ndev_for_default_gid_filter - Check if a given netdevice
+/**
+ * is_ndev_for_default_gid_filter - Check if a given netdevice
  * can be considered for default GIDs or not.
  * @ib_dev:            IB device to check
  * @port:              Port to consider for adding default GID
  * @rdma_ndev:         rdma netdevice pointer
- * @cookie_ndev:       Netdevice to consider to form a default GID
+ * @cookie:             Netdevice to consider to form a default GID
  *
  * is_ndev_for_default_gid_filter() returns true if a given netdevice can be
  * considered for deriving default RoCE GID, returns false otherwise.
index 5eeae8df415ba66b43d68b3547d5851564ada3d9..c9754463836741fc832fc0570c35d78ba5262971 100644 (file)
@@ -14186,7 +14186,7 @@ static void init_kdeth_qp(struct hfi1_devdata *dd)
 }
 
 /**
- * hfi1_get_qp_map
+ * hfi1_get_qp_map - get qp map
  * @dd: device data
  * @idx: index to read
  */
@@ -14199,7 +14199,7 @@ u8 hfi1_get_qp_map(struct hfi1_devdata *dd, u8 idx)
 }
 
 /**
- * init_qpmap_table
+ * init_qpmap_table - init qp map
  * @dd: device data
  * @first_ctxt: first context
  * @last_ctxt: first context
index 3b7bbc7b9d105f2b1d953b630522a47f37d213ae..955c3637980e8e37849092d9c7a2520a17a6da5b 100644 (file)
@@ -736,7 +736,7 @@ static u64 kvirt_to_phys(void *addr)
 }
 
 /**
- * complete_subctxt
+ * complete_subctxt - complete sub-context info
  * @fd: valid filedata pointer
  *
  * Sub-context info can only be set up after the base context
@@ -841,7 +841,7 @@ static int assign_ctxt(struct hfi1_filedata *fd, unsigned long arg, u32 len)
 }
 
 /**
- * match_ctxt
+ * match_ctxt - match context
  * @fd: valid filedata pointer
  * @uinfo: user info to compare base context with
  * @uctxt: context to compare uinfo to.
@@ -898,7 +898,7 @@ static int match_ctxt(struct hfi1_filedata *fd,
 }
 
 /**
- * find_sub_ctxt
+ * find_sub_ctxt - fund sub-context
  * @fd: valid filedata pointer
  * @uinfo: matching info to use to find a possible context to share.
  *
index 5632ef4d159c250afb2b08d152ae5f4ef7549903..31664f43c27fefdf14c0d88f43557c2978a39b61 100644 (file)
@@ -1760,7 +1760,7 @@ static inline void pause_for_credit_return(struct hfi1_devdata *dd)
 }
 
 /**
- * sc_to_vlt() reverse lookup sc to vl
+ * sc_to_vlt() reverse lookup sc to vl
  * @dd - devdata
  * @sc5 - 5 bit sc
  */
index e3a8a420c0455e0cebf221fe11eb9aefc20b1a9d..0986aa065418aff62b5f0a89fe2b12fa8bcf23ab 100644 (file)
@@ -312,7 +312,7 @@ struct hfi1_ctxtdata *hfi1_rcd_get_by_index_safe(struct hfi1_devdata *dd,
 }
 
 /**
- * hfi1_rcd_get_by_index
+ * hfi1_rcd_get_by_index - get by index
  * @dd: pointer to a valid devdata structure
  * @ctxt: the index of an possilbe rcd
  *
@@ -499,7 +499,7 @@ bail:
 }
 
 /**
- * hfi1_free_ctxt
+ * hfi1_free_ctxt - free context
  * @rcd: pointer to an initialized rcd data structure
  *
  * This wrapper is the free function that matches hfi1_create_ctxtdata().
index ff864f6f02667bf33e9b697c5d2c9d6298755d3d..e276522104c6a604574db9d6331446a00fddf057 100644 (file)
@@ -993,7 +993,7 @@ static bool is_sc_halted(struct hfi1_devdata *dd, u32 hw_context)
 }
 
 /**
- * sc_wait_for_packet_egress
+ * sc_wait_for_packet_egress - wait for packet
  * @sc: valid send context
  * @pause: wait for credit return
  *
index ae20915f94db50c6768ace8765e790f44e5bcb4f..fa15d79eabb36e72608d5159b80ec780d2c9c9c3 100644 (file)
@@ -1082,7 +1082,7 @@ static bool hem_list_is_bottom_bt(int hopnum, int bt_level)
        return bt_level >= (hopnum ? hopnum - 1 : hopnum);
 }
 
-/**
+/*
  * calc base address entries num
  * @hopnum: num of mutihop addressing
  * @bt_level: base address table level
@@ -1115,7 +1115,7 @@ static u32 hem_list_calc_ba_range(int hopnum, int bt_level, int unit)
        return step;
 }
 
-/**
+/*
  * calc the root ba entries which could cover all regions
  * @regions: buf region array
  * @region_cnt: array size of @regions
index 601d18dda1f5a7c1f8e7ef46f3c43b5b5b951d0e..34b7af6ab9c2c1cc12e43d933984e64bd054b2d6 100644 (file)
@@ -101,8 +101,8 @@ int rvt_driver_mr_init(struct rvt_dev_info *rdi)
 }
 
 /**
- *rvt_mr_exit: clean up MR
- *@rdi: rvt dev structure
+ * rvt_mr_exit - clean up MR
+ * @rdi: rvt dev structure
  *
  * called when drivers have unregistered or perhaps failed to register with us
  */
index 9d13db68283c2790f86a08a7c3341de99b7c6392..e9f3d356b3616c4df1ad268b6cea498971dca833 100644 (file)
@@ -984,7 +984,8 @@ static void rvt_reset_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp,
        spin_unlock_irq(&qp->r_lock);
 }
 
-/** rvt_free_qpn - Free a qpn from the bit map
+/**
+ * rvt_free_qpn - Free a qpn from the bit map
  * @qpt: QP table
  * @qpn: queue pair number to free
  */
index 1db89ee4c162333a8f47c35271e16d7a1970d9b3..ac17209816cdda3c188ddc94c489cab0c3e704ae 100644 (file)
@@ -144,7 +144,7 @@ static int rvt_modify_device(struct ib_device *device,
 }
 
 /**
- * rvt_query_port: Passes the query port call to the driver
+ * rvt_query_port - Passes the query port call to the driver
  * @ibdev: Verbs IB dev
  * @port_num: port number, 1 based from ib core
  * @props: structure to hold returned properties
@@ -175,7 +175,7 @@ static int rvt_query_port(struct ib_device *ibdev, u32 port_num,
 }
 
 /**
- * rvt_modify_port
+ * rvt_modify_port - modify port
  * @ibdev: Verbs IB dev
  * @port_num: Port number, 1 based from ib core
  * @port_modify_mask: How to change the port
index 7bd110ac4fe682cf3269297975ec91de90b5854b..abf60f4d920343463263a5b0e32054ad3471e691 100644 (file)
@@ -316,7 +316,7 @@ static bool ipoib_is_dev_match_addr_rcu(const struct sockaddr *addr,
        return false;
 }
 
-/**
+/*
  * Find the master net_device on top of the given net_device.
  * @dev: base IPoIB net_device
  *
@@ -361,8 +361,9 @@ static int ipoib_upper_walk(struct net_device *upper,
 }
 
 /**
- * Find a net_device matching the given address, which is an upper device of
- * the given net_device.
+ * ipoib_get_net_dev_match_addr - Find a net_device matching
+ * the given address, which is an upper device of the given net_device.
+ *
  * @addr: IP address to look for.
  * @dev: base IPoIB net_device
  *
index 136f6c4492e0d42890f8d0fe840e98c656de18fa..b44cbb8e84ebf74dd44cde68f2e19b680de005ac 100644 (file)
@@ -761,7 +761,7 @@ void iser_conn_init(struct iser_conn *iser_conn)
        ib_conn->reg_cqe.done = iser_reg_comp;
 }
 
- /**
+/*
  * starts the process of connecting to the target
  * sleeps until the connection is established or rejected
  */
index 160efef660319575d3a8fbf675ba813a3d8c8f4a..8634c83067da0e3946b0e97126e9c86776b462ff 100644 (file)
@@ -2397,10 +2397,10 @@ accept_wait:
                spin_unlock_bh(&np->np_thread_lock);
                isert_dbg("np_thread_state %d\n",
                         np->np_thread_state);
-               /**
+               /*
                 * No point in stalling here when np_thread
                 * is in state RESET/SHUTDOWN/EXIT - bail
-                **/
+                */
                return -ENODEV;
        }
        spin_unlock_bh(&np->np_thread_lock);
index 125e0bead262ce9f54e28bcd234f7e8bbd0cd9a7..cc530e09d778824547a6159da4a2c751b2d8f106 100644 (file)
@@ -915,7 +915,7 @@ static inline void path_it_deinit(struct path_it *it)
 }
 
 /**
- * rtrs_clt_init_req() Initialize an rtrs_clt_io_req holding information
+ * rtrs_clt_init_req() Initialize an rtrs_clt_io_req holding information
  * about an inflight IO.
  * The user buffer holding user control message (not data) is copied into
  * the corresponding buffer of rtrs_iu (req->iu->buf), which later on will
@@ -1221,7 +1221,7 @@ static int rtrs_clt_read_req(struct rtrs_clt_io_req *req)
 }
 
 /**
- * rtrs_clt_failover_req() Try to find an active path for a failed request
+ * rtrs_clt_failover_req() Try to find an active path for a failed request
  * @clt: clt context
  * @fail_req: a failed io request.
  */
index 1a30fd8337928f88eb3f339479b013cef5e34843..3df2900861697af1c2bccc24142a5d9b92a83e5a 100644 (file)
@@ -1308,7 +1308,7 @@ int rtrs_srv_get_sess_name(struct rtrs_srv *srv, char *sessname, size_t len)
 EXPORT_SYMBOL(rtrs_srv_get_sess_name);
 
 /**
- * rtrs_srv_get_sess_qdepth() - Get rtrs_srv qdepth.
+ * rtrs_srv_get_queue_depth() - Get rtrs_srv qdepth.
  * @srv:       Session
  */
 int rtrs_srv_get_queue_depth(struct rtrs_srv *srv)