[openib-general] Re: [PATCH] kDAPL: remove dapl_os_assert()

Tom Duffy tduffy at sun.com
Thu Jun 23 14:31:06 PDT 2005


On Thu, 2005-06-23 at 16:55 -0400, James Lentini wrote:
> Why did you remove the asserts in dapl_ib_post_recv() and 
> dapl_ib_post_send()? 

The code continues 6 lines down:

< assert if cookie is null here >

        sg_list = ep->recv_iov;
        for (i = 0; i < num_segments; i++, sg_list++) {
                sg_list->addr = local_iov[i].virtual_address;
                sg_list->length = (u32) local_iov[i].segment_length;
                sg_list->lkey = local_iov[i].lmr_context;
                total_len += sg_list->length;
        }
        cookie->val.dto.size = total_len;
	^^^^^^^^
		would crash here anyways.

-tduffy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openfabrics.org/pipermail/general/attachments/20050623/65c8aae3/attachment.sig>


More information about the general mailing list