Thread (5 messages) 5 messages, 3 authors, 2022-11-02

Re: [PATCH net-next] ethtool: Fail number of channels change when it conflicts with rxnfc

From: Jakub Kicinski <kuba@kernel.org>
Date: 2022-11-01 01:23:53

On Mon, 31 Oct 2022 12:00:16 +0200 Gal Pressman wrote:
Similar to what we do with the hash indirection table [1], when network
flow classification rules are forwarding traffic to channels greater
than the requested number of channels, fail the operation.
Without this, traffic could be directed to channels which no longer
exist (dropped) after changing number of channels.

[1] commit d4ab4286276f ("ethtool: correctly ensure {GS}CHANNELS doesn't conflict with GS{RXFH}")
Have you made sure there are no magic encodings of queue numbers this
would break? I seem to recall some vendors used magic queue values to
redirect to VFs before TC and switchdev. If that's the case we'd need
to locate the drivers that do that and flag them so we can enforce this
only going forward?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help