> > Hi Steve, > > I reviewed my reports from various static checkers and saw strange code > in build_tpte_memreg() function that in drivers/infiniband/hw/cxgb4/qp.c. > > That function has following code: > 614 fr->r2 = cpu_to_be32(0); > 615 fr->stag = cpu_to_be32(mhp->ibmr.rkey); > > The r2, stag and rkey are u32, why do you need cpu_to_be32() conversion? > Hey Leon, The r2 and stag fields in struct fw_ri_fr_nsmr_tpte_wr should be __be32. Steve. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html