Oops... I see you asked that for mthca. Well, I might do it at a later time.<br><br>
<div><span class="gmail_quote">On 4/26/07, <b class="gmail_sendername">Eli Cohen</b> <<a href="mailto:eli@dev.mellanox.co.il">eli@dev.mellanox.co.il</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid"><br><br>
<div><span class="q"><span class="gmail_quote">On 4/25/07, <b class="gmail_sendername">Roland Dreier</b> <<a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:rdreier@cisco.com" target="_blank">rdreier@cisco.com
</a>> wrote:</span> 
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">> > Also is it worth taking the semaphore in the mthca reset function?<br>><br>> Why not? PRM says we should - the point of this is protecting against someone 
<br>> accidentally touching flash while device is reset,<br>> which will result is hangs, system errors or even data corruption.<br><br>OK... that was just my subtle way of asking for a patch to do that.</blockquote>

<div> </div></span>
<div>I don't follow you: the code that uses the semaphore is already in the patch I sent. Apart from the the change in the vendor id field, I think it can be applied as is.</div><span class="q"><br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">_______________________________________________<br>general mailing list<br><a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:general@lists.openfabrics.org" target="_blank">
general@lists.openfabrics.org</a><br><a onclick="return top.js.OpenExtLink(window,event,this)" href="http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general" target="_blank">http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
</a><br><br>To unsubscribe, please visit <a onclick="return top.js.OpenExtLink(window,event,this)" href="http://openib.org/mailman/listinfo/openib-general" target="_blank">http://openib.org/mailman/listinfo/openib-general
</a><br></blockquote></span></div><br></blockquote></div><br>