<!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: UD AV in UL return the wrong </TITLE>
</HEAD>
<BODY>
<P><FONT SIZE=2>Fab</FONT>
</P>
<P><FONT SIZE=2>Few comments:</FONT>
<BR><FONT SIZE=2>1. I think that you miss to include ..\..\user in the sources of mt23108\user </FONT>
</P>
<P><FONT SIZE=2>Index: D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES</FONT>
<BR><FONT SIZE=2>===================================================================</FONT>
<BR><FONT SIZE=2>--- D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES (revision 253)</FONT>
<BR><FONT SIZE=2>+++ D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES (working copy)</FONT>
<BR><FONT SIZE=2>@@ -52,6 +52,7 @@</FONT>
<BR><FONT SIZE=2> $(THH_HOME)\ddrmm; \</FONT>
<BR><FONT SIZE=2> $(THH_HOME)\cmdif; \</FONT>
<BR><FONT SIZE=2> $(MT_HOME)\mlxsys\os_dep\win\tdriver; \</FONT>
<BR><FONT SIZE=2>+ ..\..\user; \</FONT>
<BR><FONT SIZE=2> ..\..\..\..\inc; \</FONT>
<BR><FONT SIZE=2> ..\..\..\..\inc\user; \</FONT>
<BR><FONT SIZE=2> ..\..\..\..\core\al; \</FONT>
</P>
<BR>
<P><FONT SIZE=2>2. inc/user/iba/ib_uvp.h you forgot to change the DESCRIPTION of the function pre_query_ca in the header file.</FONT>
<BR><FONT SIZE=2>It will be great if you add it</FONT>
</P>
<P><FONT SIZE=2>Except from these comments it looks like the patch work at least for our simple test</FONT>
<BR><FONT SIZE=2>So I think you can commit it</FONT>
</P>
<P><FONT SIZE=2>10x</FONT>
<BR><FONT SIZE=2>Yossi </FONT>
</P>
<P><FONT SIZE=2>-----Original Message-----</FONT>
<BR><FONT SIZE=2>From: Fab Tillier [<A HREF="mailto:ftillier@silverstorm.com">mailto:ftillier@silverstorm.com</A>] </FONT>
<BR><FONT SIZE=2>Sent: Thursday, August 04, 2005 11:49 PM</FONT>
<BR><FONT SIZE=2>To: 'Yossi Leybovich'</FONT>
<BR><FONT SIZE=2>Cc: openib-windows@openib.org</FONT>
<BR><FONT SIZE=2>Subject: RE: UD AV in UL return the wrong </FONT>
</P>
<BR>
<P><FONT SIZE=2>> From: Yossi Leybovich [<A HREF="mailto:sleybo@mellanox.co.il">mailto:sleybo@mellanox.co.il</A>]</FONT>
<BR><FONT SIZE=2>> Sent: Thursday, August 04, 2005 7:31 AM</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> We tried to run simple UL test over UD QP .</FONT>
<BR><FONT SIZE=2>></FONT>
<BR><FONT SIZE=2>> It looks like the av handle use the wrong value ( we got completion </FONT>
<BR><FONT SIZE=2>> with error (local operational error)</FONT>
<BR><FONT SIZE=2>...</FONT>
<BR><FONT SIZE=2>> Either mlnx_post_create_av return the wrong structure to ib_creat_av </FONT>
<BR><FONT SIZE=2>> or the ib_post_send use the wrong h_av structure.</FONT>
</P>
<P><FONT SIZE=2>The VAPI code includes hca_data.h, which defines a structure for what the AV looks like. The problem is this structure was created to get things to build, and doesn't match the actual usage.</FONT></P>
<P><FONT SIZE=2>I moved the hca_data.h file from hw\mt23108\vapi\user to hw\mt23108\user.</FONT>
</P>
<P><FONT SIZE=2>Do that and then apply this patch and see if that solves your problem. If it does, let me know and I'll commit this change.</FONT></P>
<P><FONT SIZE=2>- Fab</FONT>
</P>
</BODY>
</HTML>