Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20201114195303.25967-1-naveenm@marvell.com>
In-Reply-To: <20201114195303.25967-1-naveenm@marvell.com>
From:   Alexander Duyck <alexander.duyck@gmail.com>
Date:   Sat, 14 Nov 2020 18:10:35 -0800
Message-ID: <CAKgT0Ue=yQmFoazwU29PjVU41ywOUx8vKLxdiNBqhNrj4S=g=w@mail.gmail.com>
Subject: Re: [PATCH v4 net-next 00/13] Add ethtool ntuple filters support
To:     Naveen Mamindlapalli <naveenm@marvell.com>
Cc:     Netdev <netdev@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Jakub Kicinski <kuba@kernel.org>,
        David Miller <davem@davemloft.net>, saeed@kernel.org,
        sgoutham@marvell.com, lcherian@marvell.com, gakula@marvell.com,
        jerinj@marvell.com, sbhatta@marvell.com, hkelam@marvell.com
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Sat, Nov 14, 2020 at 11:53 AM Naveen Mamindlapalli
<naveenm@marvell.com> 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@fb.com>