<!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: [openib-general] [PATCH] Osmtest - Byte Swapping issue and resource cleanup</TITLE>
</HEAD>
<BODY>

<P><FONT SIZE=2>Committed to OpenSM - gen1.</FONT>
<BR><FONT SIZE=2>Thanks,</FONT>
<BR><FONT SIZE=2>Yael</FONT>
</P>
<BR>

<P><FONT SIZE=2>-----Original Message-----</FONT>
<BR><FONT SIZE=2>From: Jan Daley [<A HREF="mailto:jdaley@systemfabricworks.com">mailto:jdaley@systemfabricworks.com</A>]</FONT>
<BR><FONT SIZE=2>Sent: Saturday, August 07, 2004 12:10 AM</FONT>
<BR><FONT SIZE=2>To: openib-general@openib.org</FONT>
<BR><FONT SIZE=2>Subject: [openib-general] [PATCH] Osmtest - Byte Swapping issue and</FONT>
<BR><FONT SIZE=2>resource cleanup</FONT>
</P>
<BR>

<P><FONT SIZE=2>Hi,</FONT>
</P>

<P><FONT SIZE=2>1) In osmtest_stress_port_recs_small, the lid passed into the call to</FONT>
<BR><FONT SIZE=2>osmtest_get_port_rec wasn't being byte swapped.  This caused all</FONT>
<BR><FONT SIZE=2>portinfo records to be retrieved instead of just one.</FONT>
</P>

<P><FONT SIZE=2>2) I added a couple of calls to cleanup qpools that were created.</FONT>
</P>
<BR>
<BR>
<BR>
<BR>

<P><FONT SIZE=2>Index: osmtest/osmtest.c</FONT>
<BR><FONT SIZE=2>===================================================================</FONT>
<BR><FONT SIZE=2>--- osmtest/osmtest.c   (revision 590)</FONT>
<BR><FONT SIZE=2>+++ osmtest/osmtest.c   (working copy)</FONT>
<BR><FONT SIZE=2>@@ -479,6 +479,8 @@</FONT>
<BR><FONT SIZE=2>       osm_vendor_delete( &p_osmt->p_vendor );</FONT>
<BR><FONT SIZE=2>     }</FONT>
<BR><FONT SIZE=2> </FONT>
<BR><FONT SIZE=2>+  cl_qpool_destroy(&p_osmt->port_pool);</FONT>
<BR><FONT SIZE=2>+  cl_qpool_destroy(&p_osmt->node_pool);</FONT>
<BR><FONT SIZE=2>   osm_log_destroy( &p_osmt->log );</FONT>
<BR><FONT SIZE=2> }</FONT>
<BR><FONT SIZE=2> </FONT>
<BR><FONT SIZE=2>@@ -1308,7 +1310,7 @@</FONT>
<BR><FONT SIZE=2>   /*</FONT>
<BR><FONT SIZE=2>    * Do a blocking query for our own PortRecord in the subnet.</FONT>
<BR><FONT SIZE=2>    */</FONT>
<BR><FONT SIZE=2>-  status = osmtest_get_port_rec( p_osmt, p_osmt->local_port.lid,</FONT>
<BR><FONT SIZE=2>&context );</FONT>
<BR><FONT SIZE=2>+  status = osmtest_get_port_rec( p_osmt,</FONT>
<BR><FONT SIZE=2>cl_hton16(p_osmt->local_port.lid), &context );</FONT>
<BR><FONT SIZE=2> </FONT>
<BR><FONT SIZE=2>   if( status != IB_SUCCESS )</FONT>
<BR><FONT SIZE=2>     {</FONT>
</P>
<BR>
<BR>

<P><FONT SIZE=2>Jan Daley</FONT>
<BR><FONT SIZE=2>System Fabric Works</FONT>
<BR><FONT SIZE=2>(512) 343-6101 x 13</FONT>
</P>
<BR>
<BR>
<BR>

<P><FONT SIZE=2>_______________________________________________</FONT>
<BR><FONT SIZE=2>openib-general mailing list</FONT>
<BR><FONT SIZE=2>openib-general@openib.org</FONT>
<BR><FONT SIZE=2><A HREF="http://openib.org/mailman/listinfo/openib-general" TARGET="_blank">http://openib.org/mailman/listinfo/openib-general</A></FONT>
</P>

<P><FONT SIZE=2>To unsubscribe, please visit <A HREF="http://openib.org/mailman/listinfo/openib-general" TARGET="_blank">http://openib.org/mailman/listinfo/openib-general</A></FONT>
</P>

</BODY>
</HTML>