<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=iso-8859-1">
<META content="MSHTML 6.00.2800.1400" name=GENERATOR>
<STYLE></STYLE>
</HEAD>
<BODY bgColor=#ffffff>
<DIV><FONT face=Arial size=2>Hi Sean,</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>A couple of minor questions about this
patch:</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>1. In ib_mad.h:</FONT></DIV>
<DIV><!--StartFragment --> /**<BR>+ * ib_mad_snoop_handler - Callback
handler for snooping sent MADs.<BR>+ * @mad_agent: MAD agent that snooped the
MAD.<BR>+ * @send_wr: Work request information on the sent MAD.<BR>+ *
@mad_send_wc: Work completion information on the sent MAD. Valid<BR>+
* only for snooping that occurs on a send completion.<BR>+ *<BR>+ *
Clients snooping MADs should not modify data referenced by the @send_wr<BR>+ *
or @mad_send_wc.<BR>+ */</DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>I presume snoop clients should also not free the
MAD either. If so, should that comment</FONT></DIV>
<DIV><FONT face=Arial size=2>also be added ?</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>2. Should MAD snooping be exposed to user space too
?</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>Thanks.</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>- Hal</FONT></DIV></BODY></HTML>