linux-kernel - Re: [PATCH for-next] RDMA/hns: Support DSCP

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240401181204.GC11187@unreal>
Date: Mon, 1 Apr 2024 21:12:04 +0300
From: Leon Romanovsky <leon@...nel.org>
To: Junxian Huang <huangjunxian6@...ilicon.com>
Cc: jgg@...pe.ca, linux-rdma@...r.kernel.org, linuxarm@...wei.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH for-next] RDMA/hns: Support DSCP

On Mon, Apr 01, 2024 at 09:25:39PM +0800, Junxian Huang wrote:
> 
> 
> On 2024/4/1 19:48, Leon Romanovsky wrote:
> > On Fri, Mar 15, 2024 at 05:35:51PM +0800, Junxian Huang wrote:
> >> Add support for DSCP configuration. For DSCP, get dscp-prio mapping
> >> via hns3 nic driver api .get_dscp_prio() and fill the SL (in WQE for
> >> UD or in QPC for RC) with the priority value. The prio-tc mapping is
> >> configured to HW by hns3 nic driver. HW will select a corresponding
> >> TC according to SL and the prio-tc mapping.
> >>
> >> Signed-off-by: Junxian Huang <huangjunxian6@...ilicon.com>
> >> ---
> >>  drivers/infiniband/hw/hns/hns_roce_ah.c     | 32 +++++---
> >>  drivers/infiniband/hw/hns/hns_roce_device.h |  6 ++
> >>  drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 86 ++++++++++++++++-----
> >>  drivers/infiniband/hw/hns/hns_roce_qp.c     | 13 ++++
> >>  include/uapi/rdma/hns-abi.h                 |  9 ++-
> >>  5 files changed, 117 insertions(+), 29 deletions(-)
> > 
> > 1. What is TC mode?
> 
> TC mode indicates whether the HW is configured as DSCP mode or VLAN priority
> mode currently.
> 
> > 2. Did you post rdma-core PR?
> 
> Not yet. I was meant to wait until this patch is applied. Should I post it
> right now?

Yes, for any UAPI changes, we require to have rdma-core PR.

Thanks

> 
> Junxian
> 
> > 
> > Thanks