[PATCH] IB: Optimize canceling a MAD
Optimize canceling a MAD. - Eliminate searching timeout list in cancel case. - Remove duplicate calls to queue work item. - Eliminate resending a MAD before MAD is completed. 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>
This commit is contained in:
parent
03b61ad2f2
commit
29bb33dd87
|
@ -1754,6 +1754,7 @@ static void wait_for_response(struct ib_mad_send_wr_private *mad_send_wr)
|
||||||
delay = mad_send_wr->timeout;
|
delay = mad_send_wr->timeout;
|
||||||
mad_send_wr->timeout += jiffies;
|
mad_send_wr->timeout += jiffies;
|
||||||
|
|
||||||
|
if (delay) {
|
||||||
list_for_each_prev(list_item, &mad_agent_priv->wait_list) {
|
list_for_each_prev(list_item, &mad_agent_priv->wait_list) {
|
||||||
temp_mad_send_wr = list_entry(list_item,
|
temp_mad_send_wr = list_entry(list_item,
|
||||||
struct ib_mad_send_wr_private,
|
struct ib_mad_send_wr_private,
|
||||||
|
@ -1762,6 +1763,9 @@ static void wait_for_response(struct ib_mad_send_wr_private *mad_send_wr)
|
||||||
temp_mad_send_wr->timeout))
|
temp_mad_send_wr->timeout))
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
else
|
||||||
|
list_item = &mad_agent_priv->wait_list;
|
||||||
list_add(&mad_send_wr->agent_list, list_item);
|
list_add(&mad_send_wr->agent_list, list_item);
|
||||||
|
|
||||||
/* Reschedule a work item if we have a shorter timeout */
|
/* Reschedule a work item if we have a shorter timeout */
|
||||||
|
@ -2197,7 +2201,8 @@ static void timeout_sends(void *data)
|
||||||
}
|
}
|
||||||
|
|
||||||
list_del(&mad_send_wr->agent_list);
|
list_del(&mad_send_wr->agent_list);
|
||||||
if (!retry_send(mad_send_wr))
|
if (mad_send_wr->status == IB_WC_SUCCESS &&
|
||||||
|
!retry_send(mad_send_wr))
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
spin_unlock_irqrestore(&mad_agent_priv->lock, flags);
|
spin_unlock_irqrestore(&mad_agent_priv->lock, flags);
|
||||||
|
|
Loading…
Reference in New Issue