From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com [209.85.212.176]) by dpdk.org (Postfix) with ESMTP id 74873924B for ; Wed, 21 Oct 2015 10:15:23 +0200 (CEST) Received: by wikq8 with SMTP id q8so81337088wik.1 for ; Wed, 21 Oct 2015 01:15:23 -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=DYiyCSHUVtm8BkQQCbZQ4eXeqxL8vjSArlwKv7P+4TI=; b=P4qWElcWBvNgB2Qe1FGX98Fl+3cq3cyybXTTPIMzZXnQ3gKs6+3FyTCvVjvtib2q2X F8z52VWRVGbads3ZZwGMA4Q1tqWyMFhasWtltNCB/oq7wO0iTG6mTRxFkEpK6DYg99Sj r0Gr44HvQbc0BOLqBJJzRb54fFCxEi/lKWufZACN57PLCC+pqMyuTcw+qdPR7ZGNb5ng ZXJhJgn7Vmtv/mf2PJy0xMczsEdFGWnfdnzjoM9IGIAjVP6YlkV9c03DagPeDjOawSlC 6pv9zklNJtbsAkt/bHKKBLwk7Fk0STjXBG5HofyN5KQMxCmmh7t1u7dXc1u+9Cwjq5Tz HaGA== X-Gm-Message-State: ALoCoQlMokSvpIq0GhVs1R/5jmcYPnT3E0QJ50v+dlYu/iDeSqnBnAdQGyDUYEAhyqX3BS/UCdXx X-Received: by 10.180.221.227 with SMTP id qh3mr10119839wic.71.1445415323221; Wed, 21 Oct 2015 01:15:23 -0700 (PDT) Received: from xps13.localnet (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id y17sm6223941wib.15.2015.10.21.01.15.22 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 21 Oct 2015 01:15:22 -0700 (PDT) From: Thomas Monjalon To: Panu Matilainen Date: Wed, 21 Oct 2015 10:14:18 +0200 Message-ID: <1963689.ejoAfUUbUG@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <5620FDDE.3090100@redhat.com> References: <5620F824.4000707@redhat.com> <5620FDDE.3090100@redhat.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 4/5] eal: add an error code to plugin init for the next step 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: Wed, 21 Oct 2015 08:15:23 -0000 2015-10-16 16:38, Panu Matilainen: > On 10/16/2015 04:14 PM, Panu Matilainen wrote: > > On 10/16/2015 03:59 PM, Bruce Richardson wrote: > >> On Fri, Oct 16, 2015 at 02:58:16PM +0300, Panu Matilainen wrote: > >>> Signed-off-by: Panu Matilainen > >>> --- > >>> lib/librte_eal/bsdapp/eal/eal.c | 3 ++- > >>> lib/librte_eal/common/eal_common_options.c | 3 ++- > >>> lib/librte_eal/common/eal_options.h | 2 +- > >>> lib/librte_eal/linuxapp/eal/eal.c | 3 ++- > >>> 4 files changed, 7 insertions(+), 4 deletions(-) > >> > >> Again, another minor nit, but couldn't this be done when refactoring > >> in previous > >> patches, rather than needed a whole separate commit ? > > > > Of course it'd be possible to do this earlier, I pondered about it too > > but then went with this because > > a) otherwise I would've had to rework the earlier patches again > > b) not knowing which way people prefer it, I might've had to rework it > > back to the original > > c) didn't know we were saving commits > > d) doing it like this maintains a certain symmetry to how stuff is > > introduced > > In other words: I spent many years working with a codebase where the > policy was to never change code while moving it around otherwise. So > yeah, matter of policy, taste and all, I'm clearly still learning where > the fine line is in case of dpdk :) > > The series can easily be shrunken into two logical steps if that's > preferred: > 1) move the plugin handling code to common > 2) add the plugin directory support Yes, looks good. Thanks