From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH 4/5] eal: add an error code to plugin init for the next step Date: Fri, 16 Oct 2015 16:38:38 +0300 Message-ID: <5620FDDE.3090100@redhat.com> References: <42ab5ad11b473b964faaa2b0c622b92085d2545a.1444996480.git.pmatilai@redhat.com> <20151016125953.GC9980@bricha3-MOBL3> <5620F824.4000707@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Bruce Richardson Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 097DE8D99 for ; Fri, 16 Oct 2015 15:38:41 +0200 (CEST) In-Reply-To: <5620F824.4000707@redhat.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" 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 - Panu -