From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 9D2683975 for ; Tue, 25 Nov 2014 15:55:49 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP; 25 Nov 2014 06:59:21 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="421292337" Received: from bricha3-mobl3.ger.corp.intel.com ([10.243.20.15]) by FMSMGA003.fm.intel.com with SMTP; 25 Nov 2014 06:56:39 -0800 Received: by (sSMTP sendmail emulation); Tue, 25 Nov 2014 15:06:24 +0025 Date: Tue, 25 Nov 2014 15:06:24 +0000 From: Bruce Richardson To: Thomas Monjalon Message-ID: <20141125150624.GC6800@bricha3-MOBL3> References: <1416692622-28886-1-git-send-email-thomas.monjalon@6wind.com> <4091968.6vrJ5rTTlz@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4091968.6vrJ5rTTlz@xps13> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 00/10] eal cleanup and new options 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, 25 Nov 2014 14:55:50 -0000 On Tue, Nov 25, 2014 at 03:55:19PM +0100, Thomas Monjalon wrote: > > There are some pending patches which requires to factorize some EAL parts > > in order to be correctly implemented. > > This patchset do the required clean-up and rework these patches to improve > > lcore handling: > > > > Didier Pallard (2): > > eal: add core list input format > > config: support 128 cores > > > > Patrick Lu (1): > > eal: get relative core index > > > > Simon Kuenzer (1): > > eal: add option --master-lcore > > > > Thomas Monjalon (6): > > eal: move internal headers in source directory > > eal: factorize common headers > > eal: fix header guards > > eal: factorize internal config reset > > eal: factorize options sanity check > > eal: factorize configuration adjustment > > Applied with last comments integrated. > > The conclusions to the vote about the -c/-l options are: > - people don't vote (don't care or don't read) > - a few votes give the majority to creating the -l option. > > Neil, I'm really not sure what is the best solution. Maybe that applying > this patch will make more voices raising. We'll see, all can be changed. > > -- My additional 2c here: While I don't see much wrong with having core lists passed using a long option, more and more cores are being added to the latest CPUs, so it's likely that DPDK users are going to have systems with >64 cores and will want to move from the coremask specifier to the core list option. In that case, having it as a short option makes a lot of sense. /Bruce