<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=US-ASCII">
<META NAME="Generator" CONTENT="MS Exchange Server version 5.5.2654.45">
<TITLE>RE: [PATCH ] osmtest general cleanups</TITLE>
</HEAD>
<BODY>
<P><FONT SIZE=2>Hi , Hal.</FONT>
<BR><FONT SIZE=2>The spec I'm referring to is IB Spec Release 1.2 (Volume 1) , p.754 in the 13.5.1 section , look at the third paragraph and find the following quote : </FONT></P>
<P><FONT SIZE=2>" Note that it is not required by IBA that GS managers use QP1 as the</FONT>
<BR><FONT SIZE=2>source QP used to send management packets to GS agents. GS managers</FONT>
<BR><FONT SIZE=2>may send packets from any QP other than QP0."</FONT>
</P>
<P><FONT SIZE=2>The InformInfo test flow sends a Trap (using QP0) , this part can be implemented by umad . In response the SM should send a Report ( to a previously subscribed node).</FONT></P>
<P><FONT SIZE=2>-----Original Message-----</FONT>
<BR><FONT SIZE=2>From: Hal Rosenstock [<A HREF="mailto:halr@voltaire.com">mailto:halr@voltaire.com</A>]</FONT>
<BR><FONT SIZE=2>Sent: Tuesday, August 09, 2005 3:53 AM</FONT>
<BR><FONT SIZE=2>To: Liran Sorani</FONT>
<BR><FONT SIZE=2>Cc: openib-general@openib.org; Amit Krig</FONT>
<BR><FONT SIZE=2>Subject: RE: [PATCH ] osmtest general cleanups</FONT>
</P>
<BR>
<P><FONT SIZE=2>Hi Liran,</FONT>
</P>
<P><FONT SIZE=2>On Mon, 2005-08-08 at 09:58, Liran Sorani wrote: </FONT>
<BR><FONT SIZE=2>> Hi ,</FONT>
<BR><FONT SIZE=2>> Regarding the inform_info test flow , the reason I need (possibly)</FONT>
<BR><FONT SIZE=2>> ib_verbs is to subscribe a notice report through a QP other then QP1</FONT>
<BR><FONT SIZE=2>> (permitted by IB Spec see p754 , 13.5.1) ,</FONT>
</P>
<P><FONT SIZE=2>What version of the spec are you referring to ? Can you make the</FONT>
<BR><FONT SIZE=2>reference more specific ? Is it 13.5.1.2?</FONT>
</P>
<P><FONT SIZE=2>> generate a trap , then validate the received report through that QP.</FONT>
<BR><FONT SIZE=2>> To enable Set/Recieve notice mads through QP other then QP1 , I think</FONT>
<BR><FONT SIZE=2>> , I need the ib_verbs</FONT>
</P>
<P><FONT SIZE=2>While the subscription (InformInfo) could be through a user SA client,</FONT>
<BR><FONT SIZE=2>the Trap/Notice could be direct to some QP other than QP1 and hence</FONT>
<BR><FONT SIZE=2>would require uverbs for at least this part.</FONT>
</P>
<P><FONT SIZE=2>Do these tests cause SM generated events (ports in and out of service,</FONT>
<BR><FONT SIZE=2>multicast groups, etc.) ?</FONT>
</P>
<P><FONT SIZE=2>-- Hal</FONT>
</P>
<P><FONT SIZE=2>> Regarding the rest , will be fixed in a seperate patch .</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> --Original Message--</FONT>
<BR><FONT SIZE=2>> From: Hal Rosenstock [<A HREF="mailto:halr@voltaire.com">mailto:halr@voltaire.com</A>]</FONT>
<BR><FONT SIZE=2>> Sent: Monday, August 08, 2005 3:32 PM</FONT>
<BR><FONT SIZE=2>> To: Liran Sorani</FONT>
<BR><FONT SIZE=2>> Cc: openib-general@openib.org; Amit Krig</FONT>
<BR><FONT SIZE=2>> Subject: RE: [PATCH ] osmtest general cleanups</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Hi Liran,</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> On Sun, 2005-08-07 at 11:28, Liran Sorani wrote:</FONT>
<BR><FONT SIZE=2>> > Hi , Hal.</FONT>
<BR><FONT SIZE=2>> > I've a few minor fixes (several white space glitches) , pls take the</FONT>
<BR><FONT SIZE=2>> > attached file , instead of the previous ,</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Thanks. Applied with some minor modifications and caveats below.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> In the future can your patches be submitted as text rather than</FONT>
<BR><FONT SIZE=2>> attachments ? That is the norm for doing this.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Some comments below.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > thanks Liran.</FONT>
<BR><FONT SIZE=2>> > </FONT>
<BR><FONT SIZE=2>> > --Original Message--</FONT>
<BR><FONT SIZE=2>> > From: Liran Sorani </FONT>
<BR><FONT SIZE=2>> > Sent: Sunday, August 07, 2005 5:57 PM</FONT>
<BR><FONT SIZE=2>> > To: 'halr@voltaire.com'</FONT>
<BR><FONT SIZE=2>> > Cc: 'openib-general@openib.org'; Amit Krig</FONT>
<BR><FONT SIZE=2>> > Subject: [PATCH ] osmtest general cleanups</FONT>
<BR><FONT SIZE=2>> > </FONT>
<BR><FONT SIZE=2>> > </FONT>
<BR><FONT SIZE=2>> > Hi , Hal.</FONT>
<BR><FONT SIZE=2>> > The attached patch should be applied to osmtest repository.</FONT>
<BR><FONT SIZE=2>> > It contain several cleanups (on most of the files) :</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> It is easier if there is a patch per idea rather than an amalgam.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > -Removal of inform info flow .</FONT>
<BR><FONT SIZE=2>> > -Unique error messages</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> There are some non real error message numbers (neither hex nor</FONT>
<BR><FONT SIZE=2>> decimal)</FONT>
<BR><FONT SIZE=2>> in osmt_multicast.c. Also, osmtest.c and osmt_service.c still have</FONT>
<BR><FONT SIZE=2>> duplicates.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > -Makefile.am update for compilation (required osm_helper object)</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> osm_helper is part of the libopensm since r2973 so this part of the</FONT>
<BR><FONT SIZE=2>> patch was not applied. Please update your osm directory.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > -Remove vendor dependencies.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Remove some OSM_VENDOR_INTF_MTL vendor dependencies</FONT>
<BR><FONT SIZE=2>> (There are still some in osmt_slvl_vl_arb.c).</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > The inform info flow should be carefully ported since it requires</FONT>
<BR><FONT SIZE=2>> > direct access to ib_umad (possibly ib_verbs too) .</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> umad would be a temporary measure. </FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Why would ib_uverbs direct access be needed ?</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> > I'll send another patch for it next week .</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Thanks.</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> -- Hal</FONT>
<BR><FONT SIZE=2>> </FONT>
</P>
</BODY>
</HTML>