Re: [PATCH rdma-next] IB/cma: Set default gid type to RoCEv2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jul 28, 2017 at 01:50:41PM -0400, Doug Ledford wrote:
> On Wed, 2017-06-07 at 12:07 +0300, Leon Romanovsky wrote:
> > On Tue, May 30, 2017 at 09:47:34AM +0300, Leon Romanovsky wrote:
> > > From: Moni Shoua <monis@xxxxxxxxxxxx>
> > >
> > > RoCEv2 is the preferred RDMA protocol for Ethernet link layer
> > > because
> > > of its advantages over RoCEv1. For better user experience make it
> > > the
> > > default choice for RDMA_CM connections if device/port support it.
> > >
> > > Signed-off-by: Moni Shoua <monis@xxxxxxxxxxxx>
> > > Reviewed-by: Matan Barak <matanb@xxxxxxxxxxxx>
> > > Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx>
> > > ---
> > >  drivers/infiniband/core/cma.c | 9 +++++++--
> > >  1 file changed, 7 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/infiniband/core/cma.c
> > > b/drivers/infiniband/core/cma.c
> > > index 91b7a2fe5a55..5ce3c9e1657e 100644
> > > --- a/drivers/infiniband/core/cma.c
> > > +++ b/drivers/infiniband/core/cma.c
> > > @@ -72,6 +72,7 @@ MODULE_LICENSE("Dual BSD/GPL");
> > >  #define CMA_MAX_CM_RETRIES 15
> > >  #define CMA_CM_MRA_SETTING (IB_CM_MRA_FLAG_DELAY | 24)
> > >  #define CMA_IBOE_PACKET_LIFETIME 18
> > > +#define CMA_PREFERRED_ROCE_GID_TYPE IB_GID_TYPE_ROCE_UDP_ENCAP
> >
> > It should be (1 << IB_GID_TYPE_ROCE_UDP_ENCAP) and not as was written
> > above. We will send a fix for that.
>
> I didn't see a followup that fixed this, so I took this patch and then
> created a fix patch myself.

Thanks.

>
> --
> Doug Ledford <dledford@xxxxxxxxxx>
>     GPG KeyID: B826A3330E572FDD
>     Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD
>
> --
> 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

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux