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 A1850A04BB; Wed, 2 Sep 2020 14:39:23 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7EF761C07D; Wed, 2 Sep 2020 14:39:23 +0200 (CEST) Received: from youngberry.canonical.com (youngberry.canonical.com [91.189.89.112]) by dpdk.org (Postfix) with ESMTP id 10D90255 for ; Wed, 2 Sep 2020 14:39:22 +0200 (CEST) Received: from mail-vk1-f200.google.com ([209.85.221.200]) by youngberry.canonical.com with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.86_2) (envelope-from ) id 1kDS2r-0006dk-NY for dev@dpdk.org; Wed, 02 Sep 2020 12:39:21 +0000 Received: by mail-vk1-f200.google.com with SMTP id a4so1293690vkl.3 for ; Wed, 02 Sep 2020 05:39:21 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=4E5M63oEDvrDN7hxEzT6i320LsNiqq5UMSJq/lXUMz4=; b=H1eoClExWIRV4ekMXcP6YLj5SsprqeQOMcKz7FWNlKvdKL/Mq4psa7OnM3u2YFBe6c 8FSfqg/3k8/eVQsPSWn5zAq6fLgIX2BoH6sRgCBzoZXxwal05YA+D8Xx9jcx2g7PPsfo xGjrqzIaq09mc9Pq4EqQGl5J7VS2QiHWlgA7NNwGLlJ94qKiRoSd7WFW3XcBXGuyZChr yHGHu3a8vLbwDQmgFUXPL9LmGLGQSJ40Y2PUpAJqA8yWDTtzkGxfLKnGiAI+7Gi7tH4W /mW++C5mUid4mdl48LmwGttoB/W/pbvpoGhnmqoWP/VkU7Xi248TWqy074wcj3zkUg+s LODQ== X-Gm-Message-State: AOAM531Ri6ki5bfwShiOEbRZAzWI/hwSYkJfoeoUD2J+J94B7a1ZB5T7 2c7ikOTy/KNIUYWyzm8+DCwCHw+anfxeO6Vs1PKkA50cv4y/2K9pnndGl8BwIWFTAdfk5e0pugf sWHxiiJqnQhKr+P0ETgTUHBjGJLRNxkU8ZauS X-Received: by 2002:a67:6892:: with SMTP id d140mr5117546vsc.80.1599050360752; Wed, 02 Sep 2020 05:39:20 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxB2PR+SdQysQKNJrXbIz2C/PTsJBsFBJ3u5StDnMFMYMFSlepxTbw8icjn3Xtq/N0d22QAWlBjcobbeiAMeWo= X-Received: by 2002:a67:6892:: with SMTP id d140mr5117534vsc.80.1599050360486; Wed, 02 Sep 2020 05:39:20 -0700 (PDT) MIME-Version: 1.0 References: <20200902111030.731593-1-christian.ehrhardt@canonical.com> <20200902112502.GD1064@bricha3-MOBL.ger.corp.intel.com> In-Reply-To: <20200902112502.GD1064@bricha3-MOBL.ger.corp.intel.com> From: Christian Ehrhardt Date: Wed, 2 Sep 2020 14:38:54 +0200 Message-ID: To: Bruce Richardson Cc: dev , Thomas Monjalon , Luca Boccassi Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH] avoid libfdt checks adding full paths to pkg-config 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" On Wed, Sep 2, 2020 at 1:25 PM Bruce Richardson wrote: > > On Wed, Sep 02, 2020 at 01:10:30PM +0200, Christian Ehrhardt wrote: > > The checks for libfdt try dependency() first which would only work if > > a pkg-config would be present but libfdt has none. > > Then it probes for the lib path itself via cc.find_library. > > > > But later it adds the result of either probe to ext_deps which ends up > > in build and also the resulting pkg-config to contain toolchain versioned > > paths in Libs.private like: > > /usr/lib/gcc/x86_64-linux-gnu/9/../../../x86_64-linux-gnu/libfdt.so > > which obviously breaks on toolchain updates. > > > > In general libs used multiple times - ipn3ke + ifpga in this case - are > > checked centrally in config/meson.build so move it there and fix the > > adding of dependencies to not use the full file path. > > > > The result is libfdt in pkg-config now showing up as: > > Libs.private: -pthread -lm -ldl -lnuma -lfdt -lpcap > > > > Signed-off-by: Christian Ehrhardt > > Reviewed-by: Luca Boccassi > > --- > > config/meson.build | 9 +++++++++ > > drivers/net/ipn3ke/meson.build | 6 +----- > > drivers/raw/ifpga/meson.build | 7 +------ > > 3 files changed, 11 insertions(+), 11 deletions(-) > > > > diff --git a/config/meson.build b/config/meson.build > > index cff8b33dd2..1c8317e750 100644 > > --- a/config/meson.build > > +++ b/config/meson.build > > @@ -150,6 +150,15 @@ if numa_dep.found() and cc.has_header('numaif.h') > > dpdk_extra_ldflags += '-lnuma' > > endif > > > > +has_libfdt = 0 > > +fdt_dep = cc.find_library('libfdt', required: false) > > +if fdt_dep.found() and cc.has_header('fdt.h') > > + dpdk_conf.set10('RTE_HAS_LIBFDT', true) > > + has_libfdt = 1 > > + add_project_link_arguments('-lfdt', language: 'c') > > + dpdk_extra_ldflags += '-lfdt' > > +endif > > + > > # check for libbsd > > libbsd = dependency('libbsd', required: false) > > if libbsd.found() > > diff --git a/drivers/net/ipn3ke/meson.build b/drivers/net/ipn3ke/meson.build > > index ec9cb7daf0..d5000d807c 100644 > > --- a/drivers/net/ipn3ke/meson.build > > +++ b/drivers/net/ipn3ke/meson.build > > @@ -9,11 +9,7 @@ > > # rte_eth_switch_domain_free() > > # > > > > -dep = dependency('libfdt', required: false) > > -if not dep.found() > > - dep = cc.find_library('libfdt', required: false) > > -endif > > -if not dep.found() > > +if has_libfdt == 0 > > Minor nit, for readability, I'd suggest using "not has_libfdt", but what is > here can work too. Thanks, improved according to your suggestion, v2 submission inbound. > > build = false > > reason = 'missing dependency, "libfdt"' > > subdir_done() > > diff --git a/drivers/raw/ifpga/meson.build b/drivers/raw/ifpga/meson.build > > index 05a1711b5d..da454b1942 100644 > > --- a/drivers/raw/ifpga/meson.build > > +++ b/drivers/raw/ifpga/meson.build > > @@ -1,11 +1,7 @@ > > # SPDX-License-Identifier: BSD-3-Clause > > # Copyright(c) 2018 Intel Corporation > > > > -dep = dependency('libfdt', required: false) > > -if not dep.found() > > - dep = cc.find_library('libfdt', required: false) > > -endif > > -if not dep.found() > > +if has_libfdt == 0 > > build = false > > reason = 'missing dependency, "libfdt"' > > subdir_done() > > @@ -16,7 +12,6 @@ objs = [base_objs] > > > > deps += ['ethdev', 'rawdev', 'pci', 'bus_pci', 'kvargs', > > 'bus_vdev', 'bus_ifpga', 'net', 'pmd_i40e', 'pmd_ipn3ke'] > > -ext_deps += dep > > > > sources = files('ifpga_rawdev.c') > > > > -- > Reviewed-by: Bruce Richardson -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd