From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id ACEBB2B9C; Thu, 5 Jan 2017 14:26:50 +0100 (CET) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga101.jf.intel.com with ESMTP; 05 Jan 2017 05:26:49 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,321,1477983600"; d="scan'208";a="49833614" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by fmsmga005.fm.intel.com with ESMTP; 05 Jan 2017 05:26:48 -0800 To: Rasesh Mody , dev@dpdk.org References: <1483172217-30186-1-git-send-email-rasesh.mody@cavium.com> Cc: Harish Patil , stable@dpdk.org, Dept-EngDPDKDev@cavium.com From: Ferruh Yigit Message-ID: <787fc8d8-1003-d2c0-e8e1-a2b915517456@intel.com> Date: Thu, 5 Jan 2017 13:26:47 +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: <1483172217-30186-1-git-send-email-rasesh.mody@cavium.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [dpdk-stable] [PATCH 1/5] net/qede: fix scatter-gather issue 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: Thu, 05 Jan 2017 13:26:51 -0000 On 12/31/2016 8:16 AM, Rasesh Mody wrote: > From: Harish Patil > > - Make qede_process_sg_pkts() inline and add unlikely check > - Fix mbuf segment chaining logic in qede_process_sg_pkts() > - Change qede_encode_sg_bd() to return total segments required > - Fix first TX buffer descriptor's length > - Replace repeatitive code using a macro > > Fixes: bec0228816c0 ("net/qede: support scatter gather") > > Signed-off-by: Harish Patil > --- Hi Harish, This patch doesn't apply cleanly on top next-net, after pci_dev changes rebased into next-net tree. Can you please send the new version of the patchset rebased on top of latest next-net? Thanks, ferruh <...>