Message ID | 20180612025650.GA29025@ziepe.ca (mailing list archive) |
---|---|
State | Accepted |
Delegated to: | Jason Gunthorpe |
Headers | show |
On 2018/6/12 10:56, Jason Gunthorpe wrote: > struct rxe_global_route and struct ib_global_route are not the same thing > and should not be memcpy'd over each other, do a member by member copy > instead. This allows the layout of the in-kernel struct ib_global_route to > be changed without breaking rxe. > > Signed-off-by: Jason Gunthorpe <jgg@mellanox.com> Reviewed-by: Zhu Yanjun <yanjun.zhu@oracle.com> > --- > drivers/infiniband/sw/rxe/rxe_av.c | 19 ++++++++++++++++--- > 1 file changed, 16 insertions(+), 3 deletions(-) > > diff --git a/drivers/infiniband/sw/rxe/rxe_av.c b/drivers/infiniband/sw/rxe/rxe_av.c > index c6bce915df50bc..26fe8d7dbc55f3 100644 > --- a/drivers/infiniband/sw/rxe/rxe_av.c > +++ b/drivers/infiniband/sw/rxe/rxe_av.c > @@ -55,16 +55,29 @@ int rxe_av_chk_attr(struct rxe_dev *rxe, struct rdma_ah_attr *attr) > void rxe_av_from_attr(u8 port_num, struct rxe_av *av, > struct rdma_ah_attr *attr) > { > + const struct ib_global_route *grh = rdma_ah_read_grh(attr); > + > memset(av, 0, sizeof(*av)); > - memcpy(&av->grh, rdma_ah_read_grh(attr), > - sizeof(*rdma_ah_read_grh(attr))); > + memcpy(av->grh.dgid.raw, grh->dgid.raw, sizeof(grh->dgid.raw)); > + av->grh.flow_label = grh->flow_label; > + av->grh.sgid_index = grh->sgid_index; > + av->grh.hop_limit = grh->hop_limit; > + av->grh.traffic_class = grh->traffic_class; > av->port_num = port_num; > } > > void rxe_av_to_attr(struct rxe_av *av, struct rdma_ah_attr *attr) > { > + struct ib_global_route *grh = rdma_ah_retrieve_grh(attr); > + > attr->type = RDMA_AH_ATTR_TYPE_ROCE; > - memcpy(rdma_ah_retrieve_grh(attr), &av->grh, sizeof(av->grh)); > + > + memcpy(grh->dgid.raw, av->grh.dgid.raw, sizeof(av->grh.dgid.raw)); > + grh->flow_label = av->grh.flow_label; > + grh->sgid_index = av->grh.sgid_index; > + grh->hop_limit = av->grh.hop_limit; > + grh->traffic_class = av->grh.traffic_class; > + > rdma_ah_set_ah_flags(attr, IB_AH_GRH); > rdma_ah_set_port_num(attr, av->port_num); > } -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jun 11, 2018 at 08:56:50PM -0600, Jason Gunthorpe wrote: > struct rxe_global_route and struct ib_global_route are not the same thing > and should not be memcpy'd over each other, do a member by member copy > instead. This allows the layout of the in-kernel struct ib_global_route to > be changed without breaking rxe. > > Signed-off-by: Jason Gunthorpe <jgg@mellanox.com> > --- > drivers/infiniband/sw/rxe/rxe_av.c | 19 ++++++++++++++++--- > 1 file changed, 16 insertions(+), 3 deletions(-) > Thanks, Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
On Mon, Jun 11, 2018 at 08:56:50PM -0600, Jason Gunthorpe wrote: > struct rxe_global_route and struct ib_global_route are not the same thing > and should not be memcpy'd over each other, do a member by member copy > instead. This allows the layout of the in-kernel struct ib_global_route to > be changed without breaking rxe. > > Signed-off-by: Jason Gunthorpe <jgg@mellanox.com> > Reviewed-by: Zhu Yanjun <yanjun.zhu@oracle.com> > Reviewed-by: Leon Romanovsky <leonro@mellanox.com> > --- > drivers/infiniband/sw/rxe/rxe_av.c | 19 ++++++++++++++++--- > 1 file changed, 16 insertions(+), 3 deletions(-) Applied, thanks Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/infiniband/sw/rxe/rxe_av.c b/drivers/infiniband/sw/rxe/rxe_av.c index c6bce915df50bc..26fe8d7dbc55f3 100644 --- a/drivers/infiniband/sw/rxe/rxe_av.c +++ b/drivers/infiniband/sw/rxe/rxe_av.c @@ -55,16 +55,29 @@ int rxe_av_chk_attr(struct rxe_dev *rxe, struct rdma_ah_attr *attr) void rxe_av_from_attr(u8 port_num, struct rxe_av *av, struct rdma_ah_attr *attr) { + const struct ib_global_route *grh = rdma_ah_read_grh(attr); + memset(av, 0, sizeof(*av)); - memcpy(&av->grh, rdma_ah_read_grh(attr), - sizeof(*rdma_ah_read_grh(attr))); + memcpy(av->grh.dgid.raw, grh->dgid.raw, sizeof(grh->dgid.raw)); + av->grh.flow_label = grh->flow_label; + av->grh.sgid_index = grh->sgid_index; + av->grh.hop_limit = grh->hop_limit; + av->grh.traffic_class = grh->traffic_class; av->port_num = port_num; } void rxe_av_to_attr(struct rxe_av *av, struct rdma_ah_attr *attr) { + struct ib_global_route *grh = rdma_ah_retrieve_grh(attr); + attr->type = RDMA_AH_ATTR_TYPE_ROCE; - memcpy(rdma_ah_retrieve_grh(attr), &av->grh, sizeof(av->grh)); + + memcpy(grh->dgid.raw, av->grh.dgid.raw, sizeof(av->grh.dgid.raw)); + grh->flow_label = av->grh.flow_label; + grh->sgid_index = av->grh.sgid_index; + grh->hop_limit = av->grh.hop_limit; + grh->traffic_class = av->grh.traffic_class; + rdma_ah_set_ah_flags(attr, IB_AH_GRH); rdma_ah_set_port_num(attr, av->port_num); }
struct rxe_global_route and struct ib_global_route are not the same thing and should not be memcpy'd over each other, do a member by member copy instead. This allows the layout of the in-kernel struct ib_global_route to be changed without breaking rxe. Signed-off-by: Jason Gunthorpe <jgg@mellanox.com> --- drivers/infiniband/sw/rxe/rxe_av.c | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-)