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 E9128A0093; Mon, 7 Mar 2022 17:54:55 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7AC0140688; Mon, 7 Mar 2022 17:54:55 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id 52F2D4014E for ; Mon, 7 Mar 2022 17:54:53 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1646672094; x=1678208094; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=wjZDTBI4MemJFoYTZabogW0syZbkeh6d7r1ExphnCZk=; b=C9C0xrHERuMNLy13XDs7MyiAREkCVJQ9ZqkKbjpW2YZ3vnA7vjDgdCSC M8lbvwYr7uHt5dJMJuPOpAJWBgBClUoJup2dbALeTCuTZQiH52bIzmG8L IkUNQpK50ezBvrpcgJBn8G/441ySepW4Gw5nXhINkoT6bITmqXFO1jdzc c5pU5hi7dHtirPtKKMun7fWv3ugSkNEHjsvlNVVgIFw8LDPt9qePNITlc qrzEiY9RDvCQxGjX9/XCkhcMLF6HNQJYzYA9JTdN3cWNys9YxYcvwsifN f5mchy5Es/RE/lrBP4qtfodbqmMFjCJSopLTpG7+u0BdNztRdUJffk49P w==; X-IronPort-AV: E=McAfee;i="6200,9189,10279"; a="279149754" X-IronPort-AV: E=Sophos;i="5.90,162,1643702400"; d="scan'208";a="279149754" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2022 08:54:52 -0800 X-IronPort-AV: E=Sophos;i="5.90,162,1643702400"; d="scan'208";a="509770288" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.28.136]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 07 Mar 2022 08:54:49 -0800 Date: Mon, 7 Mar 2022 16:52:09 +0000 From: Bruce Richardson To: Thomas Monjalon Cc: Karl Bonde Torp , dev@dpdk.org Subject: Re: [PATCH] build: fix build on FreeBSD with Meson 0.61.1 Message-ID: References: <20220214114313.21258-1-k.torp@samsung.com> <3845302.yIU609i1g2@thomas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3845302.yIU609i1g2@thomas> 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 On Mon, Mar 07, 2022 at 05:43:51PM +0100, Thomas Monjalon wrote: > 15/02/2022 16:34, Bruce Richardson: > > On Tue, Feb 15, 2022 at 02:50:48PM +0100, Karl Bonde Torp wrote: > > > On Tue, 15 Feb 2022 at 12:27, Bruce Richardson > > > wrote: > > > > > > > > On Tue, Feb 15, 2022 at 12:07:55PM +0100, Karl Bonde Torp wrote: > > > > > On Mon, 14 Feb 2022 at 14:48, Bruce Richardson > > > > > wrote: > > > > > > > > > > > > On Mon, Feb 14, 2022 at 12:43:13PM +0100, Karl Bonde Torp wrote: > > > > > > > When building with Meson 0.61.1 on FreeBSD some archives become > > > > > > > corrupted. This can be avoided by using 'llvm-ar' instead of 'ar'. > > > > > > > > > > > > > > Signed-off-by: Karl Bonde Torp > > > > > > > --- > > > > > > > > > > > > Can you perhaps provide some instructions or more details as to > > > > > > how/when you see this issue? > > > > > > > > > > Some background info: > > > > > I'm working on xNVMe and that depends on SPDK which in turn depends > > > > > on DPDK. These are includes as subprojects on github which is why the > > > > > FreeBSD port of DPDK is not an option for me. > > > > > > > > > > The steps I took to reproduce the issue are listed below, this assumes > > > > > a clean img of FreeBSD 13: > > > > > (I know some of them may be obvious, but I tried to avoid ambiguity) > > > > > > > > > > * install git and the pkg manager: > > > > > pkg install git > > > > > * get kernel source: > > > > > git clone https://git.freebsd.org/src.git /usr/src > > > > > * get ports: > > > > > git clone https://git.freebsd.org/ports.git /usr/ports > > > > > * install dependencies (except meson) with pkg: > > > > > pkg install py38-pyelftools ninja pkgconf binutils > > > > > * build meson 0.61.1 from port: > > > > > cd /usr/ports/devel/meson/ && make install clean > > > > > * clone dpdk repo: > > > > > git clone https://github.com/DPDK/dpdk.git > > > > > * inside dpdk folder run: > > > > > meson builddir && meson compile -C builddir > > > > > * encounter error: > > > > > [466/1915] Generating drivers/rte_common_cpt.pmd.c with a custom > > > > > command > > > > > FAILED: drivers/rte_common_cpt.pmd.c > > > > > /usr/local/bin/python3.8 ../buildtools/gen-pmdinfo-cfile.py > > > > > /root/dpdk/builddir/buildtools ar > > > > > /root/dpdk/builddir/drivers/libtmp_rte_common_cpt.a > > > > > drivers/rte_common_cpt.pmd.c /usr/local/bin/python3.8 > > > > > ../buildtools/pmdinfogen.py elf > > > > > ar: fatal: Unrecognized archive format: Inappropriate file type or > > > > > format > > > > > Traceback (most recent call last): > > > > > File "../buildtools/gen-pmdinfo-cfile.py", line 13, in > > > > > for name in subprocess.run([ar, "t", archive], > > > > > stdout=subprocess.PIPE, > > > > > File "/usr/local/lib/python3.8/subprocess.py", line 516, in run > > > > > raise CalledProcessError(retcode, process.args, > > > > > subprocess.CalledProcessError: Command '['ar', 't', > > > > > '/root/dpdk/builddir/drivers/libtmp_rte_common_cpt.a']' returned > > > > > non-zero exit status 70. > > > > > > > > > > > A clean build of mine on FreeBSD with meson 0.61.1 didn't seem to > > > > > > flag up any problems and I'd like to reproduce the issue to verify the fix. > > > > > I'm guessing you're refering to the port of DPDK here. > > > > > If not I am very interested to hear how you go about this. > > > > > > > > > > > > > I tested with latest DPDK on a FreeBSD machine also, not using the BSD > > > > ports version. Unfortunately it wasn't a clean install, but still a fairly > > > > new one. I updated the BSD pkg config to use "latest" rather than > > > > "quarterly" snapshot and then ran an "pkg upgrade". This gave me meson > > > > 0.61.1, and updated other packages too. Build of DPDK then ran cleanly. > > > > > > > > I wonder if the version of binutils or some other package might be the > > > > difference. The version I have is "GNU ar (GNU Binutils) 2.37". What > > > > version of binutils is installed on your system? > > > My version of binutils is 2.37_2,1, so that is the same. > > > However, I think you're onto something here. When I run "ar --version" I > > > get "BSD ar 1.1.0 - libarchive 3.5.1". > > > The problem my patch aimed to solve was 'BSD ar' behaving differently > > > than 'GNU ar'. So if you somehow got the 'ar' command to invoke 'GNU > > > ar' on your system, maybe I just need to do the same. > > > > Good point, I had missed that. BSD ar is in /usr/bin and GNU ar is > > installed by binutils in /usr/local/bin/ar. > > I'm confused. > Should we apply this patch as is? > A better version would select llvm-ar or binutils ar based off the compiler on FreeBSD, but I think this patch is ok to apply as-is, as llvm-ar should be able to work fine with the files created by binutils ar even if gcc is the compiler. Really, the main gap in all this is that meson does not provide a function to return the static linker in use (though it has one for the dynamic linker). /Bruce