From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 9B4332C38 for ; Mon, 29 May 2017 19:42:56 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 May 2017 10:42:55 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,415,1491289200"; d="scan'208";a="267757102" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.81]) ([10.237.220.81]) by fmsmga004.fm.intel.com with ESMTP; 29 May 2017 10:42:54 -0700 To: Ajit Khaparde , dev@dpdk.org References: <20170526183941.80678-1-ajit.khaparde@broadcom.com> <20170526183941.80678-10-ajit.khaparde@broadcom.com> From: Ferruh Yigit Message-ID: Date: Mon, 29 May 2017 18:42:53 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170526183941.80678-10-ajit.khaparde@broadcom.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 09/25] bnxt: add support for set multicast addr list and MAC addr set X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 29 May 2017 17:42:57 -0000 On 5/26/2017 7:39 PM, Ajit Khaparde wrote: > This patch adds support for set_mc_addr_list and > mac_addr_set dev_ops > > Signed-off-by: Ajit Khaparde > > -- > v1->v2: regroup related patches and incorporate other review comments > --- > doc/guides/nics/features/bnxt.ini | 1 + > drivers/net/bnxt/bnxt_ethdev.c | 70 ++++++++++++++++++++++++++++++++++++++- > drivers/net/bnxt/bnxt_hwrm.c | 13 +++++--- > drivers/net/bnxt/bnxt_hwrm.h | 2 +- > drivers/net/bnxt/bnxt_vnic.c | 7 +++- > drivers/net/bnxt/bnxt_vnic.h | 4 +++ > 6 files changed, 90 insertions(+), 7 deletions(-) > > diff --git a/doc/guides/nics/features/bnxt.ini b/doc/guides/nics/features/bnxt.ini > index 78a738f..9b3cd94 100644 > --- a/doc/guides/nics/features/bnxt.ini > +++ b/doc/guides/nics/features/bnxt.ini > @@ -11,6 +11,7 @@ Unicast MAC filter = Y > Multicast MAC filter = Y > VLAN filter = Y > VLAN offload = Y > +Allmulticast mode = Y To have this support, .allmulticast_enable and .allmulticast_disable needs to implemented, these are already implemented in the PMD. This PMD implements ".set_mc_addr_list", I think this is "Multicast MAC filter", which seems already documented as supported. So, it is possible to have a separate patch to document "Allmulticast mode" > RSS reta update = Y > Basic stats = Y > Extended stats = Y <...>