netdev - RE: [Intel-wired-lan] [PATCH iwl-next v4] ice: Add support for devlink local

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] [day] [month] [year] [list]

Message-ID: <SJ0PR11MB5865732B1441F8CD70CF61E48FD32@SJ0PR11MB5865.namprd11.prod.outlook.com>
Date: Mon, 1 Jul 2024 07:43:01 +0000
From: "Romanowski, Rafal" <rafal.romanowski@...el.com>
To: "Drewek, Wojciech" <wojciech.drewek@...el.com>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>
CC: "Keller, Jacob E" <jacob.e.keller@...el.com>, "kuba@...nel.org"
	<kuba@...nel.org>, "intel-wired-lan@...ts.osuosl.org"
	<intel-wired-lan@...ts.osuosl.org>, "jiri@...nulli.us" <jiri@...nulli.us>
Subject: RE: [Intel-wired-lan] [PATCH iwl-next v4] ice: Add support for
 devlink local_forwarding param.

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@...osl.org> On Behalf Of
> Wojciech Drewek
> Sent: Friday, June 7, 2024 12:44 PM
> To: netdev@...r.kernel.org
> Cc: Keller, Jacob E <jacob.e.keller@...el.com>; kuba@...nel.org; intel-wired-
> lan@...ts.osuosl.org; jiri@...nulli.us
> Subject: [Intel-wired-lan] [PATCH iwl-next v4] ice: Add support for devlink
> local_forwarding param.
> 
> From: Pawel Kaminski <pawel.kaminski@...el.com>
> 
> Add support for driver-specific devlink local_forwarding param.
> Supported values are "enabled", "disabled" and "prioritized".
> Default configuration is set to "enabled".
> 
> Add documentation in networking/devlink/ice.rst.
> 
> In previous generations of Intel NICs the transmit scheduler was only limited by
> PCIe bandwidth when scheduling/assigning hairpin-badwidth between VFs.
> Changes to E810 HW design introduced scheduler limitation, so that available
> hairpin-bandwidth is bound to external port speed.
> In order to address this limitation and enable NFV services such as "service
> chaining" a knob to adjust the scheduler config was created.
> Driver can send a configuration message to the FW over admin queue and
> internal FW logic will reconfigure HW to prioritize and add more BW to VF to VF
> traffic. As end result for example 10G port will no longer limit hairpin-badwith to
> 10G and much higher speeds can be achieved.
> 
> Devlink local_forwarding param set to "prioritized" enables higher hairpin-
> badwitdh on related PFs. Configuration is applicable only to 8x10G and 4x25G
> cards.
> 
> Changing local_forwarding configuration will trigger CORER reset in order to take
> effect.
> 
> Example command to change current value:
> devlink dev param set pci/0000:b2:00.3 name local_forwarding \
>         value prioritized \
>         cmode runtime
> 
> Co-developed-by: Michal Wilczynski <michal.wilczynski@...el.com>
> Signed-off-by: Michal Wilczynski <michal.wilczynski@...el.com>
> Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@...el.com>
> Signed-off-by: Pawel Kaminski <pawel.kaminski@...el.com>
> Signed-off-by: Wojciech Drewek <wojciech.drewek@...el.com>
> ---
> v2: Extend documentation
> v3: rename loopback to local_forwarding
> v4: change in documentation about what types of functions
>     are affected
> ---
>  Documentation/networking/devlink/ice.rst      |  25 ++++
>  .../net/ethernet/intel/ice/devlink/devlink.c  | 126 ++++++++++++++++++
>  .../net/ethernet/intel/ice/ice_adminq_cmd.h   |  11 +-
>  drivers/net/ethernet/intel/ice/ice_common.c   |   4 +
>  drivers/net/ethernet/intel/ice/ice_type.h     |   1 +
>  5 files changed, 166 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/networking/devlink/ice.rst
> b/Documentation/networking/devlink/ice.rst
> index 830c04354222..e3972d03cea0 100644
> --- a/Documentation/networking/devlink/ice.rst
> +++ b/Documentation/networking/devlink/ice.rst
> @@ -11,6 +11,7 @@ Parameters


Tested-by: Rafal Romanowski <rafal.romanowski@...el.com>