Thread (42 messages) 42 messages, 6 authors, 2022-02-09

RE: [PATCH net-next 1/2] devlink: Add support to set port function as trusted

From: Parav Pandit <hidden>
Date: 2022-01-20 06:19:24

From: Saeed Mahameed <saeedm@nvidia.com>
Sent: Thursday, January 20, 2022 11:34 AM

On 20 Jan 04:52, Parav Pandit wrote:
quoted
quoted
From: Saeed Mahameed <saeedm@nvidia.com>
Sent: Thursday, January 20, 2022 6:11 AM
[..]
quoted
quoted
I do agree you and Saeed that instead of port function param, port
function
resource is more suitable here even though its bool.
quoted
I believe flexibility can be achieved with some FW message? Parav can
you investigate ? To be clear here the knob must be specific to
sw_steering exposed as memory resource.
Sure.
I currently think of user interface something like below, I will get
back with more plumbing of netlink and enum/string.

# to enable
devlink port function resource set pci/0000:03:00.0/port_index
device_memory/sw_steering 1
this looks like an abuse of the interface, I literally meant to control the amount
of ICM pages dedicated for SW steering per function, this requires some FW
support, but i think this is the correct direction.
Ok. I will evaluate and update.
quoted
# to disable
devlink port function resource set pci/0000:03:00.0/port_index
device_memory/sw_steering 0 (current default)

Thanks Jakub, Saeed for the inputs and direction.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help