Browse Source

[PATCH] IB: Simplify calling of list_del in MAD

Simplify calling of list_del.

Signed-off-by: Sean Hefty <sean.hefty@intel.com>
Signed-off-by: Hal Rosenstock <halr@voltaire.com>
Cc: Roland Dreier <rolandd@cisco.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Hal Rosenstock 20 years ago
parent
commit
dbf9227bd3
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/infiniband/core/mad.c

+ 1 - 2
drivers/infiniband/core/mad.c

@@ -2188,7 +2188,6 @@ static int retry_send(struct ib_mad_send_wr_private *mad_send_wr)
 
 
 	if (!ret) {
 	if (!ret) {
 		mad_send_wr->refcount++;
 		mad_send_wr->refcount++;
-		list_del(&mad_send_wr->agent_list);
 		list_add_tail(&mad_send_wr->agent_list,
 		list_add_tail(&mad_send_wr->agent_list,
 			      &mad_send_wr->mad_agent_priv->send_list);
 			      &mad_send_wr->mad_agent_priv->send_list);
 	}
 	}
@@ -2223,10 +2222,10 @@ static void timeout_sends(void *data)
 			break;
 			break;
 		}
 		}
 
 
+		list_del(&mad_send_wr->agent_list);
 		if (!retry_send(mad_send_wr))
 		if (!retry_send(mad_send_wr))
 			continue;
 			continue;
 
 
-		list_del(&mad_send_wr->agent_list);
 		spin_unlock_irqrestore(&mad_agent_priv->lock, flags);
 		spin_unlock_irqrestore(&mad_agent_priv->lock, flags);
 
 
 		mad_send_wc.wr_id = mad_send_wr->wr_id;
 		mad_send_wc.wr_id = mad_send_wr->wr_id;