>----------------------------------------------------
>if (qp_info->send_queue.count < qp_info->send_queue.max_active) {

This loses the locking around send_queue.count, which can allow trying to post
more sends than the QP supports.

>                ret = ib_post_send(mad_agent->qp, &mad_send_wr->send_wr,
>                                   &bad_send_wr);
>                spin_lock_irqsave(&qp_info->send_queue.lock, flags);
>                list = &qp_info->send_queue.list;
>} else {
>                ret = 0;
>                spin_lock_irqsave(&qp_info->send_queue.lock, flags);
>                list = &qp_info->overflow_list;
>}

I think the code will end up quite a bit more complex not locking around
ib_post_send, but maybe there's a clever way to handle this.

- Sean

_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to