From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] net/sfc: fix Tx reap behaviour on port stop on EF10 datapath Date: Mon, 23 Oct 2017 11:08:43 -0700 Message-ID: <5a35c104-6c8c-6b08-3a71-aa02a6142282@intel.com> References: <1508319597-21039-1-git-send-email-arybchenko@solarflare.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Ivan Malov To: Andrew Rybchenko , dev@dpdk.org Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 9D8461B685 for ; Mon, 23 Oct 2017 20:08:45 +0200 (CEST) In-Reply-To: <1508319597-21039-1-git-send-email-arybchenko@solarflare.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/18/2017 2:39 AM, Andrew Rybchenko wrote: > From: Ivan Malov > > Tx reap mechanism on EF10 native datapath was altered by one > of the recent patches to introduce performance optimisations > using the common technique of freeing mbuf segments in bulks. > > From this perspective, the way of associating SW descriptors > with individual mbuf segments rather than with whole packets > was adopted as a key requirement for the entire optimisation. > > However, only the fast path reap function was amended to fit > the new scheme whilst the corresponding function on the port > stop path was left intact by mistake. This implies incorrect > usage of rte_pktmbuf_free() with regard to separate segments > rather than calling rte_pktmbuf_free_seg() and must be fixed. Good to know you are OK Ivan, I was worried when previous a few commit logs were not perfectly aligned :) > > Fixes: d321954343c8 ("net/sfc: free mbufs in bulks on EF10 native Tx reap") > > Signed-off-by: Ivan Malov > Signed-off-by: Andrew Rybchenko > Reviewed-by: Andy Moreton Applied to dpdk-next-net/master, thanks.