From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id D4D03F94D for ; Wed, 18 Jan 2017 11:06:27 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 18 Jan 2017 02:06:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,249,1477983600"; d="scan'208";a="1084371148" Received: from dartyukh-mobl8.ger.corp.intel.com (HELO [10.252.26.107]) ([10.252.26.107]) by orsmga001.jf.intel.com with ESMTP; 18 Jan 2017 02:06:24 -0800 To: Sergey Vyazmitinov , olivier.matz@6wind.com References: <1484731386-3752-1-git-send-email-s.vyazmitinov@brain4net.com> Cc: konstantin.ananyev@intel.com, stephen@networkplumber.org, yuanhan.liu@linux.intel.com, dev@dpdk.org From: Ferruh Yigit Message-ID: Date: Wed, 18 Jan 2017 10:06:24 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <1484731386-3752-1-git-send-email-s.vyazmitinov@brain4net.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH] kni: add bulk function to free mbufs 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: Wed, 18 Jan 2017 10:06:28 -0000 On 1/18/2017 9:23 AM, Sergey Vyazmitinov wrote: > Suggested-by: Stephen Hemminger > Signed-off-by: Sergey Vyazmitinov > --- > v3: > * Fixed issue with possible different mempools in buffer list. > * Fixed issue with wrong rte_pktmbuf_alloc_bulk function return value > processing in the kni_allocate_mbufs. Hi Sergey, Thank you for the patch. I aware you sent this patch to separate mbuf changes, but kni v3 patch should reflect this update. Would you mind doing a v4, as patchset, which consists of two pathes: 1- This patch 2- kni v3 patch without mbuf changes. There are a few copies of patch around, I am marking all as "Changes Requested" in patchwork, fyi. Thanks, ferruh