On Wed, Jan 03, 2018 at 10:47:14PM +0200, Or Gerlitz wrote: > On Wed, Jan 3, 2018 at 10:43 PM, Daniel Jurgens wrote: > > On 1/3/2018 2:39 PM, Or Gerlitz wrote: > >> On Wed, Jan 3, 2018 at 10:35 PM, Daniel Jurgens wrote: > >>> On 12/25/2017 3:17 AM, Or Gerlitz wrote: > >>>> On Mon, Dec 25, 2017 at 9:16 AM, Leon Romanovsky wrote: > >>>>> On Mon, Dec 25, 2017 at 12:16:15AM +0200, Or Gerlitz wrote: > >>>>>> On Sun, Dec 24, 2017 at 2:57 PM, Leon Romanovsky wrote: > >>>>>>> From: Daniel Jurgens > >>>>>>> > >>>>>>> When operating in dual port RoCE mode RAW QPs are not supposed > >>>>>>> to work on the slave port. > >>>>>> why? > >>>>> HW create flow table restriction. > >>>> cryptic to me, either put a proper explanation on the V2 change log > >>>> for this patch > >>>> so we can review it then or explain it here if you want a review on V1 > >>> We don't report RAW QP support on a per port basis. It's per device. > >>> The user doesn't know which port is the master/slave, so we shouldn't > >>> support RAW QP at all in this mode. I'll update the commit message to state that. > >> in mlx4 we do have IB device with two ports and RAW QP is supported, > >> what is different there? > > > > mlx4 has a completely different HW architecture. It's 2 ports on 1 PCI function. This feature doesn't do that for mlx5, there are still 2 PCI functions. It uses steering rules to route RDMA traffic arriving at the 2nd port to resources in the domain of the 1st port. RDMA traffic is easy to identify and steer accordingly, generic traffic over a Raw QP is not. > > ok, so this is basically uneasy but not undoable. Aren't we creating > possible regressions for people that used RAW QP on mlx5 devices? > these > are open-systems and the code is out there for people to use it. Now > the systems that move to the new mode has inferior functionality. No, we are not creating regression mainly because of two reasons: 1. This dual-port functionality is enabled by special FW and not available by default. 2. Programmers were supposed to query device capabilities and ensure that RAW QP is supported prior to use it, like any other feature advertised by those capabilities. This is the whole idea of device capabilities. Thanks > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html