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 1516245903; Wed, 4 Sep 2024 17:08:45 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 790B24025A; Wed, 4 Sep 2024 17:08:44 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) by mails.dpdk.org (Postfix) with ESMTP id AF0904014F for ; Wed, 4 Sep 2024 17:08:42 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1725462523; x=1756998523; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding; bh=jYziNpVD5h2ZcgP2Fnc4LUz74qyCeGbji0J25Oxo4a8=; b=jqd1gwZCzq/vYAGgMQNqgBS9yyCSJm9CeiiD+LWh1oV2CNtZjbVWWbZy tFnu+BRln/UW2lH5Gw7FSFJQvXwqbGAjJU1ngDo67k4RQopcnyjCUDwgG gKF1+rBMRau9Wy1+KZC4YPHsLoPZdV8ztq9+t7ONIMXDkblwOxVfPYG1Q jONIal3y1gxefbG/tsnAX3EVWl4c3H73Gn7hQPOQpc7YH6GDL04Zfbpgy hA6XpSfICBX2c64iVuVi2f86hAVOjSBO/BWRBFBN8KRrlX63ryXSsxJYd KtjZejV7/V6HYTdPWhyeiVyqR6LWiS1dppvFPisJbKTTUIADqF00Ens85 w==; X-CSE-ConnectionGUID: fHnItTtVQUq8AjJscjGAbQ== X-CSE-MsgGUID: fe5ITjSET2q+rK2PQBm5VQ== X-IronPort-AV: E=McAfee;i="6700,10204,11185"; a="23641056" X-IronPort-AV: E=Sophos;i="6.10,202,1719903600"; d="scan'208";a="23641056" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Sep 2024 08:08:42 -0700 X-CSE-ConnectionGUID: D6YMNsx6Q+Kmg5xEFvQCYg== X-CSE-MsgGUID: AT3vN8GIRyKmQkle+2Dx3w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.10,202,1719903600"; d="scan'208";a="96107820" Received: from silpixa00401119.ir.intel.com ([10.55.129.167]) by fmviesa001.fm.intel.com with ESMTP; 04 Sep 2024 08:08:41 -0700 From: Anatoly Burakov To: dev@dpdk.org Subject: [PATCH v3 0/8] record and rework component dependencies Date: Wed, 4 Sep 2024 16:08:28 +0100 Message-ID: X-Mailer: git-send-email 2.43.5 In-Reply-To: <20240730145508.551075-1-bruce.richardson@intel.com> References: <20240730145508.551075-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 As part of the meson build, we can record the dependencies for each component as we process it, logging them to a file. This file can be used as input to a number of other scripts and tools, for example, to graph the dependencies, or to allow higher-level build-config tools to automatically enable component requirements, etc. The first patch of this set separates dependencies inside meson into optional or mandatory. The second patch of this set generates the basic dependency tree. The third patch does some processing of that dependency tree to identify cases where dependencies are being unnecessarily specified. Reducing these makes it easier to have readable dependency graphs in future, without affecting the build. The following 4 patches are based on the output of the second patch, and greatly cut down the number of direct dependency links between components. Even with the cut-down dependencies, the full dependency graph is nigh-unreadable, so the final patch adds a new script to generate dependency tree subgraphs, creating dot files for e.g. the dependencies of a particular component, or a component class such as mempool drivers. v2 -> v3: - Split dependencies into optional and mandatory - Fixup graph scripts to read and generate graphs that encode optional dependencies into the graph - Python version fixes to avoid using features not available in minimum supported Python version - Formatting with Ruff, and PEP-484 compliance Anatoly Burakov (1): build: split dependencies into mandatory and optional Bruce Richardson (7): build: output a dependency log in build directory devtools: add script to flag unneeded dependencies build: remove kvargs from driver class dependencies build: reduce library dependencies build: reduce driver dependencies build: reduce app dependencies devtools: add script to generate DPDK dependency graphs app/dumpcap/meson.build | 2 +- app/graph/meson.build | 2 +- app/meson.build | 11 +- app/pdump/meson.build | 2 +- app/proc-info/meson.build | 4 +- app/test-bbdev/meson.build | 8 +- app/test-crypto-perf/meson.build | 4 +- app/test-fib/meson.build | 2 +- app/test-pmd/meson.build | 26 +-- app/test-sad/meson.build | 2 +- app/test/meson.build | 14 +- buildtools/log-deps.py | 81 ++++++++ buildtools/meson.build | 2 + devtools/draw-dependency-graphs.py | 223 +++++++++++++++++++++ devtools/find-duplicate-deps.py | 53 +++++ drivers/baseband/fpga_5gnr_fec/meson.build | 2 +- drivers/baseband/fpga_lte_fec/meson.build | 2 +- drivers/baseband/la12xx/meson.build | 2 +- drivers/baseband/null/meson.build | 2 +- drivers/baseband/turbo_sw/meson.build | 2 +- drivers/bus/auxiliary/meson.build | 2 - drivers/bus/dpaa/meson.build | 2 +- drivers/bus/fslmc/meson.build | 2 +- drivers/bus/ifpga/meson.build | 2 +- drivers/bus/pci/meson.build | 4 +- drivers/bus/platform/meson.build | 1 - drivers/bus/uacce/meson.build | 2 - drivers/bus/vdev/meson.build | 2 - drivers/common/cnxk/meson.build | 4 +- drivers/common/cpt/meson.build | 2 +- drivers/common/idpf/meson.build | 2 +- drivers/common/mlx5/meson.build | 2 +- drivers/compress/mlx5/meson.build | 2 +- drivers/compress/nitrox/meson.build | 2 +- drivers/compress/octeontx/meson.build | 2 +- drivers/crypto/bcmfs/meson.build | 2 +- drivers/crypto/cnxk/meson.build | 2 +- drivers/crypto/dpaa_sec/meson.build | 2 +- drivers/crypto/ipsec_mb/meson.build | 2 +- drivers/crypto/mlx5/meson.build | 2 +- drivers/crypto/nitrox/meson.build | 2 +- drivers/dma/cnxk/meson.build | 2 +- drivers/dma/dpaa/meson.build | 2 +- drivers/dma/dpaa2/meson.build | 2 +- drivers/dma/odm/meson.build | 2 +- drivers/dma/skeleton/meson.build | 2 +- drivers/event/cnxk/meson.build | 2 +- drivers/event/dlb2/meson.build | 2 +- drivers/event/dpaa2/meson.build | 2 +- drivers/event/meson.build | 2 +- drivers/event/octeontx/meson.build | 3 +- drivers/event/sw/meson.build | 2 +- drivers/mempool/cnxk/meson.build | 2 +- drivers/mempool/dpaa/meson.build | 2 +- drivers/mempool/dpaa2/meson.build | 2 +- drivers/mempool/octeontx/meson.build | 2 +- drivers/meson.build | 9 +- drivers/net/cnxk/meson.build | 3 +- drivers/net/iavf/meson.build | 2 +- drivers/net/ice/meson.build | 2 +- drivers/net/mana/meson.build | 2 +- drivers/net/meson.build | 2 +- drivers/net/mlx5/meson.build | 2 +- drivers/net/sfc/meson.build | 2 +- drivers/net/softnic/meson.build | 2 +- drivers/raw/cnxk_bphy/meson.build | 2 +- drivers/raw/cnxk_gpio/meson.build | 2 +- drivers/raw/ntb/meson.build | 2 +- drivers/raw/skeleton/meson.build | 2 +- drivers/regex/meson.build | 2 +- drivers/regex/mlx5/meson.build | 2 +- drivers/vdpa/ifc/meson.build | 2 +- drivers/vdpa/meson.build | 3 +- drivers/vdpa/mlx5/meson.build | 2 +- drivers/vdpa/sfc/meson.build | 2 +- examples/ethtool/meson.build | 2 +- examples/l2fwd-crypto/meson.build | 2 +- examples/l3fwd/meson.build | 2 +- examples/meson.build | 11 +- examples/vm_power_manager/meson.build | 6 +- lib/argparse/meson.build | 2 +- lib/bbdev/meson.build | 2 +- lib/bitratestats/meson.build | 2 +- lib/bpf/meson.build | 2 +- lib/cmdline/meson.build | 2 +- lib/compressdev/meson.build | 2 +- lib/cryptodev/meson.build | 2 +- lib/dispatcher/meson.build | 2 +- lib/distributor/meson.build | 2 +- lib/dmadev/meson.build | 2 - lib/eal/meson.build | 5 +- lib/efd/meson.build | 2 +- lib/ethdev/meson.build | 2 +- lib/eventdev/meson.build | 3 +- lib/fib/meson.build | 2 +- lib/gpudev/meson.build | 2 +- lib/graph/meson.build | 2 +- lib/gro/meson.build | 2 +- lib/gso/meson.build | 2 +- lib/hash/meson.build | 4 +- lib/ip_frag/meson.build | 2 +- lib/ipsec/meson.build | 2 +- lib/kvargs/meson.build | 2 +- lib/latencystats/meson.build | 2 +- lib/lpm/meson.build | 3 +- lib/mbuf/meson.build | 2 +- lib/member/meson.build | 2 +- lib/mempool/meson.build | 2 +- lib/meson.build | 8 +- lib/metrics/meson.build | 2 +- lib/mldev/meson.build | 2 +- lib/net/meson.build | 2 +- lib/node/meson.build | 2 +- lib/pcapng/meson.build | 2 +- lib/pdcp/meson.build | 2 +- lib/pdump/meson.build | 2 +- lib/pipeline/meson.build | 2 +- lib/port/meson.build | 2 +- lib/power/meson.build | 2 +- lib/rawdev/meson.build | 2 - lib/rcu/meson.build | 2 +- lib/regexdev/meson.build | 2 +- lib/reorder/meson.build | 2 +- lib/rib/meson.build | 2 +- lib/ring/meson.build | 1 - lib/sched/meson.build | 2 +- lib/security/meson.build | 2 +- lib/table/meson.build | 2 +- lib/telemetry/meson.build | 2 +- lib/vhost/meson.build | 2 +- 130 files changed, 533 insertions(+), 171 deletions(-) create mode 100644 buildtools/log-deps.py create mode 100755 devtools/draw-dependency-graphs.py create mode 100755 devtools/find-duplicate-deps.py -- 2.43.5