> So does anyone see anything obviously missing or wrong here? It looks fine to me with one minor comment. > +struct ibv_mw_bind { > + struct ibv_mr *mr; > + uint64_t wr_id; > + uint64_t addr; > + uint64_t length; The memory region uses size_t for length. Do we care about matching the data type? - Sean