> --- a/drivers/infiniband/hw/cxgb3/iwch_provider.c
 > +    dev->ibdev.local_dma_lkey = 0;

why this specific initialization but not in nes or amso1100?  I think
the device structure is zeroed out on allocation anyway.

 >      __be64                       node_guid;
 >      u8                           node_type;
 >      u8                           phys_port_cnt;
 > +    u32                          local_dma_lkey;

It's a little silly of me to mention this, given how few of these
structures are used, but I think it would be nicer to put local_dma_lkey
after node_guid to avoid leaving padding in the structure.

 - R.
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to