Re: [PATCH for-next 1/3] IB/core: Add optional AV attribute for Raw Packet QP in INIT to RTR

[Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1301 bytes --]

On Mon, Jul 04, 2016 at 09:33:37AM -0600, Jason Gunthorpe wrote:
> On Mon, Jul 04, 2016 at 12:51:00AM +0300, Or Gerlitz wrote:
> > On Sun, Jul 3, 2016 at 3:42 PM, Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
> > > From: Talat Batheesh <talatb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> > >
> > > In order to configure QoS for Raw Packet QP, the user passes the SL
> > > in the AV attribute when modifying the QP from INIT to RTR state.
> > >
> > > This SL is used later on to set the Ethernet Priority of the QP.
> > >
> > > Until today, we couldn't pass the AV attribute for Raw Packet QP
> > > because it was blocked in the QP state transition matrix. This
> > > patch adds the AV as optional in INIT to RTR.
> > >
> > > Fixes: 8a51866f0810 ('IB: Add ib_modify_qp_is_ok() library function')
> > 
> > This commit is 10y old and was done way before RAW Packet QP was added
> > to the kernel, please fix your Fixes: line...
> 
> It isn't a fixes if you are adding a new feature.

You are right, I'll resubmit.

> 
> Jason
> --
> 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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]