[PATCH v4 4/7] scripts/make_fit: Provide a way to add built modules
From: Simon Glass <sjg@chromium.org>
Date: 2025-10-06 23:02:51
Also in:
lkml
Subsystem:
arm64 fit support, the rest · Maintainers:
Simon Glass, Linus Torvalds
Provide arguments to support building a ramdisk from a directory tree of modules. This is a convenient way to try out a kernel with its modules. This makes use of the cpio tool rather than attempting to use a python module or our own code. The list of modules is provided in a file. Signed-off-by: Simon Glass <sjg@chromium.org> Suggested-by: Ahmad Fatoum <a.fatoum@pengutronix.de> --- Changes in v4: - Provide the list of modules from the Makefile - Reduce verbosity (don't print every module filename) Changes in v3: - Add a way to add built modules into the FIT scripts/make_fit.py | 98 +++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 90 insertions(+), 8 deletions(-)
diff --git a/scripts/make_fit.py b/scripts/make_fit.py
index 1a74a9dcd85e..3db129f40b20 100755
--- a/scripts/make_fit.py
+++ b/scripts/make_fit.py@@ -13,11 +13,17 @@ Usage: -r /boot/initrd.img-6.14.0-27-generic @arch/arm64/boot/dts/dtbs-list -E -c gzip + # Build with modules ramdisk instead of external ramdisk: + make_fit.py -A arm64 -n 'Linux-6.17' -O linux + -o arch/arm64/boot/image.fit -k /tmp/kern/arch/arm64/boot/image.itk + -m module1.ko module2.ko module3.ko @arch/arm64/boot/dts/dtbs-list + Creates a FIT containing the supplied kernel, an optional ramdisk, and a set of devicetree files, either specified individually or listed in a file (with an '@' prefix). Use -r to specify an existing ramdisk/initrd file. +Use -m to build a ramdisk from specified kernel module files. Use -E to generate an external FIT (where the data is placed after the FIT data structure). This allows parsing of the data without loading
@@ -38,6 +44,7 @@ as U-Boot, Linuxboot, Tianocore, etc. import argparse import collections import os +import shutil import subprocess import sys import tempfile
@@ -83,8 +90,14 @@ def parse_args(): help='Specifies the operating system') parser.add_argument('-k', '--kernel', type=str, required=True, help='Specifies the (uncompressed) kernel input file (.itk)') - parser.add_argument('-r', '--ramdisk', type=str, + + # Create mutually exclusive group for ramdisk options + rd_group = parser.add_mutually_exclusive_group() + rd_group.add_argument('-r', '--ramdisk', type=str, help='Specifies the ramdisk/initrd input file') + rd_group.add_argument('-m', '--modules', type=str, nargs='+', + help='List of module filenames to include in ramdisk') + parser.add_argument('-v', '--verbose', action='store_true', help='Enable verbose output') parser.add_argument('dtbs', type=str, nargs='*',
@@ -240,6 +253,60 @@ def output_dtb(fsw, seq, fname, arch, compress): fsw.property('data', compressed) +def build_ramdisk(args, tmpdir): + """Build a cpio ramdisk containing kernel modules + + Similar to mkinitramfs, this creates a compressed cpio-archive containing + the kernel modules for the current kernel version. + + Args: + args (Namespace): Program arguments + tmpdir (str): Temporary directory to use for modules installation + + Returns: + tuple: + bytes: Compressed cpio data containing modules + int: total uncompressed size + """ + suppress = None if args.verbose else subprocess.DEVNULL + + if args.verbose: + print(f'Copying {len(args.modules)} modules to ramdisk') + + # Create output-directory structure + outdir = os.path.join(tmpdir, 'initramfs') + modules_dir = os.path.join(outdir, 'usr', 'lib', 'modules') + os.makedirs(modules_dir, exist_ok=True) + + # Copy in the specified modules + for module in args.modules: + dest_path = os.path.join(modules_dir, os.path.basename(module)) + shutil.copy2(module, dest_path) + + if args.verbose: + print(f'Creating cpio archive from {outdir}') + + with tempfile.NamedTemporaryFile() as cpio_file: + # Change to initramfs directory and create cpio archive + with subprocess.Popen(['find', '.', '-print0'], cwd=outdir, + stdout=subprocess.PIPE) as find: + with subprocess.Popen(['cpio', '-o', '-0', '-H', 'newc'], + stdin=find.stdout, stdout=cpio_file, + stderr=suppress, cwd=outdir) as cpio: + find.stdout.close() + cpio.wait() + find.wait() + + if cpio.returncode != 0: + raise RuntimeError('Failed to create cpio archive') + + cpio_file.seek(0) # Reset to beginning for reading + if args.verbose: + print('Reading ramdisk...' if args.compress == 'none' else + f'Compressing ramdisk with {args.compress}...') + return compress_data(cpio_file, args.compress), cpio_file.tell() + + def process_dtb(fname, args): """Process an input DTB, decomposing it if requested and is possible
@@ -318,11 +385,12 @@ def _process_dtbs(args, fsw, entries, fdts): return seq, size -def build_fit(args): +def build_fit(args, tmpdir): """Build the FIT from the provided files and arguments Args: args (Namespace): Program arguments + tmpdir (str): Temporary directory for any temporary files Returns: tuple:
@@ -344,20 +412,29 @@ def build_fit(args): # Handle the ramdisk if provided. Compression is not supported as it is # already compressed. + ramdisk_data = None if args.ramdisk: with open(args.ramdisk, 'rb') as inf: - data = inf.read() - size += len(data) - write_ramdisk(fsw, data, args) + ramdisk_data = inf.read() + size += len(ramdisk_data) + elif args.modules: + if args.verbose: + print('Building modules ramdisk...') + ramdisk_data, uncomp_size = build_ramdisk(args, tmpdir) + size += uncomp_size + + if ramdisk_data: + write_ramdisk(fsw, ramdisk_data, args) count, fdt_size = _process_dtbs(args, fsw, entries, fdts) size += fdt_size - finish_fit(fsw, entries, bool(args.ramdisk)) + finish_fit(fsw, entries, has_ramdisk=bool(ramdisk_data)) - # Include the kernel itself in the returned file count fdt = fsw.as_fdt() fdt.pack() + + # Count FDT files, kernel, plus ramdisk if present return fdt.as_bytearray(), count + 1 + bool(args.ramdisk), size
@@ -365,7 +442,12 @@ def run_make_fit(): """Run the tool's main logic""" args = parse_args() - out_data, count, size = build_fit(args) + tmpdir = tempfile.mkdtemp(prefix='make_fit_') + try: + out_data, count, size = build_fit(args, tmpdir) + finally: + shutil.rmtree(tmpdir) + with open(args.output, 'wb') as outf: outf.write(out_data)
--
2.43.0