From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v6 0/5] net/softnic: sw fall-back pmd for traffic mgmt and others Date: Fri, 6 Oct 2017 19:57:22 +0100 Message-ID: <4307e4ed-a1bf-4da8-de3e-b8469ed6aa12@intel.com> References: <20170929140444.83959-3-jasvinder.singh@intel.com> <20171006170003.77311-1-jasvinder.singh@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: cristian.dumitrescu@intel.com, thomas@monjalon.net, wenzhuo.lu@intel.com To: Jasvinder Singh , dev@dpdk.org Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 427B01B205 for ; Fri, 6 Oct 2017 20:57:26 +0200 (CEST) In-Reply-To: <20171006170003.77311-1-jasvinder.singh@intel.com> Content-Language: en-US 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 10/6/2017 5:59 PM, Jasvinder Singh wrote: > The SoftNIC PMD is intended to provide SW fall-back options for specific > ethdev APIs in a generic way to the NICs not supporting those features. > > Currently, the only implemented ethdev API is Traffic Management (TM), > but other ethdev APIs such as rte_flow, traffic metering & policing, etc > can be easily implemented. <...> > Cristian Dumitrescu (4): > Jasvinder Singh (4): > net/softnic: add softnic PMD > net/softnic: add traffic management support > net/softnic: add TM capabilities ops > net/softnic: add TM hierarchy related ops > > Jasvinder Singh (1): > app/testpmd: add traffic management forwarding mode Hi Jasvinder, There is a patchset has been merged into main and next-net repo to convert port_id from 8bits to 16bits. This PMD also should reflect this conversion, can you please rebase PMD on top of latest next-net? Please keep existing Acks in next version. Also there are lots of UNNECESSARY_PARENTHESES checkpatch warnings, can you please fix them? > > MAINTAINERS | 5 + Checkpatch also gives warning about MAINTAINERS file updates, not sure why, can you please check this? <...>