From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 51FE5C388 for ; Thu, 4 Jun 2015 16:55:05 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga102.fm.intel.com with ESMTP; 04 Jun 2015 07:55:04 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,553,1427785200"; d="scan'208";a="502740920" Received: from orsmsx107.amr.corp.intel.com ([10.22.240.5]) by FMSMGA003.fm.intel.com with ESMTP; 04 Jun 2015 07:55:04 -0700 Received: from orsmsx154.amr.corp.intel.com (10.22.226.12) by ORSMSX107.amr.corp.intel.com (10.22.240.5) with Microsoft SMTP Server (TLS) id 14.3.224.2; Thu, 4 Jun 2015 07:55:04 -0700 Received: from fmsmsx104.amr.corp.intel.com (10.18.124.202) by ORSMSX154.amr.corp.intel.com (10.22.226.12) with Microsoft SMTP Server (TLS) id 14.3.224.2; Thu, 4 Jun 2015 07:55:03 -0700 Received: from fmsmsx113.amr.corp.intel.com ([169.254.13.213]) by fmsmsx104.amr.corp.intel.com ([169.254.3.159]) with mapi id 14.03.0224.002; Thu, 4 Jun 2015 07:55:02 -0700 From: "Wiles, Keith" To: David Marchand Thread-Topic: [dpdk-dev] [RFC PATCH] eal:Add new API for parsing args at rte_eal_init time Thread-Index: AQHQnltA/wkwK4V9H0iqXqFk9uBgcZ2cXhyAgAB20AD//7UYAIAAWCQA//+vhAA= Date: Thu, 4 Jun 2015 14:55:02 +0000 Message-ID: References: <1433357393-54434-1-git-send-email-keith.wiles@intel.com> <20150603171255.545e0df8@urahara> <20150604135542.GC24585@hmsreliant.think-freely.org> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.252.134.161] Content-Type: text/plain; charset="us-ascii" Content-ID: <88EAC35488A979459D27DF0542EA650D@intel.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [RFC PATCH] eal:Add new API for parsing args at rte_eal_init time 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: Thu, 04 Jun 2015 14:55:05 -0000 Hmmm, replied in HTML. >On Thu, Jun 4, 2015 at 4:27 PM, Wiles, Keith > wrote: > >Hi Neil and Stephen, > >I agree this is not saving instructions and adding performance, but of >code clutter and providing a layered model for the developer. The >rte_eal_init() routine still exists and I was not trying to remove that >API only layer a convenient API for common constructs. >> >>Its not a bad addition, I'm just not sure its worth having to take on the >>additional API surface to include. I'd be more supportive if you could >>enhance >>the function to allow the previously mentioned before/after flexibiilty. >>Then >>we could just deprecate rte_eal_init as an API call entirely, and use >>this >>instead. > >I can see we can create an API to add support for doing the applications >args first or after, but would that even be acceptable? > > > >What's the point ? >Adding stuff just for saving lines ? >Are you serious about this ? Wow, OK it is dropped. > > >--=20 > >David Marchand