mbox series

[for-next,0/3,v2] qedr: consider dscp prio for vlan tag and update tos

Message ID 20210730065001.805-1-smalin@marvell.com (mailing list archive)
Headers show
Series qedr: consider dscp prio for vlan tag and update tos | expand

Message

Shai Malin July 30, 2021, 6:49 a.m. UTC
From: Prabhakar Kushwaha <pkushwaha@marvell.com>

Add functions to check get/set dscp priority in qed and these functions
are further used in deciding vlan priority based on dscp state.

Also update RDMA CM TOS.

Changes for v2:
	- incorporated Leon Romanovsky's comments

Prabhakar Kushwaha (3):
  qed: add get and set support for dscp priority
  qedr: Consider dscp priority for vlan priority
  qedr: Use grh layer traffic_class as RDMA CM TOS

 drivers/infiniband/hw/qedr/qedr_roce_cm.c  | 13 +++-
 drivers/net/ethernet/qlogic/qed/qed_dcbx.c | 71 ++++++++++++++++++++++
 drivers/net/ethernet/qlogic/qed/qed_dcbx.h | 10 +++
 drivers/net/ethernet/qlogic/qed/qed_rdma.c | 25 ++++++++
 include/linux/qed/qed_if.h                 |  6 ++
 include/linux/qed/qed_rdma_if.h            |  7 +++
 6 files changed, 130 insertions(+), 2 deletions(-)

Comments

Jason Gunthorpe Aug. 20, 2021, 5:49 p.m. UTC | #1
On Fri, Jul 30, 2021 at 09:49:58AM +0300, Shai Malin wrote:
> From: Prabhakar Kushwaha <pkushwaha@marvell.com>
> 
> Add functions to check get/set dscp priority in qed and these functions
> are further used in deciding vlan priority based on dscp state.
> 
> Also update RDMA CM TOS.
> 
> Changes for v2:
> 	- incorporated Leon Romanovsky's comments
> 
> Prabhakar Kushwaha (3):
>   qed: add get and set support for dscp priority
>   qedr: Consider dscp priority for vlan priority
>   qedr: Use grh layer traffic_class as RDMA CM TOS
> 
>  drivers/infiniband/hw/qedr/qedr_roce_cm.c  | 13 +++-
>  drivers/net/ethernet/qlogic/qed/qed_dcbx.c | 71 ++++++++++++++++++++++
>  drivers/net/ethernet/qlogic/qed/qed_dcbx.h | 10 +++
>  drivers/net/ethernet/qlogic/qed/qed_rdma.c | 25 ++++++++
>  include/linux/qed/qed_if.h                 |  6 ++
>  include/linux/qed/qed_rdma_if.h            |  7 +++
>  6 files changed, 130 insertions(+), 2 deletions(-)

Since this is mostly netdev stuff can someone from netdev ack this if
you want it to go through the rdma tree?

Have you checked that there are no conflicts with things in net-next?

Thanks,
Jason
Jakub Kicinski Aug. 20, 2021, 6:02 p.m. UTC | #2
On Fri, 20 Aug 2021 14:49:44 -0300 Jason Gunthorpe wrote:
> Since this is mostly netdev stuff can someone from netdev ack this if
> you want it to go through the rdma tree?

It'd be great to get it CCed to netdev@ for that.
Jason Gunthorpe Aug. 20, 2021, 6:03 p.m. UTC | #3
On Fri, Aug 20, 2021 at 11:02:51AM -0700, Jakub Kicinski wrote:
> On Fri, 20 Aug 2021 14:49:44 -0300 Jason Gunthorpe wrote:
> > Since this is mostly netdev stuff can someone from netdev ack this if
> > you want it to go through the rdma tree?
> 
> It'd be great to get it CCed to netdev@ for that.

Indeed, Shai please respost it.

Jason