[Openib-windows] RE: UD AV in UL return the wrong

[Yossi Leybovich]

Fab

Few comments:
1. I think that you miss to include ..\..\user in the sources of
mt23108\user 

Index: D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES
===================================================================
--- D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES	(revision 253)
+++ D:/sleybo/MSI/hw/mt23108/vapi/user/SOURCES	(working copy)
@@ -52,6 +52,7 @@
 	$(THH_HOME)\ddrmm; \
 	$(THH_HOME)\cmdif; \
 	$(MT_HOME)\mlxsys\os_dep\win\tdriver; \
+	..\..\user; \
 	..\..\..\..\inc; \
 	..\..\..\..\inc\user; \
 	..\..\..\..\core\al; \


2. inc/user/iba/ib_uvp.h you forgot to change the DESCRIPTION of the
function pre_query_ca in the header file.
It will be great if you add it

Except from these comments it looks like the patch work at least for our
simple test
So I think you can commit it

10x
Yossi 

-----Original Message-----
From: Fab Tillier [mailto:ftillier at silverstorm.com] 
Sent: Thursday, August 04, 2005 11:49 PM
To: 'Yossi Leybovich'
Cc: openib-windows at openib.org
Subject: RE: UD AV in UL return the wrong 


> From: Yossi Leybovich [mailto:sleybo at mellanox.co.il]
> Sent: Thursday, August 04, 2005 7:31 AM
> 
> We tried to run simple UL test over UD QP .
>
> It looks like the av handle use the wrong value ( we got completion 
> with error (local operational error)
...
> Either mlnx_post_create_av return the wrong structure to ib_creat_av 
> or the ib_post_send use the wrong h_av structure.

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.

I moved the hca_data.h file from hw\mt23108\vapi\user to hw\mt23108\user.

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.

- Fab
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20050807/fac5dea8/attachment.html>