[ofw] changes to ib_wc committed to svn

[Tzachi Dar]

Checksum offload will only work UD QPC. In other words, for connected
mode ipoib it will not work.

I don't mind putting the 8 bits in the recv_opt field.
Currently recv_opt is defined as ib_recv_opt_t which is uint_32.

There is also a list of macros that is defined as 
#define IB_RECV_OPT_IMMEDIATE		0x00000001
#define IB_RECV_OPT_FORWARD		0x00000002
#define IB_RECV_OPT_GRH_VALID		0x00000004
#define IB_RECV_OPT_VEND_MASK		0xFFFF0000

So I suggest to make one more mask:
#define IB_RECV_OPT_CHECK_SUM		0x0000FF00
And we will be able to use this extra place.

Thanks
Tzachi


> -----Original Message-----
> From: Fab Tillier [mailto:ftillier at windows.microsoft.com] 
> Sent: Tuesday, July 29, 2008 7:12 PM
> To: Tzachi Dar; Sean Hefty; ofw at lists.openfabrics.org
> Subject: RE: [ofw] changes to ib_wc committed to svn
> 
> Hi Tzachi
> 
> >Just to summarize:
> >The structure ib_ca_attr_t alread contains the field ipoib_csum 
> >(according to your request).
> >The structure ib_wc_t will look as:
> 
> The structure looks good I think.  Does checksum offload also 
> apply to RC QPs or only UD?  What about using 8 bits in the 
> recv_opt field rather than adding a new field, would that 
> make any more sense?  It would seem that this would minimize 
> changes needed in existing HCA drivers that don't support 
> checksum offload as the extra bits would be zero in the 
> recv_opt field.
> 
> -Fab
> 
> 
>