From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f44.google.com (mail-wm0-f44.google.com [74.125.82.44]) by dpdk.org (Postfix) with ESMTP id B8AD428F3 for ; Tue, 28 Mar 2017 17:41:03 +0200 (CEST) Received: by mail-wm0-f44.google.com with SMTP id t189so2119555wmt.1 for ; Tue, 28 Mar 2017 08:41:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:user-agent:in-reply-to :references:mime-version:content-transfer-encoding; bh=Xy/Nz5EjVl3zZYTw/tmyOI6zNFV3HNel3L7WQjEaUOo=; b=lM1HSx4QQK2fkDJQHjoXVRi9sf1An9CUqrubRMMTYbwSXu/toVk/KHvZgvVe1Dm11j mJrXWbiVjw7uqhoJCrGi6daDPcenfLheAPFo6sjl64hLD6pve/9idfYdenzT5zxowh83 GADf9vD9BmTo25FWj1SdKTCaVp/DeuA1VvKFwgFi7cKm67BfLOo4Mb3g08cgdoi4kVE5 tYDrmSZ6EKSR9YpnMMw8kU08hdruhwKqMWXVeqd2Dmw1M0O2NGik55ZyzBxjgeBdsXjN oRIRsA57jpq77q8mk+cgqaWoprWkwBdS3T+B7kb56IIDBgV5sjNSjeU5HTe5kUt8kV5k 38PQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=Xy/Nz5EjVl3zZYTw/tmyOI6zNFV3HNel3L7WQjEaUOo=; b=Xa/ybWEe0EjL/I+JG7eFnA9c5CWZGvgxoYHXwZukQH6lZ0Bi/pXRtj1nn9Xe04sit+ nvNuwJiE4nZtY449L8qcNERW13LJ/D6gi2MQBz6pWfofsRZwyYJeImGXMhalD6KjB22Y 13ld2tH86O1kQkz5uQJ0xln/+tpSVw/vXdbHidobwUUny0cZjhEd97ndYtJyCdqJhYW0 kXN/awKCuCDqfeAdToDp68mxwMz6dMkDtm8R1wPnXsL3sIiGiESYZoqmWPCxxz5wrymY OcD3WFAZgRjX7TjYBUE9io8hOSABAQO+Mijza1XbUgYvAF8y33UTK0iw8+RXZbxwiRbu 0H4A== X-Gm-Message-State: AFeK/H0dY2Xt9jl+xjQL92nZv7Kb5EdqTrf+CA3IeLr9rpNtydnWcYlJQpBycwf7SekBvouC X-Received: by 10.28.46.213 with SMTP id u204mr14637602wmu.136.1490715663460; Tue, 28 Mar 2017 08:41:03 -0700 (PDT) Received: from xps13.localnet (184.203.134.77.rev.sfr.net. [77.134.203.184]) by smtp.gmail.com with ESMTPSA id g5sm5310880wrb.48.2017.03.28.08.41.01 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 28 Mar 2017 08:41:02 -0700 (PDT) From: Thomas Monjalon To: Bruce Richardson Cc: "Dumitrescu, Cristian" , "Legacy, Allain (Wind River)" , dev@dpdk.org, "yuanhan.liu@linux.intel.com" , techboard@dpdk.org Date: Tue, 28 Mar 2017 17:41:01 +0200 Message-ID: <13936610.WgrjQNdGoS@xps13> User-Agent: KMail/4.14.10 (Linux/4.5.4-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <20170328152421.GA22176@bricha3-MOBL3.ger.corp.intel.com> References: <1488482971-170522-1-git-send-email-allain.legacy@windriver.com> <82821129.A9hgLN2u53@xps13> <20170328152421.GA22176@bricha3-MOBL3.ger.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v2 0/6] librte_cfgfile enhancements 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, 28 Mar 2017 15:41:04 -0000 2017-03-28 16:24, Bruce Richardson: > On Tue, Mar 28, 2017 at 12:12:26PM +0200, Thomas Monjalon wrote: > > 2017-03-28 09:58, Dumitrescu, Cristian: > > > > > As follow-up to my own mail, for this specific library example, I > > > > > wouldn't look to remove it from DPDK anyway. Parsing ini files is fairly > > > > > trivial, so I think it's not a big deal to keep our own version and not > > > > > have an external dependency - especially since it's already there and not > > > > > a big maintenance burden. > > > > > > > > Removing this lib would not disable anything as it is used only by examples. > > > > I don't see what would be the issue. > > > > We just have to download the lib when building the example app. > > > > It can be done quite easily in the makefile. > > > > > > Thomas, more than 3 quarters of DPDK libs are only used by applications, is this a reason to remove them? > > > > > > Also, I think the purpose of DPDK is to enable people to write applications, not more libraries. Would you agree? We should make the life easier for the application developers, not libraries. > > > > > > This library is an important utility for applications, similar to librte_cmdline and others. I think it is not fair from your side to refer to librte_cfgfile without any reference to librte_cmdline. > > > > I agree Cristian. > > I was just writing another email about removing librte_cmdline: > > http://dpdk.org/ml/archives/dev/2017-March/061777.html > > This thread was about librte_cfgfile. I hope you'll agree I am really fair :) > > > > It is really a scope question and should be managed by the techboard (CC). > > > Sure. > > As for my 2c right now on this lib, I'm very much in favour of keeping > it. I also think we should look to reuse it as an alternative way of > passing parameters to EAL. The existing method of using argc/argv makes > passing a lot of args, e.g for devices clunky, and re-using code from > cfgfile library gives us an alternative without adding extra > dependencies. I also think it could be useful for testpmd, which is > similarly "blessed" with lots of cmdline args to pass. I think we must improve the API and later deprecate argc/argv. The configuration must be done by the application. It should be the application choice to get its input from the command line, a CLI or a config file.