Hi Saeed,
Thanks for the review.
> -----Original Message-----
> From: Saeed Mahameed <sa...@kernel.org>
> Sent: Friday, November 13, 2020 1:47 AM
> To: Naveen Mamindlapalli <nave...@marvell.com>; netdev@vger.kernel.org;
> linux-ker...@vger.kernel.org
> Cc: k...@kernel.org; da...@davemloft.net; Sunil Kovvuri Goutham
> <sgout...@marvell.com>; Linu Cherian <lcher...@marvell.com>;
> Geethasowjanya Akula <gak...@marvell.com>; Jerin Jacob Kollanukkaran
> <jer...@marvell.com>; Subbaraya Sundeep Bhatta <sbha...@marvell.com>;
> Hariprasad Kelam <hke...@marvell.com>
> Subject: [EXT] Re: [PATCH v3 net-next 00/13] Add ethtool ntuple filters
> support
>
> External Email
>
> ----------------------------------------------------------------------
> On Wed, 2020-11-11 at 12:43 +0530, Naveen Mamindlapalli 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:
> > 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.
> >
>
> Reviewed-by: Saeed Mahameed <sae...@nvidia.com>