RDMA/iwcm: Use list_first_entry() where appropriate

Improve source code readability by using list_first_entry() where appropriate.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Link: https://lore.kernel.org/r/20240605145117.397751-2-bvanassche@acm.org
Reviewed-by: Zhu Yanjun <yanjun.zhu@linux.dev>
Signed-off-by: Leon Romanovsky <leon@kernel.org>

diff --git a/drivers/infiniband/core/iwcm.c b/drivers/infiniband/core/iwcm.c
index 0301fcad4b48bebe3378adc9e6dbd6184f4ce07f..90d8f3d66990d23ef8df35f4a724af74d66195c5 100644 (file)
--- a/drivers/infiniband/core/iwcm.c
+++ b/drivers/infiniband/core/iwcm.c
@@ -143,8 +143,8 @@ static struct iwcm_work *get_work(struct iwcm_id_private *cm_id_priv)
 
        if (list_empty(&cm_id_priv->work_free_list))
                return NULL;
-       work = list_entry(cm_id_priv->work_free_list.next, struct iwcm_work,
-                         free_list);
+       work = list_first_entry(&cm_id_priv->work_free_list, struct iwcm_work,
+                               free_list);
        list_del_init(&work->free_list);
        return work;
 }
@@ -1023,8 +1023,8 @@ static void cm_work_handler(struct work_struct *_work)
        spin_lock_irqsave(&cm_id_priv->lock, flags);
        empty = list_empty(&cm_id_priv->work_list);
        while (!empty) {
-               work = list_entry(cm_id_priv->work_list.next,
-                                 struct iwcm_work, list);
+               work = list_first_entry(&cm_id_priv->work_list,
+                                       struct iwcm_work, list);
                list_del_init(&work->list);
                empty = list_empty(&cm_id_priv->work_list);
                levent = work->event;