On 5/24/07, <b class="gmail_sendername">Sean Hefty</b> <<a href="mailto:sean.hefty@intel.com">sean.hefty@intel.com</a>> wrote:<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
>The ib_sa/rdmacm multicast code in ofed 1.2 comes in the form of one big<br>>  undocumented patch (merged_sean_rdma_dev_ofed_1_2.patch), is the below<br>>fine as a change log comment?<br><br>Yes - this should work for the change log comment.
</blockquote><div><br>
This for itself would not work for the change log since as i wrote you,
the giant patch also contain the local sa code. If its fine with you,
best if you can send Vlad a giant change log (sa multicast support,
rdma-cm mcast support, local sa) to be placed at the top of the patch.<br>
<br>
I hope the motivation here is clear: undocumented patches makes ofed
kind of impossible to be deployed in a multi tier support model (in
other words, to understsand/change a bit, support people have to run to
the code maintainer...)<br>
<br>
thanks,<br>
<br>
Or.<br>
 </div><br></div><br>