The attached patch fixes this. ----- Forwarded message from Ben Hutchings <ben@xxxxxxxxxxxxxxx> ----- From: Ben Hutchings <ben@xxxxxxxxxxxxxxx> To: Debian Bug Tracking System <submit@xxxxxxxxxxxxxxx> Subject: Bug#970871: "modinfo -F" always shows name for built-ins Package: kmod Version: 27+20200310-2 Severity: normal Now that the kernel provides module information for potentially modular code that's actually built-in, it's possible to query these built-ins with "modinfo -F". However, this doesn't work quite right: $ modinfo -Flicense e1000e GPL v2 $ modinfo -Flicense bitrev name: bitrev GPL Ben. -- System Information: Debian Release: bullseye/sid APT prefers unstable-debug APT policy: (500, 'unstable-debug'), (500, 'stable-updates'), (500, 'unstable'), (500, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 5.8.0-1-amd64 (SMP w/2 CPU threads) Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), LANGUAGE not set Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system) LSM: AppArmor: enabled Versions of packages kmod depends on: ii libc6 2.31-3 ii libkmod2 27+20200310-2 ii liblzma5 5.2.4-1+b1 ii libssl1.1 1.1.1g-1 ii lsb-base 11.1.0 kmod recommends no packages. kmod suggests no packages. -- no debconf information ----- End forwarded message ----- -- ciao, Marco
--- a/tools/modinfo.c +++ b/tools/modinfo.c @@ -178,7 +178,11 @@ static int modinfo_do(struct kmod_module is_builtin = (filename == NULL); if (is_builtin) { - printf("%-16s%s%c", "name:", kmod_module_get_name(mod), separator); + if (field == NULL) + printf("%-16s%s%c", "name:", + kmod_module_get_name(mod), separator); + else if (field != NULL && streq(field, "name")) + printf("%s%c", kmod_module_get_name(mod), separator); filename = "(builtin)"; }