mbox series

[net-next,0/3,pull,request] 100GbE Intel Wired LAN Driver Updates 2021-11-22

Message ID 20211122211119.279885-1-anthony.l.nguyen@intel.com (mailing list archive)
Headers show
Series 100GbE Intel Wired LAN Driver Updates 2021-11-22 | expand

Message

Tony Nguyen Nov. 22, 2021, 9:11 p.m. UTC 
Shiraz Saleem says:

Currently E800 devices come up as RoCEv2 devices by default.

This series add supports for users to configure iWARP or RoCEv2 functionality
per PCI function. devlink parameters is used to realize this and is keyed
off similar work in [1].

[1] https://lore.kernel.org/linux-rdma/20210810132424.9129-1-parav@nvidia.com/

The following are changes since commit 3b0e04140bc30f9f5c254a68013a901e5390b0a8:
  Merge branch 'qca8k-next'
and are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-queue 100GbE

Shiraz Saleem (3):
  devlink: Add 'enable_iwarp' generic device param
  net/ice: Add support for enable_iwarp and enable_roce devlink param
  RDMA/irdma: Set protocol based on PF rdma_mode flag

 .../networking/devlink/devlink-params.rst     |   3 +
 drivers/infiniband/hw/irdma/main.c            |   3 +-
 drivers/net/ethernet/intel/ice/ice.h          |   1 +
 drivers/net/ethernet/intel/ice/ice_devlink.c  | 144 ++++++++++++++++++
 drivers/net/ethernet/intel/ice/ice_devlink.h  |   6 +
 drivers/net/ethernet/intel/ice/ice_idc.c      |   4 +-
 drivers/net/ethernet/intel/ice/ice_main.c     |   9 +-
 include/linux/net/intel/iidc.h                |   7 +-
 include/net/devlink.h                         |   4 +
 net/core/devlink.c                            |   5 +
 10 files changed, 180 insertions(+), 6 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 23, 2021, 12:20 p.m. UTC | #1 
Hello:

This series was applied to netdev/net-next.git (master)
by Tony Nguyen <anthony.l.nguyen@intel.com>:

On Mon, 22 Nov 2021 13:11:16 -0800 you wrote:
> Shiraz Saleem says:
> 
> Currently E800 devices come up as RoCEv2 devices by default.
> 
> This series add supports for users to configure iWARP or RoCEv2 functionality
> per PCI function. devlink parameters is used to realize this and is keyed
> off similar work in [1].
> 
> [...]

Here is the summary with links:
  - [net-next,1/3] devlink: Add 'enable_iwarp' generic device param
    https://git.kernel.org/netdev/net-next/c/325e0d0aa683
  - [net-next,2/3] net/ice: Add support for enable_iwarp and enable_roce devlink param
    https://git.kernel.org/netdev/net-next/c/e523af4ee560
  - [net-next,3/3] RDMA/irdma: Set protocol based on PF rdma_mode flag
    https://git.kernel.org/netdev/net-next/c/774a90c1e1a3

You are awesome, thank you!
Parav Pandit Nov. 23, 2021, 12:31 p.m. UTC | #2 
Hello patchwork-bot,

> From: patchwork-bot+netdevbpf@kernel.org <patchwork-
> bot+netdevbpf@kernel.org>
> 
> Hello:
> 
> This series was applied to netdev/net-next.git (master) by Tony Nguyen
> <anthony.l.nguyen@intel.com>:
> 
> On Mon, 22 Nov 2021 13:11:16 -0800 you wrote:
> > Shiraz Saleem says:
> >
> > Currently E800 devices come up as RoCEv2 devices by default.
> >
> > This series add supports for users to configure iWARP or RoCEv2
> > functionality per PCI function. devlink parameters is used to realize
> > this and is keyed off similar work in [1].
> >
> > [...]
> 
> Here is the summary with links:
>   - [net-next,1/3] devlink: Add 'enable_iwarp' generic device param
>     https://git.kernel.org/netdev/net-next/c/325e0d0aa683
>   - [net-next,2/3] net/ice: Add support for enable_iwarp and enable_roce
> devlink param

Above patch has a bug and review comments provided.
Jason Gunthorpe Nov. 23, 2021, 5:22 p.m. UTC | #3 
On Tue, Nov 23, 2021 at 12:20:12PM +0000, patchwork-bot+netdevbpf@kernel.org wrote:
> Hello:
> 
> This series was applied to netdev/net-next.git (master)
> by Tony Nguyen <anthony.l.nguyen@intel.com>:
> 
> On Mon, 22 Nov 2021 13:11:16 -0800 you wrote:
> > Shiraz Saleem says:
> > 
> > Currently E800 devices come up as RoCEv2 devices by default.
> > 
> > This series add supports for users to configure iWARP or RoCEv2 functionality
> > per PCI function. devlink parameters is used to realize this and is keyed
> > off similar work in [1].
> > 
> > [...]
> 
> Here is the summary with links:
>   - [net-next,1/3] devlink: Add 'enable_iwarp' generic device param
>     https://git.kernel.org/netdev/net-next/c/325e0d0aa683
>   - [net-next,2/3] net/ice: Add support for enable_iwarp and enable_roce devlink param
>     https://git.kernel.org/netdev/net-next/c/e523af4ee560
>   - [net-next,3/3] RDMA/irdma: Set protocol based on PF rdma_mode flag
>     https://git.kernel.org/netdev/net-next/c/774a90c1e1a3

Isn't 15 hours of review time a bit aggressive for patches that
introduce new devlink uAPI?!?!

Jason