From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id E18EBA0613 for ; Wed, 28 Aug 2019 17:23:48 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id DE5691C123; Wed, 28 Aug 2019 17:23:47 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 2F4A01DBC for ; Wed, 28 Aug 2019 17:23:46 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 2784B308FB82; Wed, 28 Aug 2019 15:23:45 +0000 (UTC) Received: from dhcp-25.97.bos.redhat.com (unknown [10.18.25.8]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 8026D5D9E2; Wed, 28 Aug 2019 15:23:44 +0000 (UTC) From: Aaron Conole To: Christian Ehrhardt Cc: dev , Kevin Laatz , Luca Boccassi , Thomas Monjalon References: <20190828122752.27887-1-christian.ehrhardt@canonical.com> Date: Wed, 28 Aug 2019 11:23:43 -0400 In-Reply-To: (Aaron Conole's message of "Wed, 28 Aug 2019 11:14:44 -0400") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Wed, 28 Aug 2019 15:23:45 +0000 (UTC) Subject: Re: [dpdk-dev] [PATCH] build: avoid --as-needed as it causes overlinking X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" Aaron Conole writes: > Christian Ehrhardt writes: > >> On Wed, Aug 28, 2019 at 3:53 PM Aaron Conole wrote: >>> >>> Christian Ehrhardt writes: >>> >>> > A while ago telemetry was added in 57ae0ec6 and it also added as-needed >>> > to config/meson.build. This seems no more needed these days as due to other >>> > build changes the ordering in buildlogs is: >>> > [...] -lrte_telemetry [...] -Wl,--no-as-needed [...] >>> > Which means telemetry no more benefits from --no-as-needed anyway. >>> > >>> > Overlinking problems get triggered by the meson generated pkgconfig which >>> > will have: >>> > [...] -Wl,--no-as-needed >>> > This will overlink and in addition anything that follows >>> > as it also doesn't wrap back to --as-needed. So if a projects includes >>> > dpdk libs + it will also consider with --no-as-needed. >>> > >>> > Fixes: https://bugs.launchpad.net/ubuntu/+source/dpdk/+bug/1841759 >>> > >>> > Signed-off-by: Christian Ehrhardt >>> > --- >>> >>> Hi Christian, >>> >>> I agree this is something to be fixed. It will need additional work, >>> though: >>> >>> https://travis-ci.com/ovsrobot/dpdk/builds/124909245 >>> >> >> Thanks for the Link Aaron, yet I'm puzzled what to do there atm. >> >> The kind of error I found in the failing logs were misleading at first: >> - linker can't find -lvirt / -lpqos / ... >> well the test env needs to install them, maybe it was added as >> dependency by accident before? > > Not sure about this. It's strange to require that we *install* the > libraries before we can unit test them. After all, if I'm going to > potentially replace my previously installed libraries, I definitely want > to know that the unit tests are passing. > >> I'd understand (due to the change) if it would complain about missing symbols >> (no more added due to as-needed, but then for some reason needed) >> But this is vice versa, it just doesn't find the libs in the build env >> - error: unrecognized command line option '-Wformat-truncation' >> I don't see how I'd cause this ... >> => Maybe this is just an artifact that is even part of the normal/good tests? > > I don't think so - but there's a simple change. I've pushed to my own > branch and you can see the builds: > > https://travis-ci.org/orgcandman/dpdk/branches using the same > series_6154 branch name. > >> Comparing former logs - last good test was >> https://travis-ci.com/ovsrobot/dpdk/builds/124875383 >> This first seemed more helpful. >> >> DPDK:fast-tests / eal_flags_w_opt_autotest FAIL >> DPDK:fast-tests / func_reentrancy_autotest FAIL >> DPDK:fast-tests / mbuf_autotest FAIL >> DPDK:fast-tests / mempool_autotest FAIL >> DPDK:fast-tests / ring_pmd_autotest FAIL >> DPDK:fast-tests / sched_autotest FAIL >> DPDK:fast-tests / table_autotest FAIL >> [...] >> Overall about 14/60 of the tests failed with no recognizable pattern >> why just those and not the others. > > Good question :) > >> I only see "Full log written ... on_error", so I can't directly >> compare how a good run would look in the configure/build stage. >> Looking just at the bad case there are plenty of messages like >> - "no available hugepages" >> - "cannot reserve memory", .. >> But all those indicate more a flaky test(-env) than an error in the >> commit, there must be more to it. > > Okay. Fair enough. > >> @Aaron is there a good way to get the rest of the log for a good case >> to compare? > > Let's wait for https://travis-ci.org/orgcandman/dpdk/builds/577910388 to > spit out some details. Oops - forgot to push the revert: https://travis-ci.org/orgcandman/dpdk/builds/577918381 is the correct build. Sorry. >> Maybe I'm yet to blind for all the potential side effects of the change?