Skip to content

Commit b1bc15f

Browse files
bvanasscherleon
authored andcommitted
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 <[email protected]> Link: https://lore.kernel.org/r/[email protected] Reviewed-by: Zhu Yanjun <[email protected]> Signed-off-by: Leon Romanovsky <[email protected]>
1 parent c8683b9 commit b1bc15f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

drivers/infiniband/core/iwcm.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -143,8 +143,8 @@ static struct iwcm_work *get_work(struct iwcm_id_private *cm_id_priv)
143143

144144
if (list_empty(&cm_id_priv->work_free_list))
145145
return NULL;
146-
work = list_entry(cm_id_priv->work_free_list.next, struct iwcm_work,
147-
free_list);
146+
work = list_first_entry(&cm_id_priv->work_free_list, struct iwcm_work,
147+
free_list);
148148
list_del_init(&work->free_list);
149149
return work;
150150
}
@@ -1023,8 +1023,8 @@ static void cm_work_handler(struct work_struct *_work)
10231023
spin_lock_irqsave(&cm_id_priv->lock, flags);
10241024
empty = list_empty(&cm_id_priv->work_list);
10251025
while (!empty) {
1026-
work = list_entry(cm_id_priv->work_list.next,
1027-
struct iwcm_work, list);
1026+
work = list_first_entry(&cm_id_priv->work_list,
1027+
struct iwcm_work, list);
10281028
list_del_init(&work->list);
10291029
empty = list_empty(&cm_id_priv->work_list);
10301030
levent = work->event;

0 commit comments

Comments
 (0)