mbox series

[0/4] drivers/net: remove legacy filter API and switch to rte flow

Message ID 20200318014710.13577-1-chenxux.di@intel.com (mailing list archive)
Headers
Series drivers/net: remove legacy filter API and switch to rte flow |

Message

Chenxu Di March 18, 2020, 1:47 a.m. UTC
  remove legacy filter functions already implemented in rte_flow
 for drivers igb, ixgbe, and i40e.
implement hash function include set hash function and set hash
input set in rte_flow for driver i40e.

Chenxu Di (4):
  net/e1000: remove the legacy filter functions
  net/ixgbe: remove the legacy filter functions
  net/i40e: remove the legacy filter functions
  net/i40e: implement hash function in rte flow API

 doc/guides/nics/i40e.rst               |  14 +
 doc/guides/rel_notes/release_20_05.rst |   9 +
 drivers/net/e1000/igb_ethdev.c         |  36 -
 drivers/net/i40e/i40e_ethdev.c         | 913 +++++++++++--------------
 drivers/net/i40e/i40e_ethdev.h         |  26 +-
 drivers/net/i40e/i40e_fdir.c           | 393 -----------
 drivers/net/i40e/i40e_flow.c           | 186 ++++-
 drivers/net/ixgbe/ixgbe_ethdev.c       |  78 ---
 drivers/net/ixgbe/ixgbe_fdir.c         |  11 -
 9 files changed, 610 insertions(+), 1056 deletions(-)
  

Comments

Stephen Hemminger March 18, 2020, 3 a.m. UTC | #1
On Wed, 18 Mar 2020 01:47:06 +0000
Chenxu Di <chenxux.di@intel.com> wrote:

> remove legacy filter functions already implemented in rte_flow
>  for drivers igb, ixgbe, and i40e.
> implement hash function include set hash function and set hash
> input set in rte_flow for driver i40e.
> 
> Chenxu Di (4):
>   net/e1000: remove the legacy filter functions
>   net/ixgbe: remove the legacy filter functions
>   net/i40e: remove the legacy filter functions
>   net/i40e: implement hash function in rte flow API
> 
>  doc/guides/nics/i40e.rst               |  14 +
>  doc/guides/rel_notes/release_20_05.rst |   9 +
>  drivers/net/e1000/igb_ethdev.c         |  36 -
>  drivers/net/i40e/i40e_ethdev.c         | 913 +++++++++++--------------
>  drivers/net/i40e/i40e_ethdev.h         |  26 +-
>  drivers/net/i40e/i40e_fdir.c           | 393 -----------
>  drivers/net/i40e/i40e_flow.c           | 186 ++++-
>  drivers/net/ixgbe/ixgbe_ethdev.c       |  78 ---
>  drivers/net/ixgbe/ixgbe_fdir.c         |  11 -
>  9 files changed, 610 insertions(+), 1056 deletions(-)
> 

This looks like an API break for users using the legacy filter API.
Even though filter_ctrl is marked as deprecated. That probably has
to wait for 20.11 until it is removed. At that point, drop the ethdev
ops handle, the rte_eth_dev_filter_ctrl API (etc) and fix all the test
code.