netdev - Re: [PATCH v2 0/2] devlink: Support setting max_io

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: <a0c707e8-5075-43a2-9c29-00bc044b07b4@intel.com>
Date: Wed, 10 Apr 2024 18:22:30 -0500
From: "Samudrala, Sridhar" <sridhar.samudrala@...el.com>
To: Parav Pandit <parav@...dia.com>, <netdev@...r.kernel.org>,
	<dsahern@...nel.org>, <stephen@...workplumber.org>
CC: <jiri@...dia.com>, <shayd@...dia.com>
Subject: Re: [PATCH v2 0/2] devlink: Support setting max_io_eqs



On 4/10/2024 6:58 AM, Parav Pandit wrote:
> Devices send event notifications for the IO queues,
> such as tx and rx queues, through event queues.
> 
> Enable a privileged owner, such as a hypervisor PF, to set the number
> of IO event queues for the VF and SF during the provisioning stage.

How do you provision tx/rx queues for VFs & SFs?
Don't you need similar mechanism to setup max tx/rx queues too?


> 
> example:
> Get maximum IO event queues of the VF device::
> 
>    $ devlink port show pci/0000:06:00.0/2
>    pci/0000:06:00.0/2: type eth netdev enp6s0pf0vf1 flavour pcivf pfnum 0 vfnum 1
>        function:
>            hw_addr 00:00:00:00:00:00 ipsec_packet disabled max_io_eqs 10
> 
> Set maximum IO event queues of the VF device::
> 
>    $ devlink port function set pci/0000:06:00.0/2 max_io_eqs 32
> 
>    $ devlink port show pci/0000:06:00.0/2
>    pci/0000:06:00.0/2: type eth netdev enp6s0pf0vf1 flavour pcivf pfnum 0 vfnum 1
>        function:
>            hw_addr 00:00:00:00:00:00 ipsec_packet disabled max_io_eqs 32
> 
> patch summary:
> patch-1 updates devlink uapi
> patch-2 adds print, get and set routines for max_io_eqs field
> 
> changelog:
> v1->v2:
> - addressed comments from Jiri
> - updated man page for the new parameter
> - corrected print to not have EQs value as optional
> - replaced 'value' with 'EQs'
> 
> Parav Pandit (2):
>    uapi: Update devlink kernel headers
>    devlink: Support setting max_io_eqs
> 
>   devlink/devlink.c            | 29 ++++++++++++++++++++++++++++-
>   include/uapi/linux/devlink.h |  1 +
>   man/man8/devlink-port.8      | 12 ++++++++++++
>   3 files changed, 41 insertions(+), 1 deletion(-)
>