[ofa-general] [PATCH RFC] RDMA/cxgb3: MEM_MGT_EXTENSIONS support

Steve Wise swise at opengridcomputing.com
Fri Jun 20 08:18:21 PDT 2008


Roland Dreier wrote:
> Thanks, applied, with one minor fix:
>
>  > +static int fw_supports_fastreg(struct iwch_dev *iwch_dev)
>
> ...
>
>  > +	return (fw_maj >= 6 && fw_min > 0);
>
> You really mean
>
> 	return fw_maj > 6 || (fw_maj == 6 && fw_min > 0);
>
> right?
>
>   
yes.

> and also I changed the 'p' pointer to __be64 in iwch_build_fastreg(),
> but that still leaves
>
>     drivers/infiniband/hw/cxgb3/iwch_qp.c:174:6: warning: incorrect type in assignment (different base types)
>     drivers/infiniband/hw/cxgb3/iwch_qp.c:174:6: expected restricted unsigned long long [usertype] *[assigned] p
>     drivers/infiniband/hw/cxgb3/iwch_qp.c:174:6: got unsigned long long *
>
> for the line
>
> 			p = &wqe->flit[1];
>
> because flit[] is u64.  What's the best way to fix this?
>
>   

I think flit[] should be __be64.





More information about the general mailing list