From mboxrd@z Thu Jan 1 00:00:00 1970 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) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=bIov3NYQdi2Iwc4b5sPqf7HE/X0onFuvOZhsYQPZQk4=; b=iYqLp0ji9efMakVAxt1kUsyE8WvbycYIAD9jb1/BxTE2WqH7RKuazsFCH2tV6o3rg7 0ThMnw/rai+yFbJOOdZYMF7NQCNzl1MJJAxvhSjOo3g1KjQZQPBIEa4wXULkLVss2VPE NekfFyqllfwFd4IsLC3qVEJ5mDuL30p/ntDNN1FbNzyncF/804l2nQWdFNfRMKfEgoUH x2ulcn9dZztEiix1L0kslsetDXulsgvvwCitaYF7ym7u1+ZdKPyU5QuIcKO+rQXJ63OL dUTWR0nhPpFq8Ba/BxTgHRwmFHX3Z46PWsRQCLzU/pJsuTgfFkpCEC0SY7tLRtSelXjI pCjA== X-Gm-Message-State: ALoCoQl3sZj0DAy42LMS6LCkADPzCxmc5SVCXttpcUKAV9m/XSRfLd3yACSr1oXT/iG7HofoBrr2 X-Received: by 10.194.71.168 with SMTP id w8mr11909852wju.80.1432077590764; Tue, 19 May 2015 16:19:50 -0700 (PDT) Received: from xps13.localnet (2E6B6683.dsl.pool.telekom.hu. [46.107.102.131]) by mx.google.com with ESMTPSA id r9sm23876938wjo.26.2015.05.19.16.19.49 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 19 May 2015 16:19:49 -0700 (PDT) From: Thomas Monjalon To: "Dumitrescu, Cristian" Date: Wed, 20 May 2015 01:19:05 +0200 Message-ID: <5551241.fJIiJj5JWx@xps13> Organization: 6WIND User-Agent: KMail/4.14.7 (Linux/4.0.1-1-ARCH; KDE/4.14.7; x86_64; ; ) In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891263236A14B@IRSMSX108.ger.corp.intel.com> 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-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 1/3] port: added WRITER_APPROACH == 1 implementation to ring port X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 May 2015 23:19:51 -0000 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