From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 2DB8A5F11; Wed, 6 Feb 2019 15:17:49 +0100 (CET) X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Feb 2019 06:17:48 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,340,1544515200"; d="scan'208";a="114148095" Received: from bricha3-mobl.ger.corp.intel.com ([10.237.221.54]) by orsmga006.jf.intel.com with SMTP; 06 Feb 2019 06:17:46 -0800 Received: by (sSMTP sendmail emulation); Wed, 06 Feb 2019 14:17:45 +0000 Date: Wed, 6 Feb 2019 14:17:45 +0000 From: Bruce Richardson To: Neil Horman Cc: dev@dpdk.org, stable@dpdk.org, David Marchand , Anatoly Burakov Message-ID: <20190206141744.GA236864@bricha3-MOBL.ger.corp.intel.com> References: <20190110111104.56464-1-bruce.richardson@intel.com> <20190206110130.55135-1-bruce.richardson@intel.com> <20190206122254.GA16887@hmswarspite.think-freely.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190206122254.GA16887@hmswarspite.think-freely.org> User-Agent: Mutt/1.11.2 (2019-01-07) Subject: Re: [dpdk-stable] [PATCH v3] compat: merge compat library into EAL X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Feb 2019 14:17:50 -0000 On Wed, Feb 06, 2019 at 07:22:54AM -0500, Neil Horman wrote: > On Wed, Feb 06, 2019 at 11:01:30AM +0000, Bruce Richardson wrote: > > Since compat library is only a single header, we can easily move it into > > the EAL common headers instead of tracking it separately. The downside of > > this is that it becomes a little more difficult to have any libs that are > > built before EAL depend on it. Thankfully, this is not a major problem as > > the only library which uses rte_compat.h and is built before EAL (kvargs) > > already has the path to the compat.h header file explicitly called out as > > an include path. > > > > However, to ensure that we don't hit problems later with this, we can add > > EAL common headers folder to the global include list in the meson build > > which means that all common headers can be safely used by all libraries, no > > matter what their build order. > > > This assumes that the compat lib will always just be a header though, no? Will > this work in the event that someone wants to add some compatibility code that > requires its own C compilation unit? > No, it probably won't work, you'll hit an issue with any libraries that don't depend on EAL and need that functionality. The question is whether this is likely to be an issue in the future for us. I'd say the possiblity is fairly remote, but I'm open to input on it. > > As a side-effect, this patch also fixes an issue with building on BSD using > > meson, due to compat lib no longer needing to be listed as a dependency. > > > Can you elaborate here a bit please? listing a lib as a dependency seems like a > fundamental function of a build system, was there a bug with meson in this > capacity? > It was a bug in DPDK. There was already a dependency on the compat library from libeal from linux, but not from BSD, so when a further dependency was added globally, the BSD build broke, but the linux one didn't. /Bruce