From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 5C81F28FD for ; Tue, 19 Apr 2016 18:25:42 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 19 Apr 2016 09:25:37 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,506,1455004800"; d="scan'208";a="962038304" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.132]) by fmsmga002.fm.intel.com with SMTP; 19 Apr 2016 09:25:24 -0700 Received: by (sSMTP sendmail emulation); Tue, 19 Apr 2016 17:25:23 +0025 Date: Tue, 19 Apr 2016 17:25:23 +0100 From: Bruce Richardson To: Thomas Monjalon Cc: dev@dpdk.org Message-ID: <20160419162523.GD15456@bricha3-MOBL3> References: <1461061640-7611-1-git-send-email-bruce.richardson@intel.com> <1699866.SDLcjre87A@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1699866.SDLcjre87A@xps13> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] scripts: add script for generating customised build config 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 Apr 2016 16:25:42 -0000 On Tue, Apr 19, 2016 at 02:30:08PM +0200, Thomas Monjalon wrote: > Hi Bruce, > > Thanks for pushing this idea. > > 2016-04-19 11:27, Bruce Richardson: > > This patch adds in the dpdk_config.py script file. It can be used > > to generate custom build-time configurations for DPDK either manually on > > the commandline or by calling it from other scripts. It takes as parameters > > the base config template to use, and output directory to which the modified > > configuration will be written. Other optional parameters are then taken > > as y/n values which should be adjusted in the config file, and a special > > -m flag is provided to override the default RTE_MACHINE setting in the > > config template too. > > > > Example, to create a build configuration with extra non-default PMDs > > enabled, and the kernel modules disabled: > > > > ./scripts/dpdk_config.py -b $RTE_TARGET -o $RTE_TARGET PMD_PCAP=y \ > > IGB_UIO=n KNI_KMOD=n MLX4_PMD=y MLX5_PMD=y SZEDATA2=y \ > > NFP_PMD=y BNX2X_PMD=y > > Would it be possible to use it without -b option to update a configuration? > Interesting idea, but what would that really give us over manual editing? If calling from a script, the user can just rebuild the config from scratch. > Why not name it scripts/configure.py with a symlink ./configure in the > top directory? No objections here. It's just not really a "normal" configure script, instead it's one designed to make it easy for me to generate directories with lots of different build settings in them. If we do want to make it to be "the" way to build DPDK, we can perhaps look at additional enhancements to speed it up by working with the defconfigs directly, as I'm not happy right now with the speed of the "make config" step. On the other hand, I'm very happy with having it short and of limited scope. Actual code only takes up about 50% of the file. :-) > > Should we be able to list every options for a "-b defconfig"? > Good idea, I think. Under what conditions? Only if an invalid config is provided, or as part of regular help text or otherwise? > Would it be a good idea to manage dependencies checks in this script? > I'd rather not do so here. a) I don't think there are a huge number of dependencies to manage b) I'd like to keep it from getting too long and complicated. For 95% of use cases, being able to set a few y/n values on and off, and change the machine type should suffice. I wasn't going for a 100% solution. /Bruce