From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 1/3] port: added WRITER_APPROACH == 1 implementation to ring port Date: Wed, 20 May 2015 01:19:05 +0200 Message-ID: <5551241.fJIiJj5JWx@xps13> References: <1430395084-6172-1-git-send-email-michalx.k.jastrzebski@intel.com> <22373976.yBms5me9Vj@xps13> <3EB4FA525960D640B5BDFFD6A3D891263236A14B@IRSMSX108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Dumitrescu, Cristian" Return-path: Received: from mail-wi0-f174.google.com (mail-wi0-f174.google.com [209.85.212.174]) by dpdk.org (Postfix) with ESMTP id E7C625952 for ; Wed, 20 May 2015 01:19:50 +0200 (CEST) Received: by wichy4 with SMTP id hy4so40360878wic.1 for ; Tue, 19 May 2015 16:19:50 -0700 (PDT) In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891263236A14B@IRSMSX108.ger.corp.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Cristian, 2015-05-19 22:49, Dumitrescu, Cristian: > HI Thomas and David, > > We can remove one of the code branches if you guys feel strongly about it. We don't feel strongly as we are not authors nor testers of this new code. But we want to avoid the maintenance nightmare of #ifdef. > The reason we recommended to keep both is because both of them are working, > and we wanted to keep both of them for a while to get some feedback from > other people about which one they prefer. It might be that some apps would > prefer one over the other for performance reasons. But again, we can resend > this patch with only one code path present. In general, RFC patches are used to request feedbacks. I think it's better to have only one implementation at a time. Comments are welcome. > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon > > Sent: Sunday, May 17, 2015 10:45 PM > > To: Jastrzebski, MichalX K; Gajdzica, MaciejX T > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v2 1/3] port: added WRITER_APPROACH == 1 > > implementation to ring port > > > > 2015-04-30 13:58, Michal Jastrzebski: > > > From: Maciej Gajdzica > > > > > > Added better optimized implementation of tx_bulk for ring writer port > > > based on > > > similar solution in ethdev_writer port. New implementation sends burst > > > without > > > copying data to internal buffer if it is possible. > > > > > > Signed-off-by: Maciej Gajdzica > > [...] > > > +#if RTE_PORT_RING_WRITER_APPROACH == 0 > > > > Nack > > Maybe you missed the previous comment: > > http://dpdk.org/ml/archives/dev/2015-March/015999.html