From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id DF8A92FDD for ; Tue, 4 Apr 2017 10:53:48 +0200 (CEST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP; 04 Apr 2017 01:53:47 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,274,1486454400"; d="scan'208";a="84402765" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.140]) by fmsmga006.fm.intel.com with SMTP; 04 Apr 2017 01:53:44 -0700 Received: by (sSMTP sendmail emulation); Tue, 04 Apr 2017 09:53:43 +0100 Date: Tue, 4 Apr 2017 09:53:42 +0100 From: Bruce Richardson To: Olivier MATZ Cc: Thomas Monjalon , dev@dpdk.org, "Ananyev, Konstantin" , "mb@smartsharesystems.com" , "Chilikin, Andrey" , "jblunck@infradead.org" , "nelio.laranjeiro@6wind.com" , "arybchenko@solarflare.com" Message-ID: <20170404085342.GA9080@bricha3-MOBL3.ger.corp.intel.com> References: <1488966121-22853-1-git-send-email-olivier.matz@6wind.com> <20170331105925.135c7377@platinum> <2601191342CEEE43887BDE71AB9772583FAE30C3@IRSMSX109.ger.corp.intel.com> <2731871.jrfrpyQStK@xps13> <20170404095849.24fbad19@neon> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170404095849.24fbad19@neon> 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: Tue, 04 Apr 2017 08:53:49 -0000 On Tue, Apr 04, 2017 at 09:58:49AM +0200, Olivier MATZ wrote: > On Mon, 03 Apr 2017 18:15:25 +0200 > Thomas Monjalon wrote: > > > 2017-03-31 09:18, Ananyev, Konstantin: > > > > On Fri, 31 Mar 2017 09:41:39 +0100, Bruce Richardson > > > > wrote: > > > > > On Fri, Mar 31, 2017 at 10:26:10AM +0200, Olivier Matz wrote: > > > > > > I replayed my tests, and I can also see a performance loss > > > > > > with 1c/1t (ixgbe), not in the same magnitude however. Here > > > > > > is what I have in MPPS: > > > > > > > > > > > > 1c/1t 1c/2t > > > > > > 53.3 58.7 current > > > > > > 52.1 58.8 original patchset > > > > > > 53.3 58.8 removed patches 3 and 9 > > > > > > 53.1 58.7 with konstantin's patch > > > > > > > > > > > > So we have 2 options here: > > > > > > > > > > > > 1/ integrate Konstantin's patch in the patchset (thank you, > > > > > > by the way) 2/ remove patch 3, and keep it for later until we > > > > > > have something that really no impact > > > > > > > > > > > > I'd prefer 1/, knowing that the difference is really small in > > > > > > terms of cycles per packet. > > > > > > > > > > > > > > > > > 1 is certainly the more attractive option. However, I think we > > > > > can afford to spend a little more time looking at this before > > > > > we decide. I'll try and check out the perf numbers I get with > > > > > i40e with Konstantin's patch today. We also need to double > > > > > check the other possible issues he reported in his other > > > > > emails. While I don't want this patchset held up for a long > > > > > time, I think an extra 24/48 hours is probably needed on it. > > > > > > > > > > > > > Yes, now that we have the "test momentum", try not to loose it ;) > > > > > > > > I'm guilty to have missed the performance loss, but honnestly, > > > > I'm a bit sad that nobody tried to this patchset before (it > > > > is available for more than 2 months), knowing this is probably > > > > one of the most critical part of dpdk. I think we need to be > > > > better next time. > > > > > > > > Anyway, thank you for your test and feedback now. > > > > > > I am also leaning towards option 1, but agree that some extra > > > testing first need to be done before making the final decision. > > > BTW, path #9 need to be removed anyway, even if will go for path #1. > > > Konstantin > > > > Please, can we have a conclusion now? > > I think we sholuld go with proposition 1, I can resubmit an updated > patch today. > > This rework is needed at least for metrics libraries. > > To summarize the perf data we have: > - There is a small impact on Intel NICs (-0.4MPPS on ixgbe in iofwd > mode according to Konstantin's test, which is less than 1%). I guess > it can be optimized. > - On mlx5, there is a gain (+0.8MPPS). > - On sfc, there is also a gain. > > Any comment? > > Olivier Hi, As you have probably seen from the patches I sent yesterday, there are optimizations we can make to our i40e (and ixgbe) drivers on top of this patchset which should compensate for any performance loss due to the mbuf rework. Therefore, we are ok to have this merged, so long as our PMD enhancements based on this set can also be merged (they are not large, so I assume this should not be controvertial). The i40e patches are on the list; an equivalent set for ixgbe should be submitted by Konstantin shortly. Regards, /Bruce