From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 2/7] net/qede: refactor filtering code Date: Thu, 8 Dec 2016 16:45:00 +0000 Message-ID: References: <1480733039-13046-1-git-send-email-harish.patil@qlogic.com> <1480733039-13046-2-git-send-email-harish.patil@qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Cc: Dept-EngDPDKDev@qlogic.com To: Harish Patil , dev@dpdk.org Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id C14C632A5 for ; Thu, 8 Dec 2016 17:45:04 +0100 (CET) In-Reply-To: <1480733039-13046-2-git-send-email-harish.patil@qlogic.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 12/3/2016 2:43 AM, Harish Patil wrote: > The filter_config in qed_eth_ops_pass is a wrapper call driving all the > filtering configuration. This requires defining multiple structures and > passing different function arguments for each filter type which is > unnecessary. So as part of this refactoring we remove filter_config from > qed_eth_ops_pass and invoke base apis directly. Another change is to > use a singly list for unicast/multicast macs and thereby prevent duplicate singly linked list? > entries. > > This change is primarily intended to support future tunneling support > which makes use of existing L2 filtering/classifications. > > Fixes: 2ea6f76a ("qede: add core driver") What is fixed in this patch, isn't it a refactor? btw, all Fixes formats are wrong in the patchset, can you please use the git alias provided: http://dpdk.org/doc/guides/contributing/patches.html#commit-messages-body > > Signed-off-by: Harish Patil > --- <...>