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:
 <PH0PR12MB5481898C4B58CF660B1603DDDC052@PH0PR12MB5481.namprd12.prod.outlook.com>
Date: Thu, 11 Apr 2024 02:32:49 +0000
From: Parav Pandit <parav@...dia.com>
To: "Samudrala, Sridhar" <sridhar.samudrala@...el.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>, "dsahern@...nel.org"
	<dsahern@...nel.org>, "stephen@...workplumber.org"
	<stephen@...workplumber.org>
CC: Jiri Pirko <jiri@...dia.com>, Shay Drori <shayd@...dia.com>
Subject: RE: [PATCH v2 0/2] devlink: Support setting max_io_eqs

Hi Sridhar,

> From: Samudrala, Sridhar <sridhar.samudrala@...el.com>
> Sent: Thursday, April 11, 2024 4:53 AM
> 
> 
> 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?

Currently we don’t. They are derived from the IO event queues.
As you know, sometimes more txqs than IO event queues needed for XDP, timestamp, multiple TCs.
If needed, probably additional knob for txq, rxq can be added to restrict device resources.

> 
> 
> >
> > 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(-)
> >