Hi.

Sean Hefty wrote:
In cancel_mads(), MADs are moved from the wait_list and local_list
to a cancel_list for processing.  However, the structures on these two
lists are not the same.  The wait_list references struct
ib_mad_send_wr_private, but local_list references struct
ib_mad_local_private.  Cancel_mads() treats all items moved to the
cancel_list as struct ib_mad_send_wr_private.  This leads to a system
crash when requests are moved from the local_list to the cancel_list.

Fix this by leaving local_list alone.  All requests on the local_list
have completed are just awaiting processing by a queued worker thread.

Bug (crash) reported by Dotan Barak <[EMAIL PROTECTED]>.
Problem with local_list access reported by Robert Reynolds
<[EMAIL PROTECTED]>.

Signed-off-by: Sean Hefty <[EMAIL PROTECTED]>
---
This patch is untested.  Dotan, can you see if this fixes the crash that
you were seeing?
Just want to let me know that i didn't forget about this issue.

I tried to reproduce the failure before applying the bug, but this one is not easy to reproduce.

I will give you a feedback as soon as I'll have one ..


Dotan
_______________________________________________
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