On Tue, May 01, 2018 at 05:35:36AM -0700, Dennis Dalessandro wrote: > @@ -719,7 +719,7 @@ void return_cnp(struct hfi1_ibport *ibp, struct rvt_qp *qp, u32 remote_qpn, > > lrh0 |= (sc5 & 0xf) << 12 | sl << 4; > > - bth0 = pkey | (IB_OPCODE_CNP << 24); > + bth0 = (u32)pkey | (IB_OPCODE_CNP << 24); What do you think this cast does?? void return_cnp_16B(struct hfi1_ibport *ibp, struct rvt_qp *qp, u32 remote_qpn, u32 pkey, u32 slid, u32 dlid, u8 sc5, const struct ib_grh *old_grh) { Jason