From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8B66DA0C3F; Thu, 15 Apr 2021 15:52:01 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0695D162287; Thu, 15 Apr 2021 15:52:01 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id 9292F162252 for ; Thu, 15 Apr 2021 15:51:58 +0200 (CEST) IronPort-SDR: uDVUkKJC1bT6VZNPyMtSW0KAw5h5OmolS8zkh8ceQ0b4iCgAjOQroY1q8nBU0Pfo7wT/26NYd7 3JJed0TXGPew== X-IronPort-AV: E=McAfee;i="6200,9189,9955"; a="182352043" X-IronPort-AV: E=Sophos;i="5.82,225,1613462400"; d="scan'208";a="182352043" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Apr 2021 06:51:53 -0700 IronPort-SDR: Yasryrbfk/5LLUVvl9SKkT9yPD/Sgw70o9+UcnWopGQ8I+vzGIweBEEdTV61l0b0B5SxaDc1+3 /d+UK6WhXqvA== X-IronPort-AV: E=Sophos;i="5.82,225,1613462400"; d="scan'208";a="452922270" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.6.228]) by fmsmga002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 15 Apr 2021 06:51:51 -0700 Date: Thu, 15 Apr 2021 14:51:48 +0100 From: Bruce Richardson To: Juraj =?utf-8?Q?Linke=C5=A1?= Cc: "thomas@monjalon.net" , "david.marchand@redhat.com" , "Honnappa.Nagarahalli@arm.com" , "dev@dpdk.org" Message-ID: References: <1617015818-13240-1-git-send-email-juraj.linkes@pantheon.tech> <1617022234-13618-1-git-send-email-juraj.linkes@pantheon.tech> <09b4324c8bd845e8a8ee221f10afb816@pantheon.tech> <20210331123927.GB249@bricha3-MOBL.ger.corp.intel.com> <99700db0dcbf42d592771689de61aa9f@pantheon.tech> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <99700db0dcbf42d592771689de61aa9f@pantheon.tech> Subject: Re: [dpdk-dev] [PATCH v4] build: add platform meson option X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Apr 15, 2021 at 01:32:08PM +0000, Juraj Linkeš wrote: > > > > -----Original Message----- > > From: Bruce Richardson > > Sent: Wednesday, March 31, 2021 2:39 PM > > To: Juraj Linkeš > > Cc: thomas@monjalon.net; david.marchand@redhat.com; > > Honnappa.Nagarahalli@arm.com; dev@dpdk.org > > Subject: Re: [PATCH v4] build: add platform meson option > > > > On Wed, Mar 31, 2021 at 12:16:59PM +0000, Juraj Linkeš wrote: > > > Bruce, what do you think of the patch now? Do we need to add/change > > anything else, like documentation? > > > > > > One thing to note is that we're changing the default behavior in this patch > > from machine=native to machine=generic (or more accurately, to > > cpu_instruction_set=generic). Do we want to do that? > > > > > The patch in general looks ok, but I am uncertain about this change indeed. > > Especially since the -march flag we mirror to the pkg-config file. I'd like to see > > something like [1] included along with such a change. It allows us per-arch to > > select the flags to send to the pkg-config file, rather than just always using > > machine_args blindly. > > > > Feedback appreciated. > > > > I'm not sure what feedback do you want here. I'm not that familiar with pkg-config files. I think this is about DPDK being built a set of instructions and an app using DPDK being able to be built another set of instructions and the pkg-config file bridges these? Could you expand a bit? I read the other commit msg (in [1]) as well and I don't know how it's related to this patch. > The main thing I suppose I'm concerned about is that I'd like more input from people on the change from "native" to "generic" build by default. On top of that, the patch I linked to adjusts things a little further allowing separation of the ISA level used for building DPDK and for building the app using DPDK. For example, for x86, it would probably be more respectful for applications to have "-msse4" in the pkg-config file, rather than -march=native or -march=corei7. The fact that DPDK was compiled for a particular micro-arch should not force the user to use the same micro-arch. Instead, each architecture should be able to output a separate value that is suitable - in x86 case, turning on SSE4.2 as the minimum needed ISA level. However, I'm prepared to accept that discussion of the latter change can be dealt with separately from this patch. So long as others have no objection to change in default machine flags, I'm ok with it too. /Bruce