Commit 434495c5 authored by Ben Hutchings's avatar Ben Hutchings Committed by David S. Miller

ethtool: Explicitly state that RX NFC rule locations are priorities

The location of an RX flow classification rule is needed to identify
it for retrieval, replacement or deletion.  However it also defines
the priority of the rule in the case that a flow is matched by
multiple rules.  This is what I intended to imply by referring to the
use of a TCAM, commonly used to implement that behaviour.

However there are other ways this can be done, and it is better to
specify this explicitly.  Further, I want to add the option for
automatic selection of rule locations.
Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9927c893
...@@ -456,7 +456,9 @@ struct ethtool_flow_ext { ...@@ -456,7 +456,9 @@ struct ethtool_flow_ext {
* includes the %FLOW_EXT flag. * includes the %FLOW_EXT flag.
* @ring_cookie: RX ring/queue index to deliver to, or %RX_CLS_FLOW_DISC * @ring_cookie: RX ring/queue index to deliver to, or %RX_CLS_FLOW_DISC
* if packets should be discarded * if packets should be discarded
* @location: Location of rule in the table * @location: Location of rule in the table. Locations must be
* numbered such that a flow matching multiple rules will be
* classified according to the first (lowest numbered) rule.
*/ */
struct ethtool_rx_flow_spec { struct ethtool_rx_flow_spec {
__u32 flow_type; __u32 flow_type;
...@@ -502,9 +504,6 @@ struct ethtool_rx_flow_spec { ...@@ -502,9 +504,6 @@ struct ethtool_rx_flow_spec {
* *
* For %ETHTOOL_SRXCLSRLDEL, @fs.@location specifies the location of an * For %ETHTOOL_SRXCLSRLDEL, @fs.@location specifies the location of an
* existing rule on entry. * existing rule on entry.
*
* Implementation of indexed classification rules generally requires a
* TCAM.
*/ */
struct ethtool_rxnfc { struct ethtool_rxnfc {
__u32 cmd; __u32 cmd;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment