Commit b1bc15f8 authored by Bart Van Assche's avatar Bart Van Assche Committed by Leon Romanovsky

RDMA/iwcm: Use list_first_entry() where appropriate

Improve source code readability by using list_first_entry() where appropriate.
Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Link: https://lore.kernel.org/r/20240605145117.397751-2-bvanassche@acm.orgReviewed-by: default avatarZhu Yanjun <yanjun.zhu@linux.dev>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent c8683b99
...@@ -143,7 +143,7 @@ static struct iwcm_work *get_work(struct iwcm_id_private *cm_id_priv) ...@@ -143,7 +143,7 @@ static struct iwcm_work *get_work(struct iwcm_id_private *cm_id_priv)
if (list_empty(&cm_id_priv->work_free_list)) if (list_empty(&cm_id_priv->work_free_list))
return NULL; return NULL;
work = list_entry(cm_id_priv->work_free_list.next, struct iwcm_work, work = list_first_entry(&cm_id_priv->work_free_list, struct iwcm_work,
free_list); free_list);
list_del_init(&work->free_list); list_del_init(&work->free_list);
return work; return work;
...@@ -1023,7 +1023,7 @@ static void cm_work_handler(struct work_struct *_work) ...@@ -1023,7 +1023,7 @@ static void cm_work_handler(struct work_struct *_work)
spin_lock_irqsave(&cm_id_priv->lock, flags); spin_lock_irqsave(&cm_id_priv->lock, flags);
empty = list_empty(&cm_id_priv->work_list); empty = list_empty(&cm_id_priv->work_list);
while (!empty) { while (!empty) {
work = list_entry(cm_id_priv->work_list.next, work = list_first_entry(&cm_id_priv->work_list,
struct iwcm_work, list); struct iwcm_work, list);
list_del_init(&work->list); list_del_init(&work->list);
empty = list_empty(&cm_id_priv->work_list); empty = list_empty(&cm_id_priv->work_list);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment