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 92AE145DB7; Wed, 27 Nov 2024 15:57:04 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 487AD402AB; Wed, 27 Nov 2024 15:57:04 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.19]) by mails.dpdk.org (Postfix) with ESMTP id 0EFDF4025D for ; Wed, 27 Nov 2024 15:57:02 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1732719423; x=1764255423; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding; bh=lhIy95kRWO8BCvdGGoVfNW2tjvQVfTS2119UTnZsYoE=; b=AQNB59KmyGLRzSN/z0oXJ18+8Bolp/cFA3QYBx8h5hYW1plftNzhP8et zShHg91N+S3pQGSXwigUbgMU9YPlEQCtU0YqjjUjOhWXx274KTNZDyWHb rmiMEd+57H21Y3Qw+HrOtZ/sQAFcpxm1zYkxtbzMPGgrKTB75DSfJ1DLj d1aiGNq4cP+W7RCaLzO3fGcX/M6lvxO7X3GD5vTcLUtHJW47ZQcf0ncQB yv5p4/cl/HmgQ6ZIdg7emip3ZzxNjdMctGVrK3BTrC4vKR2isxvZNEOHu ToyX5dkhfbAntLppgrBA4PNODFCwg0NIWerrXY/mGAdi1Jy+YUSkR7gEz g==; X-CSE-ConnectionGUID: ibF/LJuTTLuuU85YjOB++w== X-CSE-MsgGUID: wBC+xz9sTtOCugimddODUw== X-IronPort-AV: E=McAfee;i="6700,10204,11269"; a="32782123" X-IronPort-AV: E=Sophos;i="6.12,189,1728975600"; d="scan'208";a="32782123" Received: from orviesa005.jf.intel.com ([10.64.159.145]) by orvoesa111.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Nov 2024 06:57:02 -0800 X-CSE-ConnectionGUID: XvBdQ4qlS6eTBcB0/k7oxw== X-CSE-MsgGUID: wQ86goSVT2KJ/xK1WPFeqw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.12,189,1728975600"; d="scan'208";a="96911416" Received: from silpixa00401119.ir.intel.com ([10.55.129.167]) by orviesa005.jf.intel.com with ESMTP; 27 Nov 2024 06:57:01 -0800 From: Anatoly Burakov To: dev@dpdk.org Subject: [PATCH v5 0/8] Record and rework component dependencies Date: Wed, 27 Nov 2024 14:56:48 +0000 Message-ID: X-Mailer: git-send-email 2.43.5 In-Reply-To: References: 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. v4 -> v5: - Improve optional dependency handling by making it (mostly) automatic - Fix CI issue from accidentally changing testpmd binary filename - Explicitly encode display name vs binary name information in the graph - Reduce some more dependencies - Update all scripts to handle new graph output format v3 -> v4: - Update to latest main 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: introduce optional internal dependencies 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 | 31 ++- app/pdump/meson.build | 2 +- app/proc-info/meson.build | 6 +- app/test-bbdev/meson.build | 18 +- app/test-crypto-perf/meson.build | 6 +- app/test-fib/meson.build | 2 +- app/test-pmd/meson.build | 56 ++---- app/test-sad/meson.build | 2 +- app/test/meson.build | 14 +- buildtools/log-deps.py | 94 +++++++++ buildtools/meson.build | 2 + devtools/draw-dependency-graphs.py | 223 +++++++++++++++++++++ devtools/find-duplicate-deps.py | 62 ++++++ 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/common/qat/meson.build | 3 +- 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 | 27 ++- drivers/ml/cnxk/meson.build | 2 +- drivers/net/cnxk/meson.build | 4 +- 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/net/virtio/meson.build | 2 +- drivers/raw/cnxk_bphy/meson.build | 2 +- drivers/raw/cnxk_gpio/meson.build | 2 +- drivers/raw/gdtc/meson.build | 2 +- drivers/raw/ntb/meson.build | 2 +- drivers/raw/skeleton/meson.build | 2 +- drivers/regex/cn9k/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 | 4 +- examples/l2fwd-crypto/meson.build | 4 +- examples/l3fwd/meson.build | 4 +- examples/meson.build | 31 ++- examples/vm_power_manager/meson.build | 16 +- 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 | 4 +- 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 | 4 +- lib/mbuf/meson.build | 2 +- lib/member/meson.build | 2 +- lib/mempool/meson.build | 2 +- lib/meson.build | 27 ++- 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 | 3 +- 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 +- 135 files changed, 647 insertions(+), 232 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