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 BF4BA1100A for ; Thu, 30 Mar 2017 14:23:10 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1490876590; x=1522412590; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=2EU2E36B8DtEZ+sQ8hZs1K423Wk72YnQQKwc2oqvlYM=; b=njMbOvgRTHHn6d7lX/uSMy3NaGLzKXY3qFdxygBsaHEcJD5d0wAwVbn7 FWF7mAk4ig3J70B0dTsPUghAY5YN4A==; Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Mar 2017 05:23:09 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,246,1486454400"; d="scan'208";a="840020019" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.140]) by FMSMGA003.fm.intel.com with SMTP; 30 Mar 2017 05:23:06 -0700 Received: by (sSMTP sendmail emulation); Thu, 30 Mar 2017 13:23:06 +0100 Date: Thu, 30 Mar 2017 13:23:06 +0100 From: Bruce Richardson To: Olivier Matz Cc: dev@dpdk.org, konstantin.ananyev@intel.com, mb@smartsharesystems.com, andrey.chilikin@intel.com, jblunck@infradead.org, nelio.laranjeiro@6wind.com, arybchenko@solarflare.com Message-ID: <20170330122305.GA14272@bricha3-MOBL3.ger.corp.intel.com> References: <1485271173-13408-1-git-send-email-olivier.matz@6wind.com> <1488966121-22853-1-git-send-email-olivier.matz@6wind.com> <20170329175629.68810924@platinum> <20170329200923.GA11516@bricha3-MOBL3.ger.corp.intel.com> <20170330093108.GA10652@bricha3-MOBL3.ger.corp.intel.com> <20170330140236.0d2ebac8@platinum> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170330140236.0d2ebac8@platinum> Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.8.0 (2017-02-23) Subject: Re: [dpdk-dev] [PATCH 0/9] mbuf: structure reorganization 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, 30 Mar 2017 12:23:11 -0000 On Thu, Mar 30, 2017 at 02:02:36PM +0200, Olivier Matz wrote: > On Thu, 30 Mar 2017 10:31:08 +0100, Bruce Richardson wrote: > > On Wed, Mar 29, 2017 at 09:09:23PM +0100, Bruce Richardson wrote: > > > On Wed, Mar 29, 2017 at 05:56:29PM +0200, Olivier Matz wrote: > > > > Hi, > > > > > > > > Does anyone have any other comment on this series? > > > > Can it be applied? > > > > > > > > > > > > Thanks, > > > > Olivier > > > > > > > > > > I assume all driver maintainers have done performance analysis to check > > > for regressions. Perhaps they can confirm this is the case. > > > > > > /Bruce > > > > > > In the absence, of anyone else reporting performance numbers with this > > patchset, I ran a single-thread testpmd test using 2 x 40G ports (i40e) > > driver. With RX & TX descriptor ring sizes of 512 or above, I'm seeing a > > fairly noticable performance drop. I still need to dig in more, e.g. do > > an RFC2544 zero-loss test, and also bisect the patchset to see what > > parts may be causing the problem. > > > > Has anyone else tried any other drivers or systems to see what the perf > > impact of this set may be? > > I did, of course. I didn't see any noticeable performance drop on > ixgbe (4 NICs, one port per NIC, 1 core). I can replay the test with > current version. > I had no doubt you did some perf testing! :-) Perhaps the regression I see is limited to i40e driver. I've confirmed I still see it with that driver in zero-loss tests, so next step is to try and localise what change in the patchset is causing it. Ideally, though, I think we should see acks or other comments from driver maintainers at least confirming that they have tested. You cannot be held responsible for testing every DPDK driver before you submit work like this. Regards, /Bruce