Re: [PATCH v4 net-next 00/13] Add ethtool ntuple filters support

[Alexander Duyck]

On Sat, Nov 14, 2020 at 11:53 AM Naveen Mamindlapalli
<naveenm@xxxxxxxxxxx> wrote:
>
> This patch series adds support for ethtool ntuple filters, unicast
> address filtering, VLAN offload and SR-IOV ndo handlers. All of the
> above features are based on the Admin Function(AF) driver support to
> install and delete the low level MCAM entries. Each MCAM entry is
> programmed with the packet fields to match and what actions to take
> if the match succeeds. The PF driver requests AF driver to allocate
> set of MCAM entries to be used to install the flows by that PF. The
> entries will be freed when the PF driver is unloaded.
>
> * The patches 1 to 4 adds AF driver infrastructure to install and
>   delete the low level MCAM flow entries.
> * Patch 5 adds ethtool ntuple filter support.
> * Patch 6 adds unicast MAC address filtering.
> * Patch 7 adds support for dumping the MCAM entries via debugfs.
> * Patches 8 to 10 adds support for VLAN offload.
> * Patch 10 to 11 adds support for SR-IOV ndo handlers.
> * Patch 12 adds support to read the MCAM entries.
>
> Misc:
> * Removed redundant mailbox NIX_RXVLAN_ALLOC.
>
> Change-log:
> v4:
> - Fixed review comments from Alexander Duyck on v3.
>         - Added macros for KEX profile configuration values.
>         - TCP/UDP SPORT+DPORT extracted using single entry.
>         - Use eth_broadcast_addr() instead of memcpy to avoid one extra variable.
>         - Fix "ether type" to "Ethertype" & "meta data" to "metadata" in comments.
>         - Added more comments.
> v3:
> - Fixed Saeed's review comments on v2.
>         - Fixed modifying the netdev->flags from driver.
>         - Fixed modifying the netdev features and hw_features after register_netdev.
>         - Removed unwanted ndo_features_check callback.
> v2:
> - Fixed the sparse issues reported by Jakub.

All of the fixes look like they are in place.

Reviewed-by: Alexander Duyck <alexanderduyck@xxxxxx>